Skip to content

[wasm][coreclr] Enable passing coreclr tests#125496

Merged
radekdoulik merged 5 commits intodotnet:mainfrom
radekdoulik:clr-wasm-enable-passing-library-tests
Mar 16, 2026
Merged

[wasm][coreclr] Enable passing coreclr tests#125496
radekdoulik merged 5 commits intodotnet:mainfrom
radekdoulik:clr-wasm-enable-passing-library-tests

Conversation

@radekdoulik
Copy link
Member

No description provided.

@radekdoulik radekdoulik added this to the Future milestone Mar 12, 2026
@radekdoulik radekdoulik added arch-wasm WebAssembly architecture area-Infrastructure-libraries os-browser Browser variant of arch-wasm labels Mar 12, 2026
Copilot AI review requested due to automatic review settings March 12, 2026 14:16
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enables running CoreCLR tests on the browser/WASM platform by removing the shouldRunSmokeOnly flag from the CI pipeline and adding a large exclusion list for tests that currently fail (build errors, crashes, test failures, timeouts) on browser/CoreCLR.

Changes:

  • Remove shouldRunSmokeOnly from the browser_wasm CoreCLR test pipeline to run the full test suite
  • Add ~85 test project exclusions for browser/CoreCLR gated behind RunDisabledWasmTests, tracking issue #125495

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/libraries/tests.proj Adds exclusion list for tests that don't work on browser/CoreCLR
eng/pipelines/runtime.yml Removes shouldRunSmokeOnly to enable full test runs

@radekdoulik
Copy link
Member Author

Based on today's results https://gist.github.com/radekdoulik/dfc6af9b870d7a9f1c52a1d810898346

✅ passed: 150
❌ failed: 6
💥 crashed: 62
⏰ timeout: 4
🚫 build error: 6

Copilot AI review requested due to automatic review settings March 12, 2026 20:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@radekdoulik radekdoulik marked this pull request as ready for review March 13, 2026 12:19
Copilot AI review requested due to automatic review settings March 13, 2026 12:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 13, 2026 14:00
@radekdoulik radekdoulik enabled auto-merge (squash) March 13, 2026 14:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@radekdoulik
Copy link
Member Author

/ba-g the failing tests are unrelated

@radekdoulik radekdoulik merged commit 1cfdd48 into dotnet:main Mar 16, 2026
167 of 175 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch-wasm WebAssembly architecture area-Infrastructure-libraries os-browser Browser variant of arch-wasm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants