test(tui): the config command group's guard tests count /connect and /disconnect

0ff4ffd494 added /connect and /disconnect to the config command group but
left the two tests that pin that group's registry on the old 17 commands.
Hosted CI has been red on Windows since then (run 38129356685, job
114440346849: "18309 tests run: 18307 passed, 2 failed"):
- config_policy_keeps_host_registry_order_metadata_and_exact_capabilities
  expected no connect/disconnect after login;
- command_ownership_contract_is_enforced expected exactly 17 commands, got 19.
The green run on ed3fbece60 (38129634138) skipped every Rust test as a light
change, so it is not evidence for this code.

Both tests now name the two commands in their registered position, and the
count is 19. The neighbouring comment that claimed the production groups hold
exactly 96 commands was already false and no longer states a number.

Checks (Ubuntu 24.04 container): nextest -p codewhale-tui --lib
-E 'test(/^commands::/)': 1192 tests run: 1177 passed, 15 failed. Both
guard tests passed. The 15 failures are all snapshot restore/undo tests that
fail on git 2.43's `checkout --end-of-options` (fixed separately; hosted CI
runs git 2.55 and is unaffected).

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01BsXVvReYca2UxzimQtj7Mj
This commit is contained in:
Claude committed 2026-10-11 11:57:53 +00:00
1 parent 2e8bc7bc7a
commit db29e64ab7
2 files changed
+6 -4

No files matched your search

@@ -16,6 +16,8 @@ fn config_policy_keeps_host_registry_order_metadata_and_exact_capabilities() {
"import-claude",
"permissions",
"login",
"connect",
"disconnect",
"auth",
"workbar",
"pet",
+4 -4
View File
@@ -1081,9 +1081,9 @@ mod tests {
has_config = true;
assert_eq!(
commands.len(),
17,
19,
"config group (group-local metadata exception) expected \
exactly 17 commands, got {}",
exactly 19 commands, got {}",
commands.len()
);
}
@@ -1111,8 +1111,8 @@ mod tests {
// Consistency: group-iterated command count must match registry.
// FEAT-015 registers one test-only contextual command (`/feat015ctx`)
// under `#[cfg(test)]` to prove the dual-path seam (D6); the nine
// production groups remain exactly 96 commands.
// under `#[cfg(test)]` to prove the dual-path seam (D6); it is not
// part of any production group.
let test_only_count = command_infos()
.iter()
.filter(|info| info.name == "feat015ctx")