-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Remove legacy XUnitWrapper test tree infrastructure #123110
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
Remove legacy XUnitWrapper test tree infrastructure #123110
Conversation
…without going through an xunit harness.
…standalone runner
|
Tagging subscribers to this area: @dotnet/runtime-infrastructure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 22 out of 22 changed files in this pull request and generated 3 comments.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Now that we've moved entirely to the new test system, we can remove the old XUnitWrapper logic from the test build system and the helix publishing. This will help us be able to further evolve the test system as we will now have a baseline of 1 test running mechanism instead of 2.
To ensure compatibility with some of the community's workflows, I've added a very basic local test runner for the BuildAllTestsAsStandalone=true scenario. This scenario used to fall back on the old test runner system. With that system gone, a replacement was needed.