Skip to content

When running multiple test products, report which ones had failures at the end#9906

Merged
owenv merged 1 commit into
swiftlang:mainfrom
owenv:owenv/testoutput
Apr 10, 2026
Merged

When running multiple test products, report which ones had failures at the end#9906
owenv merged 1 commit into
swiftlang:mainfrom
owenv:owenv/testoutput

Conversation

@owenv
Copy link
Copy Markdown
Contributor

@owenv owenv commented Apr 8, 2026

Currently we emit a note at the end of the log if any XCTests failed, since those run before Swift Testing tests. However, this doesn't scale to situations where we have more than one test product. With this change, we instead emit a note like:

Note: Some test targets reported failures:                                                                                                                                                      
     - TargetATests (Swift Testing)
     -  TargetBTests (XCTest)

Which will hopefully provide a better high level overview

@owenv
Copy link
Copy Markdown
Contributor Author

owenv commented Apr 8, 2026

@swift-ci test

@owenv owenv requested review from briancroom and stmontgomery April 8, 2026 23:20
Copy link
Copy Markdown
Member

@dschaefer2 dschaefer2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like it. This will make it easier for advanced progress tracking we hope to do in the future.

@owenv owenv force-pushed the owenv/testoutput branch from 85c7d51 to 544a0cd Compare April 9, 2026 20:11
@owenv
Copy link
Copy Markdown
Contributor Author

owenv commented Apr 9, 2026

@swift-ci test

@owenv
Copy link
Copy Markdown
Contributor Author

owenv commented Apr 9, 2026

@swift-ci test windows

@owenv
Copy link
Copy Markdown
Contributor Author

owenv commented Apr 9, 2026

@swift-ci test self hosted

@owenv
Copy link
Copy Markdown
Contributor Author

owenv commented Apr 9, 2026

@swift-ci test macOS

@owenv
Copy link
Copy Markdown
Contributor Author

owenv commented Apr 10, 2026

@swift-ci test windows

@owenv owenv merged commit de20c7b into swiftlang:main Apr 10, 2026
61 checks passed
@stmontgomery stmontgomery added the swift test Changes impacting `swift test` tool label Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

swift test Changes impacting `swift test` tool

Projects

Development

Successfully merging this pull request may close these issues.

3 participants