node-osc 11.2.1 → 11.2.2

package/dist/test/test-client.js

@@ -1,14 +1,13 @@
 'use strict';
 
+var node_events = require('node:events');
 var tap = require('tap');
-var util = require('./util.js');
 var nodeOsc = require('node-osc');
 
-tap.beforeEach(util.bootstrap);
-
-tap.test('client: with array', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: with array', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -23,9 +22,10 @@ tap.test('client: with array', (t) => {
   });
 });
 
-tap.test('client: array is not mutated when sent', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: array is not mutated when sent', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(3);
 
@@ -45,9 +45,10 @@ tap.test('client: array is not mutated when sent', (t) => {
   });
 });
 
-tap.test('client: with string', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: with string', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -62,9 +63,10 @@ tap.test('client: with string', (t) => {
   });
 });
 
-tap.test('client: with Message object', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: with Message object', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -88,9 +90,10 @@ tap.test('client: with Message object', (t) => {
   });
 });
 
-tap.test('client: with Bundle object', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: with Bundle object', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -114,8 +117,8 @@ tap.test('client: with Bundle object', (t) => {
   });
 });
 
-tap.test('client: failure', (t) => {
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: failure', async (t) => {
+  const client = new nodeOsc.Client('127.0.0.1', 9999);
 
   t.plan(2);
 
@@ -130,8 +133,8 @@ tap.test('client: failure', (t) => {
   });
 });
 
-tap.test('client: close with callback', (t) => {
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: close with callback', async (t) => {
+  const client = new nodeOsc.Client('127.0.0.1', 9999);
 
   t.plan(1);
 
@@ -140,9 +143,10 @@ tap.test('client: close with callback', (t) => {
   });
 });
 
-tap.test('client: send bundle with non-numeric timetag', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: send bundle with non-numeric timetag', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -169,9 +173,10 @@ tap.test('client: send bundle with non-numeric timetag', (t) => {
   client.send(bundle);
 });
 
-tap.test('client: send bundle with null timetag', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: send bundle with null timetag', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -198,9 +203,10 @@ tap.test('client: send bundle with null timetag', (t) => {
   client.send(bundle);
 });
 
-tap.test('client: send message with float type arg', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: send message with float type arg', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -219,9 +225,10 @@ tap.test('client: send message with float type arg', (t) => {
   });
 });
 
-tap.test('client: send message with blob type arg', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: send message with blob type arg', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -240,9 +247,10 @@ tap.test('client: send message with blob type arg', (t) => {
   });
 });
 
-tap.test('client: send message with double type arg', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: send message with double type arg', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 
@@ -261,9 +269,10 @@ tap.test('client: send message with double type arg', (t) => {
   });
 });
 
-tap.test('client: send message with midi type arg', (t) => {
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('client: send message with midi type arg', async (t) => {
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 

package/dist/test/test-decode.js

@@ -107,3 +107,38 @@ tap.test('decode: malformed structure with unexpected oscType', async (t) => {
   
   t.end();
 });
+
+tap.test('decode: message without args defaults to empty array', (t) => {
+  const mockFromBuffer = () => ({
+    oscType: 'message',
+    address: '/test'
+  });
+
+  t.same(
+    decode(Buffer.from('test'), mockFromBuffer),
+    ['/test'],
+    'should default args to empty array'
+  );
+  t.end();
+});
+
+tap.test('decode: bundle element must be message or bundle', (t) => {
+  const mockFromBuffer = () => ({
+    oscType: 'bundle',
+    elements: [
+      {
+        oscType: 'message',
+        address: '/ok',
+        args: []
+      },
+      {
+        oscType: 'nope'
+      }
+    ]
+  });
+
+  t.throws(() => {
+    decode(Buffer.from('test'), mockFromBuffer);
+  }, /Malformed Packet/, 'should throw for invalid bundle element');
+  t.end();
+});

package/dist/test/test-e2e.js

@@ -1,11 +1,9 @@
 'use strict';
 
+var node_events = require('node:events');
 var tap = require('tap');
-var util = require('./util.js');
 var nodeOsc = require('node-osc');
 
-tap.beforeEach(util.bootstrap);
-
 function flaky() {
   return process.release.lts === 'Dubnium' && process.platform === 'win32';
 }
@@ -15,11 +13,12 @@ function skip(t) {
   t.end();
 }
 
-tap.test('osc: argument message no callback', (t) => {
+tap.test('osc: argument message no callback', async (t) => {
   if (flaky()) return skip(t);
 
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(1);
 
@@ -35,11 +34,12 @@ tap.test('osc: argument message no callback', (t) => {
   client.send('/test', 1, 2, 'testing');
 });
 
-tap.test('osc: client with callback and message as arguments', (t) => {
+tap.test('osc: client with callback and message as arguments', async (t) => {
   if (flaky()) return skip(t);
 
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', oscServer.port);
 
   t.plan(2);
 

package/dist/test/test-encode-decode.js

@@ -295,6 +295,102 @@ tap.test('decode: error on malformed type tags (no leading comma)', (t) => {
   t.end();
 });
 
+tap.test('decode: error on truncated blob data', (t) => {
+  const addressBuf = Buffer.from('/b\0\0', 'ascii');
+  const typeTagsBuf = Buffer.from(',b\0\0', 'ascii');
+  const lengthBuf = Buffer.alloc(4);
+  lengthBuf.writeInt32BE(4, 0);
+  const dataBuf = Buffer.from([0x01, 0x02]);
+  const buffer = Buffer.concat([addressBuf, typeTagsBuf, lengthBuf, dataBuf]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet: Not enough bytes for blob/, 'should throw when blob data is truncated');
+
+  t.end();
+});
+
+tap.test('decode: error on missing blob padding', (t) => {
+  const addressBuf = Buffer.from('/b\0\0', 'ascii');
+  const typeTagsBuf = Buffer.from(',b\0\0', 'ascii');
+  const lengthBuf = Buffer.alloc(4);
+  lengthBuf.writeInt32BE(3, 0);
+  const dataBuf = Buffer.from([0x01, 0x02, 0x03]);
+  const buffer = Buffer.concat([addressBuf, typeTagsBuf, lengthBuf, dataBuf]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet: Not enough bytes for blob padding/, 'should throw when blob padding is missing');
+
+  t.end();
+});
+
+tap.test('decode: error on truncated float32', (t) => {
+  const addressBuf = Buffer.from('/f\0\0', 'ascii');
+  const typeTagsBuf = Buffer.from(',f\0\0', 'ascii');
+  const dataBuf = Buffer.from([0x3f, 0x80, 0x00]);
+  const buffer = Buffer.concat([addressBuf, typeTagsBuf, dataBuf]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet: Not enough bytes for float32/, 'should throw when float32 data is truncated');
+
+  t.end();
+});
+
+tap.test('decode: error on truncated int32', (t) => {
+  const addressBuf = Buffer.from('/i\0\0', 'ascii');
+  const typeTagsBuf = Buffer.from(',i\0\0', 'ascii');
+  const dataBuf = Buffer.from([0x00, 0x01]);
+  const buffer = Buffer.concat([addressBuf, typeTagsBuf, dataBuf]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet: Not enough bytes for int32/, 'should throw when int32 data is truncated');
+
+  t.end();
+});
+
+tap.test('decode: error on negative blob length', (t) => {
+  const addressBuf = Buffer.from('/b\0\0', 'ascii');
+  const typeTagsBuf = Buffer.from(',b\0\0', 'ascii');
+  const lengthBuf = Buffer.alloc(4);
+  lengthBuf.writeInt32BE(-1, 0);
+  const buffer = Buffer.concat([addressBuf, typeTagsBuf, lengthBuf]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet: Invalid blob length/, 'should throw when blob length is negative');
+
+  t.end();
+});
+
+tap.test('decode: error on bundle element size overflow', (t) => {
+  const bundleHeader = Buffer.from('#bundle\0', 'ascii');
+  const timetag = Buffer.alloc(8);
+  const sizeBuf = Buffer.alloc(4);
+  sizeBuf.writeInt32BE(0, 0);
+  const buffer = Buffer.concat([bundleHeader, timetag, sizeBuf]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet/, 'should throw when bundle element size is invalid');
+
+  t.end();
+});
+
+tap.test('decode: error on truncated bundle timetag', (t) => {
+  const bundleHeader = Buffer.from('#bundle\0', 'ascii');
+  const timetag = Buffer.alloc(4);
+  const buffer = Buffer.concat([bundleHeader, timetag]);
+
+  t.throws(() => {
+    nodeOsc.decode(buffer);
+  }, /Malformed Packet: Not enough bytes for timetag/, 'should throw when timetag is truncated');
+
+  t.end();
+});
+
 tap.test('encode and decode: nested bundle with message and bundle elements', (t) => {
   // Test the else branch in encodeBundleToBuffer for message elements
   const innerBundle = new nodeOsc.Bundle(['/inner/message', 123]);

package/dist/test/test-error-handling.js

@@ -1,14 +1,13 @@
 'use strict';
 
+var node_events = require('node:events');
 var tap = require('tap');
-var util = require('./util.js');
 var nodeOsc = require('node-osc');
 
-tap.beforeEach(util.bootstrap);
-
-tap.test('server: socket error event is emitted', (t) => {
+tap.test('server: socket error event is emitted', async (t) => {
   t.plan(1);
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
   
   oscServer.on('error', (err) => {
     t.ok(err, 'error event should be emitted');
@@ -19,9 +18,10 @@ tap.test('server: socket error event is emitted', (t) => {
   oscServer._sock.emit('error', new Error('test socket error'));
 });
 
-tap.test('server: error listener can be added before listening', (t) => {
+tap.test('server: error listener can be added before listening', async (t) => {
   t.plan(2);
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
   
   oscServer.on('error', (err) => {
     t.ok(err, 'error event should be emitted');
@@ -38,7 +38,7 @@ tap.test('server: error listener can be added before listening', (t) => {
 
 tap.test('client: socket error event is emitted', (t) => {
   t.plan(1);
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+  const client = new nodeOsc.Client('127.0.0.1', 9999);
   
   client.on('error', (err) => {
     t.ok(err, 'error event should be emitted');
@@ -51,7 +51,7 @@ tap.test('client: socket error event is emitted', (t) => {
 
 tap.test('client: error listener can be added at construction', (t) => {
   t.plan(2);
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+  const client = new nodeOsc.Client('127.0.0.1', 9999);
   
   client.on('error', (err) => {
     t.ok(err, 'error event should be emitted');
@@ -68,16 +68,17 @@ tap.test('client: error listener can be added at construction', (t) => {
 
 tap.test('client: is an EventEmitter instance', (t) => {
   t.plan(1);
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+  const client = new nodeOsc.Client('127.0.0.1', 9999);
   
   t.ok(typeof client.on === 'function', 'client should have EventEmitter methods');
   
   client.close();
 });
 
-tap.test('server: multiple error listeners can be attached', (t) => {
+tap.test('server: multiple error listeners can be attached', async (t) => {
   t.plan(2);
-  const oscServer = new nodeOsc.Server(t.context.port, '127.0.0.1');
+  const oscServer = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(oscServer, 'listening');
   
   oscServer.on('error', (err) => {
     t.ok(err, 'first listener should receive error');
@@ -97,7 +98,7 @@ tap.test('server: multiple error listeners can be attached', (t) => {
 
 tap.test('client: multiple error listeners can be attached', (t) => {
   t.plan(2);
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+  const client = new nodeOsc.Client('127.0.0.1', 9999);
   
   client.on('error', (err) => {
     t.ok(err, 'first listener should receive error');