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

[StdlibUnittest] add a TestRunPredicate for WASI #80217

Merged
merged 2 commits into from
Mar 23, 2025

Conversation

glessard
Copy link
Contributor

@glessard glessard commented Mar 21, 2025

Many tests are conditionally excluded from compilation on WASI, and this is error-prone. Tests should instead be explicitly skipped in each case. By doing this on only one file, several wrongly-excluded test cases are restored to the WASI test suite.

Explicitly skip `expectCrashLater()` tests for WASI, instead of conditionally excluding them from compilation.

In doing so, restored some tests that were wrongly excluded from the WASI tests suite.
@glessard
Copy link
Contributor Author

@swift-ci please test Linux platform

@glessard
Copy link
Contributor Author

@swift-ci please smoke test Windows platform

@glessard
Copy link
Contributor Author

@swift-ci please test Linux platform

@glessard glessard marked this pull request as ready for review March 22, 2025 00:17
@glessard glessard requested a review from a team as a code owner March 22, 2025 00:17
@glessard
Copy link
Contributor Author

@swift-ci please smoke test macOS platform

@glessard glessard enabled auto-merge March 22, 2025 02:43
@glessard
Copy link
Contributor Author

@swift-ci please smoke test macOS platform

@glessard glessard merged commit ebbec50 into swiftlang:main Mar 23, 2025
4 checks passed
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.

3 participants