metarhia / jstp

Fast RPC for browser and Node.js based on TCP, WebSocket, and MDSF
https://metarhia.github.io/jstp
Other
143 stars 10 forks source link

test: add tests for websocket transport #268

Closed nechaido closed 7 years ago

nechaido commented 7 years ago

Based on #266.

coveralls commented 7 years ago

Coverage Status

Changes Unknown when pulling 997751f5864ba62e403c3636f92488738a2a27d9 on test/add-websocket-tests into on master.

nechaido commented 7 years ago

@metarhia/jstp-core ping.

aqrln commented 7 years ago

LGTM. Landed in aea9cba60d560ae76deb244dd76dac38958f4778.

aqrln commented 7 years ago

This PR is a clear improvement, so I've landed it for now, but as a "todo", I'd prefer it to be split into two separate tests.