Skip to content

Commit 837922d

Browse files
fix: revert mock in mcp-session.test.ts
1 parent 6605006 commit 837922d

1 file changed

Lines changed: 14 additions & 15 deletions

File tree

src/__tests__/CodexACPAgent/mcp-session.test.ts

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,30 @@
11
// noinspection ES6RedundantAwait
22

33
import {describe, expect, it, vi, beforeEach} from 'vitest';
4-
import {createCodexMockTestFixture, createTestSessionState, type CodexMockTestFixture} from "../acp-test-utils";
4+
import {createTestFixture, type TestFixture} from "../acp-test-utils";
5+
import type {McpServerStdio} from "@agentclientprotocol/sdk";
56

67
describe('MCP session configuration', { timeout: 40_000 }, () => {
78

8-
let fixture: CodexMockTestFixture;
9+
let fixture: TestFixture;
910
beforeEach(() => {
10-
fixture = createCodexMockTestFixture();
11+
fixture = createTestFixture();
1112
vi.clearAllMocks();
1213
});
1314

1415

1516
it('should return configured mcp', async () => {
1617
const codexAcpAgent = fixture.getCodexAcpAgent();
17-
const sessionState = createTestSessionState({
18-
sessionId: "session-id",
19-
sessionMcpServers: ["test-mcp"],
20-
});
21-
vi.spyOn(codexAcpAgent, "getSessionState").mockReturnValue(sessionState);
22-
23-
vi.spyOn(fixture.getCodexAcpClient(), "listMcpServers").mockResolvedValue({
24-
data: [],
25-
nextCursor: null,
26-
});
27-
28-
await codexAcpAgent.prompt({sessionId: "session-id", prompt: [{type: "text", text: "/mcp"}]});
18+
await codexAcpAgent.initialize({protocolVersion: 1});
19+
20+
fixture.getCodexAcpClient().authRequired = vi.fn().mockResolvedValue(false);
21+
const mcpServer: McpServerStdio = {
22+
name: "test-mcp", command: "./node_modules/.bin/mcp-hello-world", args: ["example"], env: [{name:"example", value: "example"}]
23+
};
24+
25+
const newSessionResponse = await codexAcpAgent.newSession({cwd: "", mcpServers: [mcpServer]});
26+
fixture.clearAcpConnectionDump();
27+
await codexAcpAgent.prompt({sessionId: newSessionResponse.sessionId, prompt: [{type: "text", text: "/mcp"}]});
2928
const transportDump = fixture.getAcpConnectionDump([]);
3029
expect(transportDump).contain("Configured MCP servers:");
3130
expect(transportDump).contain("- test-mcp");

0 commit comments

Comments
 (0)