Update to xunit v3

[martincostello]

• Update to xunit v3.
• Avoid infinite loop and throw if test times out.

Resolves #2439.

[Copilot]

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

Files not reviewed (15)

• Directory.Packages.props: Language not supported
• eng/Test.targets: Language not supported
• src/Snippets/Snippets.csproj: Language not supported
• test/Polly.Core.Tests/CircuitBreaker/Controller/CircuitStateControllerTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/BrokenCircuitExceptionTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/IsolatedCircuitExceptionTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/Fallback/FallbackHandlerTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/CircuitBreakerResiliencePipelineBuilderTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/CircuitBreakerOptionsTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/CircuitBreakerResilienceStrategyTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/Fallback/FallbackResiliencePipelineBuilderExtensionsTests.cs: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/CircuitBreakerStateProviderTests.cs: Evaluated as low risk
• src/Snippets/Docs/Testing.cs: Evaluated as low risk
• build.cake: Evaluated as low risk
• test/Polly.Core.Tests/CircuitBreaker/CircuitBreakerPredicateArgumentsTests.cs: Evaluated as low risk

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.15%. Comparing base (593c67f) to head (2760015).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2440   +/-   ##
=======================================
  Coverage   96.15%   96.15%           
=======================================
  Files         309      309           
  Lines        7128     7128           
  Branches     1005     1005           
=======================================
  Hits         6854     6854           
  Misses        221      221           
  Partials       53       53           

Flag	Coverage Δ
linux	100.00% <ø> (+3.84%)	⬆️
macos	96.15% <ø> (ø)
windows	96.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Youssef1313]

Consider using xunit.v3.mtp-v2 instead. Latest version is 3.2.1.

[Youssef1313]

This shouldn't be needed if you intend to also run with MTP.

[martincostello]

Styker tests are still broken both without MTP (stryker-mutator/stryker-net#3117) and with MTP (stryker-mutator/stryker-net#3094).

Unit tests only work with MTP disabled, otherwise the fact GitHubActionsTestLogger isn't strong-named becomes a problem and causes test compilation to fail.

[martincostello]

I'm going to close this out as it's otherwise about to reach its first birthday. We can revisit in the future if/when stryker supports MTP.