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 exit code when multiple reporters encounter a failure #406

Merged
merged 1 commit into from Jul 27, 2015

Conversation

@hofan41
Copy link
Contributor

hofan41 commented Jul 14, 2015

Closes #393 .

We need to open a new issue, however, when a test throws an exception, since I still have not figured out how to solve it.

…zero process exit code.

Added unit tests to enforce.
geek added a commit that referenced this pull request Jul 27, 2015
Fix exit code when multiple reporters encounter a failure
@geek geek merged commit fb2960a into hapijs:master Jul 27, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@geek geek added the bug label Jul 27, 2015
@geek geek added this to the 5.14.0 milestone Jul 27, 2015
@geek geek self-assigned this Jul 27, 2015
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.