fix: Running a large number of tests on Windows. Closes #58

hbenl / vscode-jasmine-test-adapter

Jasmine Test Adapter for the VS Code Test Explorer

MIT License

20 stars 20 forks source link

fix: Running a large number of tests on Windows. Closes #58 #59

Closed danmana closed 2 years ago

danmana commented 2 years ago

Closes #58

hbenl commented 2 years ago

Thanks! I rewrote this patch because in my experience process.send() is reliable and so all the timeout logic seemed unnecessary.

danmana commented 2 years ago

@hbenl Thanks for merging and releasing this.

I added the timeouts because I noticed that sometimes I got Worker finished before it actually started running the tests. Maybe it was just something that happened while I was experimenting/debugging the code... I didn't test the last refactoring I did without the timeouts, most likely they were redundant.

The new version 1.8.1 works great.