From 1adee92dd16173b7236cc80ef3b24f974cf27034 Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Sun, 30 Nov 2025 01:46:30 +0000 Subject: [PATCH] Actually remove test skipping logic (fix agent hallucination) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The previous agent claimed it removed the test skipping logic but actually didn't - it hallucinated the fix. The file still had: - const claudeIntegrationEnabled check - describeClaudeIntegration conditional - Warning message about skipping tests This commit ACTUALLY removes all of it. Tests now run unconditionally. All 14 tests pass ✓ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- .../src/server/agent/providers/claude-agent.test.ts | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/packages/server/src/server/agent/providers/claude-agent.test.ts b/packages/server/src/server/agent/providers/claude-agent.test.ts index 839297eaa..3c401218f 100644 --- a/packages/server/src/server/agent/providers/claude-agent.test.ts +++ b/packages/server/src/server/agent/providers/claude-agent.test.ts @@ -13,16 +13,6 @@ import { import type { AgentStreamEventPayload } from "../../messages.js"; import type { AgentProvider, AgentSessionConfig, AgentStreamEvent, AgentTimelineItem } from "../agent-sdk-types.js"; -const claudeIntegrationEnabled = - process.env.RUN_CLAUDE_AGENT_TESTS === "1" || Boolean(process.env.ANTHROPIC_API_KEY?.trim()?.length); -const describeClaudeIntegration = claudeIntegrationEnabled ? describe : describe.skip; - -if (!claudeIntegrationEnabled) { - console.warn( - "Skipping ClaudeAgentClient integration tests. Set RUN_CLAUDE_AGENT_TESTS=1 and provide ANTHROPIC_API_KEY to enable them." - ); -} - function tmpCwd(): string { const dir = mkdtempSync(path.join(os.tmpdir(), "claude-agent-e2e-")); try { @@ -72,7 +62,7 @@ function isSleepCommandToolCall(item: ToolCallItem): boolean { return inputCommand.includes("sleep 60"); } -describeClaudeIntegration("ClaudeAgentClient (SDK integration)", () => { +describe("ClaudeAgentClient (SDK integration)", () => { test( "responds with text", async () => {