Mattias Åslund

socket_io_abstraction wraps emitted arguments in a single object.

1

Here is my suggestion on how to streamline the emitter in E584. This is simple enough, but it's up to coding style if you like to wrap single arguments in...

I suggest rewriting the cleanup logic to ask RealTimeServer to disconnect all open sockets before stopping the http server. This code is still somewhat a hack, because I don't quite...

RTS-stop-1 is the simplest approach to removing the function waitForConnectionCount() and my least favorable. It uses the EventEmitter from RTS-stop-2 to make RealTimeServer emit events when sockets connect and disconnect,...

This is an alternative PR to RTS-stop-3, where RealTimeServer extends the EventEmitter class we previously used in the client. Use this solution for calling the disconnectAll callback if you don't...

Since you wanted suggestions on cleanup in episode 380, here is my suggestion. I only think it is _slightly_ better, but think an array-approach to clients will give bigger benefits...

Better error handling when server fails to start during testing (see issue #7)

2

[See issue #7] This fixes the bad error message when the server port is already occupied, but it doesn't address the real problem with firefox and selenium. The firefox/selenium-bug seems...

Because of this, other tests fail to run in the same timeframe if they use the same port number.

Very long file paths

1

The file paths get very long in this project because node_modules have node-modules that ... have node_modules... :/ While I get around the limitation of that in Windows, I just...