From 5f4591f49668cdc49ab153ae76ff2b46fa3b2261 Mon Sep 17 00:00:00 2001 From: Dave Date: Wed, 18 Mar 2026 10:46:11 +0000 Subject: [PATCH] fix: update should_commit_stage test to match 5_done in COMMIT_WORTHY_STAGES The test was asserting 5_done should NOT trigger commits, but commit 74dc42c added 5_done to COMMIT_WORTHY_STAGES. Updated test to match. Co-Authored-By: Claude Opus 4.6 --- server/src/io/watcher.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/src/io/watcher.rs b/server/src/io/watcher.rs index 4ab9ea5..26a1506 100644 --- a/server/src/io/watcher.rs +++ b/server/src/io/watcher.rs @@ -938,12 +938,12 @@ mod tests { fn should_commit_stage_only_for_terminal_stages() { // Terminal stages — should commit. assert!(should_commit_stage("1_upcoming")); + assert!(should_commit_stage("5_done")); assert!(should_commit_stage("6_archived")); // Intermediate stages — broadcast-only, no commit. assert!(!should_commit_stage("2_current")); assert!(!should_commit_stage("3_qa")); assert!(!should_commit_stage("4_merge")); - assert!(!should_commit_stage("5_done")); // Unknown — no commit. assert!(!should_commit_stage("unknown")); }