Fix tests

Author: egor-baranov

src/__tests__/CodexACPAgent/CodexAcpClient.test.ts

@@ -66,7 +66,10 @@ describe('ACP server test', { timeout: 40_000 }, () => {
         const newSessionResponse = await codexAcpAgent.newSession({cwd: "", mcpServers: []});
         expect(newSessionResponse.sessionId).toBeDefined()
 
-        const transportDump = fixture.getCodexConnectionDump([...ignoredFields, "upgrade"]);
+        const transportDump = fixture.getCodexConnectionDump(
+            [...ignoredFields, "upgrade"],
+            { ignoreNotificationMethods: ["codex/event/mcp_startup_update"] }
+        );
         await expect(transportDump).toMatchFileSnapshot("data/auth-with-key.json");
 
         const authenticatedResponse = await fixture.getCodexAcpAgent().extMethod("authentication/status", {});
@@ -213,6 +216,7 @@ describe('ACP server test', { timeout: 40_000 }, () => {
         fixture.getCodexAppServerClient().onServerNotification = loadNotifications();
 
         const codexAcpAgent = fixture.getCodexAcpAgent();
+        await codexAcpAgent.initialize({ protocolVersion: 1 });
 
         fixture.getCodexAppServerClient().turnStart = vi.fn().mockResolvedValue({
             turn: { id: "turn-id", items: [], status: "inProgress", error: null }

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

@@ -26,8 +26,8 @@ describe('MCP session configuration', { timeout: 40_000 }, () => {
         fixture.clearAcpConnectionDump();
         await codexAcpAgent.prompt({sessionId: newSessionResponse.sessionId, prompt: [{type: "text", text: "/mcp"}]});
         const transportDump = fixture.getAcpConnectionDump([]);
-        console.log(transportDump)
-        expect(transportDump).contain("Configured MCP servers:\\n- test-mcp");
+        expect(transportDump).toContain("Configured MCP servers:");
+        expect(transportDump).toContain("- test-mcp");
     });
 
 });

src/__tests__/acp-test-utils.ts

@@ -39,7 +39,7 @@ export interface TestFixture {
     getCodexAcpAgent(): CodexAcpServer,
 
     onCodexConnectionEvent(handler: (event: CodexConnectionEvent) => void): void,
-    getCodexConnectionDump(ignoredFields: string[]): string,
+    getCodexConnectionDump(ignoredFields: string[], options?: { ignoreNotificationMethods?: string[] }): string,
     clearCodexConnectionDump(): void,
 
     onAcpConnectionEvent(handler: (event: MethodCallEvent) => void): void,
@@ -85,8 +85,14 @@ export function createBaseTestFixture(config: ConnectionConfig): TestFixture {
         getCodexAcpClient(): CodexAcpClient {
             return codexAcpClient;
         },
-        getCodexConnectionDump(ignoredFields: string[]): string {
-            return createArrayDump(transportEvents, ignoredFields);
+        getCodexConnectionDump(ignoredFields: string[], options?: { ignoreNotificationMethods?: string[] }): string {
+            const ignoredMethods = new Set(options?.ignoreNotificationMethods ?? []);
+            const filteredEvents = ignoredMethods.size === 0
+                ? transportEvents
+                : transportEvents.filter((event) =>
+                    !(event.eventType === "notification" && ignoredMethods.has(event.method))
+                );
+            return createArrayDump(filteredEvents, ignoredFields);
         },
         onCodexConnectionEvent(handler: (event: CodexConnectionEvent) => void): void {
             codexEventHandlers.push(handler);