V3

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imqueue/core",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "description": "Simple JSON-based messaging queue for inter service communication",
   "keywords": [
     "message-queue",

test/mocks/os.ts

@@ -22,7 +22,7 @@
 import os from 'node:os';
 import { mockBuiltin } from './mockBuiltin.js';
 
-export const networkInterfaces = () => {
+const defaultInterfaces = () => {
     return {
         lo: [
             {
@@ -37,4 +37,13 @@ export const networkInterfaces = () => {
     };
 };
 
+// mutable holder: the module mock binds `networkInterfaces` once at
+// registration, so tests swap the implementation through this object
+// (patching the CommonJS os object would not reach the mocked binding)
+export const networkInterfacesMock: { impl: () => any } = {
+    impl: defaultInterfaces,
+};
+
+export const networkInterfaces = () => networkInterfacesMock.impl();
+
 mockBuiltin('os', { ...os, networkInterfaces });

test/unit/UDPWorker.spec.ts

@@ -25,10 +25,10 @@ import '../mocks/index.js';
 import { describe, it, mock } from 'node:test';
 import assert from 'node:assert/strict';
 import { EventEmitter } from 'node:events';
-// import-equals keeps the live (mocked) CJS module object (unlike
-// `import * as`, which esModuleInterop turns into a copy), so patching
-// os.networkInterfaces below is observed by the code under test
-import os = require('node:os');
+// the module mock binds networkInterfaces once at registration, so tests
+// swap implementations through the mock's mutable holder — patching the
+// CommonJS os object would not reach the ESM binding the source uses
+import { networkInterfacesMock } from '../mocks/index.js';
 import { UDPWorker } from '../../src/UDPWorker.js';
 
 const OPTIONS: any = {
@@ -189,9 +189,9 @@ describe('UDPWorker', () => {
     });
 
     it('skips interface entries with no addresses', () => {
-        const original = os.networkInterfaces;
+        const original = networkInterfacesMock.impl;
 
-        (os as any).networkInterfaces = () => ({
+        networkInterfacesMock.impl = () => ({
             empty: undefined,
             lo: [
                 {
@@ -211,7 +211,7 @@ describe('UDPWorker', () => {
 
             assert.equal(worker.selectNetworkInterface(), '127.0.0.1');
         } finally {
-            (os as any).networkInterfaces = original;
+            networkInterfacesMock.impl = original;
         }
     });
 

test/unit/helpers/copyEventEmitter.spec.ts

@@ -23,10 +23,7 @@ import '../../mocks/index.js';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 import { EventEmitter } from 'node:events';
-// import-equals keeps the live CJS module object (unlike `import * as`,
-// which esModuleInterop turns into a copy), so patching util.inspect below
-// is observed by the code under test
-import util = require('node:util');
+import util from 'node:util';
 import { syncBuiltinESMExports } from 'node:module';
 import { copyEventEmitter } from '../../../src/helpers/index.js';
 
@@ -183,7 +180,6 @@ describe('copyEventEmitter()', () => {
         source.on(eventName, mockListener as any);
         copyEventEmitter(source, target);
 
-        // Restore original inspect
         (util as any).inspect = originalInspect;
         syncBuiltinESMExports();
 
@@ -194,7 +190,6 @@ describe('copyEventEmitter()', () => {
         const source = new EventEmitter();
         const target = new EventEmitter();
 
-        // Create a mock listener that looks like onceWrapper and has a truthy listener property
         let called = 0;
         const realListener = () => {
             called++;