Skip to content

Convert System.Collections.Tests to xUnit v3 - #134029

Draft
agocke wants to merge 2 commits into
dotnet:mainfrom
agocke:copilot/xunit3-system-collections
Draft

agocke wants to merge 2 commits into
dotnet:mainfrom
agocke:copilot/xunit3-system-collections

Conversation

@agocke

@agocke agocke commented Sep 16, 2026

Copy link
Copy Markdown
Member

Summary

  • add a per-project UseXUnitV3 opt-in to the libraries test infrastructure
  • build a separate xUnit v3 variant of TestUtilities so v2 and v3 projects can coexist
  • convert the standard desktop/archive System.Collections.Tests path to xUnit v3
  • keep single-file, NativeAOT, ReadyToRun, mobile, browser, and WASI configurations on xUnit v2 for this pilot

Validation

  • dotnet build src/libraries/System.Collections/tests/System.Collections.Tests.csproj -c Release
  • xUnit v3 smoke run of System.Collections.Tests.StructuralComparisonsTests: 21 passed
  • dotnet build src/libraries/System.Collections.Concurrent/tests/System.Collections.Concurrent.Tests.csproj -c Release to verify a neighboring xUnit v2 project remains unaffected

A full local /t:test run is currently blocked by the existing testhost layout: the Release testhost contains .NET 11 while the generated runtime configuration requires .NET 12. Refreshing the prescribed baseline is blocked because clang-18 is not installed on this machine.

Note

This pull request description was generated with GitHub Copilot.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 21b4fbf8-d548-4f4d-a033-aa8e36080965
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 21b4fbf8-d548-4f4d-a033-aa8e36080965
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@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.

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

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant