Fix Autobahn tests

[joux3]

The test files used an older API for message events.

[lpinca]

Both files can be simplified a lot. This is what I used recently:

Server

'use strict';

const WebSocket = require('..');

const wss = new WebSocket.Server({
  perMessageDeflate: { threshold: 0 },
  port: 9001
}, () => console.log('server listening on *:9001'));

wss.on('connection', (ws) => {
  ws.on('error', (err) => console.error(err.stack));
  ws.on('message', (msg) => ws.send(msg));
});

Client

'use strict';

const WebSocket = require('..');

let currentTest = 1;
let testCount;

function nextTest () {
  let ws;

  if (currentTest > testCount) {
    ws = new WebSocket('ws://localhost:9001/updateReports?&agent=ws');
    return;
  }

  console.log(`Running test case ${currentTest}/${testCount}`);

  ws = new WebSocket(`ws://localhost:9001/runCase?case=${currentTest}&agent=ws`);

  ws.on('error', (err) => console.error(err.stack));
  ws.on('message', (data) => ws.send(data));
  ws.on('close', () => {
    currentTest++;
    process.nextTick(nextTest);
  });
}

const ws = new WebSocket('ws://localhost:9001/getCaseCount');

ws.on('message', (data) => {
  testCount = +data;
});
ws.on('close', () => {
  if (testCount > 0) nextTest();
});

Since I totally forgot to do it, would you mind updating this PR to also modernize/clean up the code?

Thanks!

[3rd-Eden]

@lpinca Yeah, thats why I didn't want to merge this yet. I knew you had it working against the current code base.

[joux3]

Done. Though my files look awfully like the ones you just posted.

[lpinca]

By default Autobahn runs on port 9001, can you please use this port?

[joux3]

On the other hand the Autobahn report generator also runs on 9001. Maybe I'll add a runtime option for that while defaulting to 9001.

[lpinca]

I would drop this line, it's just useless noise.

[lpinca]

Can you merge this line with the one above?

[lpinca]

Feel free to ignore: const instead of var.

[lpinca]

Minor style nit, feel free to ignore:

const wss = new WebSocket.Server({ port }, () => {
  console.log(...);
});

[lpinca]

Thanks a lot.