Skip to content

Commit d84b24d

Browse files
committed
test: remove notification-method filtering from codex dump
1 parent c3a7be3 commit d84b24d

1 file changed

Lines changed: 3 additions & 9 deletions

File tree

src/__tests__/acp-test-utils.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export interface TestFixture {
4040
getCodexAcpAgent(): CodexAcpServer,
4141

4242
onCodexConnectionEvent(handler: (event: CodexConnectionEvent) => void): void,
43-
getCodexConnectionDump(ignoredFields: string[], options?: { ignoreNotificationMethods?: string[] }): string,
43+
getCodexConnectionDump(ignoredFields: string[]): string,
4444
clearCodexConnectionDump(): void,
4545

4646
onAcpConnectionEvent(handler: (event: MethodCallEvent) => void): void,
@@ -86,14 +86,8 @@ export function createBaseTestFixture(config: ConnectionConfig): TestFixture {
8686
getCodexAcpClient(): CodexAcpClient {
8787
return codexAcpClient;
8888
},
89-
getCodexConnectionDump(ignoredFields: string[], options?: { ignoreNotificationMethods?: string[] }): string {
90-
const ignoredMethods = new Set(options?.ignoreNotificationMethods ?? []);
91-
const filteredEvents = ignoredMethods.size === 0
92-
? transportEvents
93-
: transportEvents.filter((event) =>
94-
!(event.eventType === "notification" && ignoredMethods.has(event.method))
95-
);
96-
return createArrayDump(filteredEvents, ignoredFields);
89+
getCodexConnectionDump(ignoredFields: string[]): string {
90+
return createArrayDump(transportEvents, ignoredFields);
9791
},
9892
onCodexConnectionEvent(handler: (event: CodexConnectionEvent) => void): void {
9993
codexEventHandlers.push(handler);

0 commit comments

Comments
 (0)