dave/storkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 7 Wiki Activity

fix: update should_commit_stage test to match 5_done in COMMIT_WORTHY_STAGES

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Dave
2026-03-18 10:46:11 +00:00
parent dc7968ffbc
commit 5f4591f496
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
server/src/io/watcher.rs
View File

@@ -938,12 +938,12 @@ mod tests {
fn should_commit_stage_only_for_terminal_stages() { fn should_commit_stage_only_for_terminal_stages() {
// Terminal stages — should commit. // Terminal stages — should commit.
assert!(should_commit_stage("1_upcoming")); assert!(should_commit_stage("1_upcoming"));
assert!(should_commit_stage("5_done"));
assert!(should_commit_stage("6_archived")); assert!(should_commit_stage("6_archived"));
// Intermediate stages — broadcast-only, no commit. // Intermediate stages — broadcast-only, no commit.
assert!(!should_commit_stage("2_current")); assert!(!should_commit_stage("2_current"));
assert!(!should_commit_stage("3_qa")); assert!(!should_commit_stage("3_qa"));
assert!(!should_commit_stage("4_merge")); assert!(!should_commit_stage("4_merge"));
assert!(!should_commit_stage("5_done"));
// Unknown — no commit. // Unknown — no commit.
assert!(!should_commit_stage("unknown")); assert!(!should_commit_stage("unknown"));
} }