mirror of
https://github.com/codewhale-hq/Codewhale.git
synced 2026-11-07 17:09:13 -10:00
test(tui): the config command group's guard tests count /connect and /disconnect
0ff4ffd494added /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 oned3fbece60(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:
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",
|
||||
|
||||
@@ -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")
|
||||
|
||||
Reference in new issue
Block a user