From 87ddfafef1e2ff54fc11a39b3701f54c101d9b51 Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Wed, 13 May 2026 11:56:23 -0400 Subject: [PATCH] fix(tests): isolate bulk_skip model test from shared vault state Other tests in the workspace (e.g. secret_list_json_returns_metadata_only) write ANTHROPIC_API_KEY to the shared session daemon's vault. When that test runs before bulk_skip_exits_zero_and_prints_summary, the shared server resolves Anthropic as configured via vault env-lookup fallback, attempts to call the real Anthropic API with the leaked test value, and fails with a 401 "invalid x-api-key" instead of returning a skip. Spawn an isolated server with a fresh vault so the bulk test's "no credentials configured" precondition holds regardless of which other tests share the nextest session. Co-Authored-By: Claude Opus 4.7 (1M context) --- lib/crates/fabro-cli/tests/it/cmd/model_test.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/crates/fabro-cli/tests/it/cmd/model_test.rs b/lib/crates/fabro-cli/tests/it/cmd/model_test.rs index 6020dac3d..fc9dcbf6b 100644 --- a/lib/crates/fabro-cli/tests/it/cmd/model_test.rs +++ b/lib/crates/fabro-cli/tests/it/cmd/model_test.rs @@ -135,7 +135,12 @@ fn single_model_skip_exits_nonzero() { #[test] fn bulk_skip_exits_zero_and_prints_summary() { - let context = test_context!(); + let mut context = test_context!(); + context.write_home( + ".fabro/settings.toml", + "[server.auth]\nmethods = [\"dev-token\"]\n", + ); + context.isolated_server(); let mut cmd = context.command(); cmd.args(["model", "test"]); remove_provider_env(&mut cmd);