Skip to content
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

Fix "multiple only" error with multiple outputs #534

Merged

Conversation

@rluba
Copy link
Contributor

rluba commented Feb 25, 2016

My implementation of @geek’s suggestion for reporting general test errors fails to work when you specify multiple reporters or non-console reporters.

I’ve patched it for the "multiple only" error but I assume that the original code for this kind of error reporting has the same bug (haven’t tested it).

@geek geek added the bug label Feb 25, 2016
@geek geek added this to the 10.0.1 milestone Feb 25, 2016
@geek geek self-assigned this Feb 25, 2016
geek added a commit that referenced this pull request Feb 25, 2016
…ple_outputs

Fix "multiple only" error with multiple outputs
@geek geek merged commit 06c5818 into hapijs:master Feb 25, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.