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

Update JunitReporter to parse parallel tests #84

Merged
merged 3 commits into from Nov 2, 2022

Conversation

welshm
Copy link
Contributor

@welshm welshm commented Oct 28, 2022

  • Change JunitComponent to capture the Regex used to create the component
  • Relies on capture groups being consistent between parallel and non-parallel output
  • Add unit test

- Change `JunitComponent` to capture the Regex used to create the component
- Relies on capture groups being consistent between parallel and non-parallel output
- Add unit test
@welshm
Copy link
Contributor Author

welshm commented Oct 28, 2022

Fixes #83

@welshm
Copy link
Contributor Author

welshm commented Oct 28, 2022

@thii Let me know the best way to get this added and/or relased

@thii
Copy link
Contributor

thii commented Oct 29, 2022

cc @acecilia

@welshm
Copy link
Contributor Author

welshm commented Nov 1, 2022

@thii @acecilia we'd really like to get this in so that our Jenkins runs can report failing/passing tests from parallel executions.

I haven't found any other test parsers that handle parallel test runners yet, so I think this would be a pretty good addition to xcbeautify

@thii thii merged commit 79d9a8f into cpisciotta:main Nov 2, 2022
@thii
Copy link
Contributor

thii commented Nov 2, 2022

@welshm This is included in https://github.com/tuist/xcbeautify/releases/tag/0.16.0.

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