From 6527b8004b8cf5d3df6d66dd3142b07884ca36f2 Mon Sep 17 00:00:00 2001 From: fagemx Date: Fri, 27 Mar 2026 08:57:29 +0800 Subject: [PATCH] fix(bridge): fix test isolation for nudge_counter test (#348) Add fs::remove_dir_all before ensure_dirs in three dispatch tests to clean up stale state from previous runs. Without this, leftover nudge_ts files cause cooldown checks to suppress nudge emission, making the test fail with "should produce nudge" assertion. Co-Authored-By: Claude Opus 4.6 (1M context) --- crates/edda-bridge-claude/src/dispatch/tests.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/crates/edda-bridge-claude/src/dispatch/tests.rs b/crates/edda-bridge-claude/src/dispatch/tests.rs index f309e93..6ddb26f 100644 --- a/crates/edda-bridge-claude/src/dispatch/tests.rs +++ b/crates/edda-bridge-claude/src/dispatch/tests.rs @@ -928,6 +928,7 @@ fn post_tool_use_after_decide_cooldown_still_applies() { fn post_tool_use_cooldown_suppresses() { let pid = "test_nudge_cooldown"; let sid = "sess-nudge-3"; + let _ = fs::remove_dir_all(edda_store::project_dir(pid)); let _ = edda_store::ensure_dirs(pid); // First commit → nudge @@ -1046,6 +1047,7 @@ fn counter_increment_and_read() { fn post_tool_use_increments_nudge_counter() { let pid = "test_nudge_counter"; let sid = "sess-nudge-cnt-1"; + let _ = fs::remove_dir_all(edda_store::project_dir(pid)); let _ = edda_store::ensure_dirs(pid); // First commit → nudge emitted → counter = 1 @@ -1080,6 +1082,7 @@ fn post_tool_use_increments_nudge_counter() { fn post_tool_use_increments_decide_counter() { let pid = "test_decide_counter"; let sid = "sess-decide-cnt-1"; + let _ = fs::remove_dir_all(edda_store::project_dir(pid)); let _ = edda_store::ensure_dirs(pid); let raw = serde_json::json!({