node-osc 11.2.0 → 11.2.2

package/dist/test/test-bundle.js

@@ -1,14 +1,13 @@
 'use strict';
 
+var node_events = require('node:events');
 var tap = require('tap');
 var nodeOsc = require('node-osc');
-var util = require('./util.js');
 
-tap.beforeEach(util.bootstrap);
-
-tap.test('bundle: verbose bundle', (t) => {
-  const server = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('bundle: verbose bundle', async (t) => {
+  const server = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(server, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', server.port);
 
   t.plan(2);
 
@@ -35,9 +34,10 @@ tap.test('bundle: verbose bundle', (t) => {
   }));
 });
 
-tap.test('bundle: array syntax', (t) => {
-  const server = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('bundle: array syntax', async (t) => {
+  const server = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(server, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', server.port);
 
   t.plan(2);
 
@@ -57,9 +57,10 @@ tap.test('bundle: array syntax', (t) => {
   ));
 });
 
-tap.test('bundle: nested bundle', (t) => {
-  const server = new nodeOsc.Server(t.context.port, '127.0.0.1');
-  const client = new nodeOsc.Client('127.0.0.1', t.context.port);
+tap.test('bundle: nested bundle', async (t) => {
+  const server = new nodeOsc.Server(0, '127.0.0.1');
+  await node_events.once(server, 'listening');
+  const client = new nodeOsc.Client('127.0.0.1', server.port);
 
   t.plan(4);
 

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,33 @@ tap.test('client: with array', (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: 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);
+
+  const originalArray = ['/test', 0, 1, 'testing', true];
+  const expectedArray = ['/test', 0, 1, 'testing', true];
+
+  oscServer.on('message', (msg) => {
+    oscServer.close();
+    t.same(msg, ['/test', 0, 1, 'testing', true], 'We should receive expected payload');
+    // Verify the original array was not mutated
+    t.same(originalArray, expectedArray, 'Original array should not be mutated');
+  });
+
+  client.send(originalArray, (err) => {
+    t.error(err, 'there should be no error');
+    client.close();
+  });
+});
+
+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);
 
@@ -40,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);
 
@@ -66,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);
 
@@ -92,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);
 
@@ -108,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);
 
@@ -118,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);
 
@@ -147,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);
 
@@ -176,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);
 
@@ -197,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);
 
@@ -218,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);
 
@@ -239,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);