From 8eb4f8bcd8e6144854d7585b16027c92f712f832 Mon Sep 17 00:00:00 2001 From: siddseethepalli Date: Wed, 15 Apr 2026 04:47:47 +0000 Subject: [PATCH] fix(tests): keep email-list hermetic; don't delete real secure-store credentials --- assistant/src/cli/commands/__tests__/email-list.test.ts | 5 ----- 1 file changed, 5 deletions(-) diff --git a/assistant/src/cli/commands/__tests__/email-list.test.ts b/assistant/src/cli/commands/__tests__/email-list.test.ts index 4edc02908ec..c073be8e967 100644 --- a/assistant/src/cli/commands/__tests__/email-list.test.ts +++ b/assistant/src/cli/commands/__tests__/email-list.test.ts @@ -17,7 +17,6 @@ import { runAssistantCommand } from "../../__tests__/run-assistant-command.js"; const ASSISTANT_ID = "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee"; const API_KEY_CREDENTIAL = credentialKey("vellum", "assistant_api_key"); -const ASSISTANT_ID_CREDENTIAL = credentialKey("vellum", "platform_assistant_id"); /** * Return the recorded fetch calls, excluding the feature-flag fetch that @@ -84,10 +83,6 @@ beforeEach(async () => { _setOverridesForTesting({ "email-channel": true }); setPlatformAssistantId(ASSISTANT_ID); await setSecureKeyAsync(API_KEY_CREDENTIAL, "test-api-key"); - // Ensure the credential store does not contain a stray platform_assistant_id - // from dev machine state — the "missing assistant ID" test relies on the - // fallback lookup returning empty. - await deleteSecureKeyAsync(ASSISTANT_ID_CREDENTIAL); }); afterEach(() => {