From 959755cd6ef7c6bb9fca2c83dec56851abad3818 Mon Sep 17 00:00:00 2001 From: Dave Date: Thu, 19 Feb 2026 14:47:19 +0000 Subject: [PATCH] Fix post-merge: pass None for coverage arg in to_review_story tests Story 27 added a coverage parameter to to_review_story(). Story 29's tests called the old 2-arg signature. Pass None to fix compilation. Co-Authored-By: Claude Opus 4.6 --- server/src/http/workflow.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/server/src/http/workflow.rs b/server/src/http/workflow.rs index e30440f..82d9b2e 100644 --- a/server/src/http/workflow.rs +++ b/server/src/http/workflow.rs @@ -511,7 +511,7 @@ mod tests { }], }; - let review = to_review_story("story-29", &results); + let review = to_review_story("story-29", &results, None); assert!(review.can_accept); assert!(review.reasons.is_empty()); assert!(review.missing_categories.is_empty()); @@ -530,7 +530,7 @@ mod tests { integration: vec![], }; - let review = to_review_story("story-29", &results); + let review = to_review_story("story-29", &results, None); assert!(!review.can_accept); assert!(review.missing_categories.contains(&"integration".to_string())); } @@ -550,7 +550,7 @@ mod tests { }], }; - let review = to_review_story("story-29", &results); + let review = to_review_story("story-29", &results, None); assert!(!review.can_accept); assert_eq!(review.summary.failed, 1); }