feat: add more tests to improve coverage

Author: Mikhus

test/UDPClusterManager.parseAndStart.spec.ts

@@ -0,0 +1,38 @@
+/*!
+ * UDPClusterManager parseBroadcastedMessage and startListening branch tests
+ */
+import './mocks';
+import { expect } from 'chai';
+import { UDPClusterManager } from '../src';
+
+/**
+ * Covers default parameters in startListening(options = {}) and
+ * default destructuring for address = '' and timeout = '0' in
+ * parseBroadcastedMessage().
+ */
+describe('UDPClusterManager parse/start branches', () => {
+    it('parseBroadcastedMessage: should apply defaults for empty address and timeout', () => {
+        const parse = (UDPClusterManager as any).parseBroadcastedMessage as Function;
+        const buf = Buffer.from(['name', 'id', 'UP'].join('\t'));
+        const msg = parse(buf);
+        expect(msg).to.include({ name: 'name', id: 'id', type: 'up' });
+        // address default => '' leads to host '' and port NaN
+        expect(msg.host).to.equal('');
+        expect(Number.isNaN(msg.port)).to.equal(true);
+        // timeout default '0' => 0 ms
+        expect(msg.timeout).to.equal(0);
+    });
+
+    it('startListening: should call listenBroadcastedMessages when called without options', () => {
+        const mgr: any = new (UDPClusterManager as any)();
+        let called = false;
+        const original = mgr.listenBroadcastedMessages;
+        mgr.listenBroadcastedMessages = (..._args: any[]) => { called = true; };
+        try {
+            (mgr as any).startListening();
+            expect(called).to.equal(true);
+        } finally {
+            mgr.listenBroadcastedMessages = original;
+        }
+    });
+});

test/profile.decorator.branches.spec.ts

@@ -0,0 +1,34 @@
+/*!
+ * Additional tests to cover remaining branches in profile decorator
+ */
+import './mocks';
+import { expect } from 'chai';
+import { profile, LogLevel } from '..';
+
+// Note: We intentionally call decorated methods without a "this" context
+// to exercise the (this || target) branches inside the decorator wrapper.
+
+describe('profile decorator extra branches', () => {
+    it('should return early via original.apply(target, ...) when both debug flags are false and this is undefined', () => {
+        class T1 {
+            @profile({ enableDebugTime: false, enableDebugArgs: false, logLevel: LogLevel.LOG })
+            public m(...args: any[]) { return args; }
+        }
+        const o = new T1();
+        const fn = Object.getPrototypeOf(o).m as Function; // wrapper
+        const res = fn.call(undefined, 1, 2, 3);
+        expect(res).to.deep.equal([1, 2, 3]);
+    });
+
+    it('should execute debug path with (this || target) picking target and logLevel fallback to IMQ_LOG_LEVEL', () => {
+        class T2 {
+            // no logger on prototype; calling with undefined this picks target
+            @profile({ enableDebugTime: true, enableDebugArgs: true, logLevel: undefined as any })
+            public m(..._args: any[]) { /* noop */ }
+        }
+        const o = new T2();
+        const fn = Object.getPrototypeOf(o).m as Function; // wrapper
+        // provide serializable args to avoid logger.error path when logger is undefined
+        expect(() => fn.call(undefined, 1, { a: 2 }, 'x')).to.not.throw();
+    });
+});