Skip to content

[wasm][coreclr] Enable System.Linq.Expressions.Tests#127926

Open
radekdoulik wants to merge 1 commit intodotnet:mainfrom
radekdoulik:clr-wasm-enable-linq-expr-tests
Open

[wasm][coreclr] Enable System.Linq.Expressions.Tests#127926
radekdoulik wants to merge 1 commit intodotnet:mainfrom
radekdoulik:clr-wasm-enable-linq-expr-tests

Conversation

@radekdoulik
Copy link
Copy Markdown
Member

The suite was disabled due to suspected test failures, but investigation shows all 5,670 tests pass on Release CoreCLR. The only 3 failures are already covered by [ActiveIssue] attributes. Previous 0-tests-run results were caused by a Debug-only SkipOnCoreClr trait filter.

The suite was disabled due to suspected test failures, but investigation
shows all 5,670 tests pass on Release CoreCLR. The only 3 failures are
already covered by [ActiveIssue] attributes. Previous 0-tests-run results
were caused by a Debug-only SkipOnCoreClr trait filter.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@radekdoulik radekdoulik added this to the Future milestone May 7, 2026
Copilot AI review requested due to automatic review settings May 7, 2026 16:52
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
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 re-enables System.Linq.Expressions.Tests for the Browser WebAssembly CoreCLR libraries test runs by removing it from the Browser/CoreCLR project exclusion list in src/libraries/tests.proj.

Changes:

  • Removed the Browser/CoreCLR-specific ProjectExclusions entry that prevented System.Linq.Expressions.Tests from running.
  • Leaves existing exclusions for other scenarios (e.g., AOT/aggressive trimming and Apple mobile simulator CoreCLR) unchanged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants