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

Report jest-junit testsuite errors as failures #155

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

johnbartholomew
Copy link

Test errors are different (represented differently in the JUnit XML output from jest-junit) to test failures. Failures are tests which ran and failed, errors are for tests/test suites which did not even run because the test code itself didn't build or didn't execute correctly.

jest-junit has an option to enable reporting of test suite errors (JEST_JUNIT_REPORT_TEST_SUITE_ERRORS / reportTestSuiteErrors). This PR modifies the test-reporter to treat test suite errors as failures, instead of treating them as successful.

Test errors are different (represented differently in the JUnit XML
output from jest-junit) to test _failures_. Failures are tests
which ran and failed, errors are for tests/test suites which
did not even run because the test code itself didn't build
or didn't execute correctly.

jest-junit has an option to enable reporting of test suite errors,
but test-reporter then interprets these as successful tests.
@IanMoroney
Copy link
Contributor

@johnbartholomew , I ran into a similar issue as yourself where there were features that I needed from this project and looked like it wasn't being updated any time soon, so I took the initiative to fork the project myself and build the enhancements into it with a colleague.
I have also now taken this PR and added the contents into that repo so you can use them :)

https://github.com/phoenix-actions/test-reporting/
https://github.com/marketplace/actions/test-reporting

@johnbartholomew
Copy link
Author

Thanks for collecting these improvements together!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants