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

Allow action summary group duration to be nil #36

Merged
merged 1 commit into from
May 28, 2021

Conversation

adamawolf
Copy link
Contributor

The duration field is not included in the .xcresults bundle data when a test group is composed of only tests which crashed during the test run. Requiring duration here causes the entire test group parsing to fail.

To reproduce the issue that this fixes, try creating a test suite with a single test that has a fatalError call. Process the output and you'll see that a Summary is generated for the test suite, and metrics show 1 run test, 1 failed... but the actual Group (and hence dependent test objects) will not be included in the ActionsInvocationRecord returned by getInvocationRecord().

The `duration` field is not included in the .xcresults bundle data when
a test group is composed of only tests which crashed during the test
run. Requiring `duration` here causes the entire test group parsing to
fail.
@stampede-ci stampede-ci bot added this to the Unreleased milestone May 28, 2021
@davidahouse
Copy link
Owner

Awesome, looks good to me. Thanks for finding this!

@davidahouse davidahouse merged commit 21b70ef into davidahouse:main May 28, 2021
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.

2 participants