Actually remove test skipping logic (fix agent hallucination)
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 <noreply@anthropic.com>
This commit is contained in:
@@ -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 () => {
|
||||
|
||||
Reference in New Issue
Block a user