mocha-parallel-tests icon indicating copy to clipboard operation
mocha-parallel-tests copied to clipboard

Doesn't work with --watch?

Open laggingreflex opened this issue 9 years ago • 6 comments

It seems it doesn't work with --watch. It exits after the first run, doesn't actually wait for file changes...

[node v6.9.0, Win10]

Awesome lib btw!

laggingreflex avatar Oct 25 '16 15:10 laggingreflex

@mmotkina do you have time for this issue?

1999 avatar Dec 06 '16 17:12 1999

Any update on this? I tried 2.0.5 and --watch still doesn't do anything

KristjanTammekivi avatar Nov 20 '18 13:11 KristjanTammekivi

Frankly speaking, I don't see any profit of supporting --watch in mocha-parallel-tests. We can do this but it will not be more performant compared to mocha because only changed files will be executed. That's why this issue's priority is low. What's your use case where you need --watch support?

1999 avatar Nov 21 '18 01:11 1999

At the time of creating this issue, I think I just didn't want to have 2 test libraries. But it seems now that I could totally have had both, since mocha is a peer dependency (I guess I didn't realize that then) and works almost identically (uses same flags and everything), so keeping to use it just for --watch wouldn't have been a bad idea either.

laggingreflex avatar Nov 21 '18 03:11 laggingreflex

My usecase is executing DB integration tests. I think I can solve this with nodemon when need be and mostly rely on mocha to just execute one specific suite so it's not too much of a blocker. I just saw in the #126 that watch had a checkmark on it and I was hoping it's ready on a branch somewhere.

KristjanTammekivi avatar Nov 21 '18 07:11 KristjanTammekivi

Gotcha, that was close but not it.

1999 avatar Nov 21 '18 08:11 1999