-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Incompatible with unpublished thrid-party reporter #58
Comments
Mocha doesn't support this at the moment. |
@zinserjan What is mocha not supporting? I'm currently using third-party reporter I wrote, and it works fine. |
Fixed by #87.
|
I have a third-party mocha reporter would like to use, but
checkReporter
does not allow that.In
checkReporter
we have two checks,require("mocha/lib/reporters/" + reporter);
andrequire(reporter);
which applies to mocha's built-in reporter and published reporter. However unpublished reporter in my project will always fail.I think an extra
require("mocha/../../" + reporter);
to check against unpublished thrid-party reporter would be nice to have. It's a bad pattern though. Currently I have to useNODE_PATH=.
to solve this issue, which is not ideal either.The text was updated successfully, but these errors were encountered: