Skip to content

Disable part of a test that fails with Swift Testing in the toolchain. #7885

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

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

grynspan
Copy link
Contributor

Remove some assertions from testSwiftTestXMLOutputWhenEmpty() that aren't strictly necessary and which have started to fail as we add Swift Testing to the toolchain.

Remove some assertions from `testSwiftTestXMLOutputWhenEmpty()` that aren't
strictly necessary and which have started to fail as we add Swift Testing to the
toolchain.
@grynspan grynspan added bug test suite improvements to SwiftPM test suite swift test Changes impacting `swift test` tool labels Aug 15, 2024
@grynspan grynspan self-assigned this Aug 15, 2024
@grynspan
Copy link
Contributor Author

@swift-ci test

@grynspan grynspan merged commit 8975fb5 into main Aug 16, 2024
5 checks passed
@grynspan grynspan deleted the jgrynspan/nerf-testSwiftTestXMLOutputWhenEmpty branch August 16, 2024 01:30
xedin added a commit that referenced this pull request Aug 24, 2024
…oo… (#7914)

…lchain. (#7901)

(cherry picked from commit 5b6542d)

**Explanation:** Disables a non-critical bit of a test that's now
failing in CI due to the addition of Swift Testing.
**Scope:** Bug fix in unit test.
**Issue:** N/A
**Original PR:**
#7885
**Risk:** Low
**Testing:** Self-testing.
**Reviewer:** @MaxDesiatov @bnbarham @rintaro

Co-authored-by: Jonathan Grynspan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug swift test Changes impacting `swift test` tool test suite improvements to SwiftPM test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants