mocha-parallel-tests
mocha-parallel-tests copied to clipboard
reuse processes using a process pool
Wow, seems like a huge work has been made here! 😮
I've created my own, much simpler package, that replaces mocha-parallel-tests and is able to reuse child processes: https://www.npmjs.com/package/mochallel
@fabiosantoscode closing this PR as non-active. If you plan to update it feel free to re-open and ping me :)
Sorry. Can you reopen? I've just pushed a removal of that throw statement.
I've released an alternative to this module that already does process reusing. It's quite fast! Can you match its speed? http://npmjs.com/mochallel
@1999
@fabiosantoscode I noticed your message above. It's great that it's so fast 👍 Can it be used in mocha-parallel-tests
somehow or what do you mean by that?
Also it looks like the build is red, so you also need to fix the errors before we can merge this PR.
You can use the multiprocess-map module that I created to use in mochallel, if you like
I meant it as a challenge :) if you can make a faster mochallel, then everybody wins!
The builds are timing out -- Do you think you know what might be causing it?
Ok, I think I fixed it.
You're almost there: 1 test is failing for node@11.
Nice job! Can you think of any tests that we might add here and which might be useful for this PR?
No, I don't think so.
Why not? This PR is a huge improvement to the way mocha-parallel-tests
works. For instance if one of the tests is hanging this will stop others from re-using the same process (see #155 for more info). I would like to see all changes covered with tests and here we could for instance check that the process.pid is the same across different files.
OK, let's do that test.
Done @1999
Done @1999
Sorry, done what?
Nevermind
Implemented all the ideas for tests. Any more tests you can think of?
I think you can merge now