test: fix bug in logging assertions

Author: Codex-

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();

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(