test: fix bug in logging assertions

Codex- · Oct 27, 2024 · e2c1b11 · e2c1b11
1 parent d587c66
commit e2c1b11
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/return-dispatch.spec.ts b/src/return-dispatch.spec.ts
@@ -674,7 +674,7 @@ describe("return-dispatch", () => {
       expect(apiRetryOrTimeoutMock).toHaveBeenCalledOnce();
       await vi.advanceTimersByTimeAsync(1); // deplete queue
 
-      assertOnlyCalled(coreDebugLogMock, coreInfoLogMock);
+      assertOnlyCalled(coreInfoLogMock);
 
       expect(coreInfoLogMock).toHaveBeenCalledOnce();
       expect(coreInfoLogMock.mock.calls[0]?.[0]).toMatchSnapshot();

diff --git a/src/test-utils/logging.mock.ts b/src/test-utils/logging.mock.ts
@@ -74,11 +74,13 @@ function assertOnlyCalledInner(
   // Once Node 22 is LTS, this can be:
   // const diff = coreLogMockSet.symmetricDifference(new Set(coreLogMocks));
 
-  const diff = symDiff([[...coreLogMockSet], coreLogMocks]);
-
-  for (const logMock of diff) {
+  const notCalled = symDiff([[...coreLogMockSet], coreLogMocks]);
+  for (const logMock of notCalled) {
     expect(logMock).not.toHaveBeenCalled();
   }
+  for (const logMock of coreLogMocks) {
+    expect(logMock).toHaveBeenCalled();
+  }
 }
 
 function assertNoneCalledInner(