-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Enable swift-testing by default. #7591
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
Conversation
This PR sets the default state of `--{enable,disable}-experimental-swift-testing` to enabled. This PR is speculative.
@swift-ci please test |
@swift-ci please test |
1 similar comment
@swift-ci please test |
Hoping to see this change made soon. Some general comments, mostly related to this issue. The current behavior is rather unintuitive. WWDC24 suggests that Swift Testing is now the preferred test framework, but these tests aren't actually executed unless the command line argument
so that the stages are a bit clearer. I understand from the forum discussion that the code is compiled, then the Swift Testing cases are executed (if the flag is specified), and then the XCTest cases are executed with the 'Executed 0 tests' message originating from the later. However, it appears that plugins are re-executed between the Swift Testing and XCTest stages which (1) feels unnecessary and (2) breaks up the results from unit testing. Finally, I appreciate that the unit tests are being performed in parallel but I wish the output from
at runtime, but would also welcome an option to generate a final test report. |
Thank you for your feedback, @Jerry-Carter. The current behaviour on macOS (where you need to specify the flag) is a known issue with Xcode 16's built-in Swift toolchain. We are working to resolve it. |
Superseded by #7766. |
This PR sets the default state of
--{enable,disable}-experimental-swift-testing
to enabled.This PR is speculative.