Skip CLI-dependent tests in CI
This commit is contained in:
@@ -18,12 +18,6 @@ import { isInitializeRequest } from "@modelcontextprotocol/sdk/types.js";
|
|||||||
import { createTestLogger } from "../../../test-utils/test-logger.js";
|
import { createTestLogger } from "../../../test-utils/test-logger.js";
|
||||||
import { ClaudeAgentClient, convertClaudeHistoryEntry } from "./claude-agent.js";
|
import { ClaudeAgentClient, convertClaudeHistoryEntry } from "./claude-agent.js";
|
||||||
import { useTempClaudeConfigDir } from "../../test-utils/claude-config.js";
|
import { useTempClaudeConfigDir } from "../../test-utils/claude-config.js";
|
||||||
import {
|
|
||||||
hydrateStreamState,
|
|
||||||
type StreamItem,
|
|
||||||
type AgentToolCallData,
|
|
||||||
isAgentToolCallItem,
|
|
||||||
} from "../../../../../app/src/types/stream.js";
|
|
||||||
import type { AgentStreamEventPayload } from "../../messages.js";
|
import type { AgentStreamEventPayload } from "../../messages.js";
|
||||||
import type {
|
import type {
|
||||||
AgentProvider,
|
AgentProvider,
|
||||||
@@ -41,6 +35,9 @@ const createHTTPServer = createServer;
|
|||||||
const hasClaudeCredentials =
|
const hasClaudeCredentials =
|
||||||
!!process.env.CLAUDE_SESSION_TOKEN || !!process.env.ANTHROPIC_API_KEY;
|
!!process.env.CLAUDE_SESSION_TOKEN || !!process.env.ANTHROPIC_API_KEY;
|
||||||
|
|
||||||
|
type StreamItem = any;
|
||||||
|
type AgentToolCallData = any;
|
||||||
|
|
||||||
function tmpCwd(): string {
|
function tmpCwd(): string {
|
||||||
const dir = mkdtempSync(path.join(os.tmpdir(), "claude-agent-e2e-"));
|
const dir = mkdtempSync(path.join(os.tmpdir(), "claude-agent-e2e-"));
|
||||||
try {
|
try {
|
||||||
@@ -231,6 +228,10 @@ async function startAgentMcpServer(): Promise<AgentMcpServerHandle> {
|
|||||||
"ClaudeAgentClient (SDK integration)",
|
"ClaudeAgentClient (SDK integration)",
|
||||||
() => {
|
() => {
|
||||||
const logger = createTestLogger();
|
const logger = createTestLogger();
|
||||||
|
let hydrateStreamState: (updates: unknown[]) => unknown = () => {
|
||||||
|
throw new Error("hydrateStreamState not initialized");
|
||||||
|
};
|
||||||
|
let isAgentToolCallItem: (item: unknown) => boolean = () => false;
|
||||||
let agentMcpServer: AgentMcpServerHandle;
|
let agentMcpServer: AgentMcpServerHandle;
|
||||||
let restoreClaudeConfigDir: (() => void) | null = null;
|
let restoreClaudeConfigDir: (() => void) | null = null;
|
||||||
const buildConfig = (
|
const buildConfig = (
|
||||||
@@ -253,6 +254,11 @@ async function startAgentMcpServer(): Promise<AgentMcpServerHandle> {
|
|||||||
beforeAll(() => {
|
beforeAll(() => {
|
||||||
restoreClaudeConfigDir = useTempClaudeConfigDir();
|
restoreClaudeConfigDir = useTempClaudeConfigDir();
|
||||||
});
|
});
|
||||||
|
beforeAll(async () => {
|
||||||
|
const stream = await import("../../../../../app/src/types/stream.js");
|
||||||
|
hydrateStreamState = stream.hydrateStreamState as typeof hydrateStreamState;
|
||||||
|
isAgentToolCallItem = stream.isAgentToolCallItem as typeof isAgentToolCallItem;
|
||||||
|
});
|
||||||
beforeAll(async () => {
|
beforeAll(async () => {
|
||||||
agentMcpServer = await startAgentMcpServer();
|
agentMcpServer = await startAgentMcpServer();
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -2,6 +2,7 @@ import { beforeAll, describe, expect, test, vi } from "vitest";
|
|||||||
import { mkdtempSync, realpathSync, rmSync, writeFileSync } from "node:fs";
|
import { mkdtempSync, realpathSync, rmSync, writeFileSync } from "node:fs";
|
||||||
import os from "node:os";
|
import os from "node:os";
|
||||||
import path from "node:path";
|
import path from "node:path";
|
||||||
|
import { execFileSync } from "node:child_process";
|
||||||
|
|
||||||
import { createTestLogger } from "../../../test-utils/test-logger.js";
|
import { createTestLogger } from "../../../test-utils/test-logger.js";
|
||||||
import { OpenCodeAgentClient } from "./opencode-agent.js";
|
import { OpenCodeAgentClient } from "./opencode-agent.js";
|
||||||
@@ -74,7 +75,18 @@ async function collectTurnEvents(
|
|||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
||||||
describe("OpenCodeAgentClient", () => {
|
function isBinaryInstalled(binary: string): boolean {
|
||||||
|
try {
|
||||||
|
const out = execFileSync("which", [binary], { encoding: "utf8" }).trim();
|
||||||
|
return out.length > 0;
|
||||||
|
} catch {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const hasOpenCode = isBinaryInstalled("opencode");
|
||||||
|
|
||||||
|
(hasOpenCode ? describe : describe.skip)("OpenCodeAgentClient", () => {
|
||||||
const logger = createTestLogger();
|
const logger = createTestLogger();
|
||||||
const buildConfig = (cwd: string): AgentSessionConfig => ({
|
const buildConfig = (cwd: string): AgentSessionConfig => ({
|
||||||
provider: "opencode",
|
provider: "opencode",
|
||||||
|
|||||||
Reference in New Issue
Block a user