This repository was archived by the owner on Sep 10, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Sep 10, 2026. It is now read-only.
Can't use mocha_test on non-root packages #37
Description
Hi @pcj, I think I found an issue with mocha_test. It appears it doesn't work for packages deeper than the workspace root.
To reproduce, simply:
$ cd tests/mocha
$ mkdir W
$ mv BUILD test.js W
$ bazel test //...
It appears the mocha launcher doesn't take the leading path into account. Please see the attachment for a solution which I verified is working both for the base package and deeper ones.
0001-Make-sure-mocha_test-can-be-used-on-other-packages-t.patch.txt
Activity
xujihui1985 commentedon Oct 24, 2017
@driveraid this issue should has been fixed #39
cwoodcock commentedon Oct 25, 2017
Just ran into this using head @ f990afc. Seems #41 accidentally added a regression. Backing the truck up to dc9f8ba worked around it for me. cc @pcj
pcj commentedon Oct 25, 2017
Shucks, sorry about that. #41 was a big change. I'll take a look at this tonight.
pcj commentedon Oct 26, 2017
Should be fixed by #42