search

metarhia / jstp

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

test: fix flaky tests #363

Closed nechaido closed 6 years ago

nechaido commented 6 years ago

This should fix the https://github.com/metarhia/jstp/issues/361.

It appears to me that child process was trying to send messages to master process, but at the time, IPC channel was closed. This fix ensures that server is closed after child process and master finished exchanging messages.

nechaido commented 6 years ago

@tshemsedinov we need your answer ASAP as this tests may cause other PR to fail.

belochub commented 6 years ago

Landed in https://github.com/metarhia/jstp/commit/90ebf02068538557098debf1fe13b7969aaeb347.