pote
commented
9 years ago Nice! yeah, the test suite did need a lot of improvement (and probably still does), this certainly helps.
There's a sidenote here: Ruby's timeout is dangerous and can't be trusted, this shouldn't affect our use case considering the test process is short-lived, but worth noting nonetheless. :)
Merging, thanks for the PR @foca :heart: :tada:
Cleans up the tests and makes them easier to read, and fixes a race condition that's preventing me from actually running the tests.
cc @pote