server: ensure host argument is used (#99)

Fixes: #98

lib/Server.mjs

@@ -4,8 +4,12 @@ import { EventEmitter } from 'node:events';
 import decode from './internal/decode.mjs';
 
 class Server extends EventEmitter {
-  constructor(port, host, cb) {
+  constructor(port, host='127.0.0.1', cb) {
     super();
+    if (typeof host === 'function') {
+      cb = host;
+      host = '127.0.0.1';
+    }
     if (!cb) cb = () => {};
     let decoded;
     this.port = port;
@@ -14,7 +18,7 @@ class Server extends EventEmitter {
       type: 'udp4',
       reuseAddr: true
     });
-    this._sock.bind(port);
+    this._sock.bind(port, host);
     this._sock.on('listening', () => {
       this.emit('listening');
       cb();

test/test-server.mjs

@@ -13,8 +13,32 @@ test('server: create and close', (t) => {
   });
 });
 
-test('client: listen to message', (t) => {
-  const oscServer = new Server(t.context.port, '127.0.0.1');
+test('server: listen to message', (t) => {
+  const oscServer = new Server(t.context.port);
+  const client = new Client('127.0.0.1', t.context.port);
+
+  t.plan(3);
+
+  t.teardown(() => {
+    oscServer.close();
+    client.close();
+  });
+
+  oscServer.on('message', (msg) => {
+    t.same(msg, ['/test'], 'We should receive expected payload');
+  });
+
+  oscServer.on('/test', (msg) => {
+    t.same(msg, ['/test'], 'We should receive expected payload');
+  });
+
+  client.send('/test', (err) => {
+    t.error(err, 'there should be no error');
+  });
+});
+
+test('server: no defined host', (t) => {
+  const oscServer = new Server(t.context.port);
   const client = new Client('127.0.0.1', t.context.port);
 
   t.plan(3);
@@ -37,6 +61,31 @@ test('client: listen to message', (t) => {
   });
 });
 
+test('server: callback as second arg', (t) => {
+  t.plan(4);
+  const oscServer = new Server(t.context.port, () => {
+    t.ok('callback called');
+  });
+  const client = new Client('127.0.0.1', t.context.port);
+
+  t.teardown(() => {
+    oscServer.close();
+    client.close();
+  });
+
+  oscServer.on('message', (msg) => {
+    t.same(msg, ['/test'], 'We should receive expected payload');
+  });
+
+  oscServer.on('/test', (msg) => {
+    t.same(msg, ['/test'], 'We should receive expected payload');
+  });
+
+  client.send('/test', (err) => {
+    t.error(err, 'there should be no error');
+  });
+});
+
 test('server: bad message', (t) => {
   t.plan(2);
   const oscServer = new Server(t.context.port, '127.0.0.1');