Skip to content

Conversation

@tmat
Copy link
Member

@tmat tmat commented Sep 9, 2025

Configure Xunit to produce detaled console output, printing long-running test name every 20 seconds.

See https://dev.azure.com/dnceng-public/public/_build/results?buildId=1144401&view=logs&j=343adee7-d814-546f-dc3f-19ccb8aa9260&t=8dea7d69-d7ce-5872-c9d8-dd4fee7f95a7

Remove usage of Verify.Xunit.

@tmat tmat requested a review from a team as a code owner September 9, 2025 22:12
@drewnoakes
Copy link
Member

See also #9457

@drewnoakes
Copy link
Member

Thanks for picking this up.

The conversion from Verify to Assert.Equals with two collections is a bit of a regression for cases where there are multiple differences. I believe xUnit's output doesn't provide a lot of detail here.

Can we add a helper method that detects when there's a difference and logs the full "expected" and "actual" items out, so that a test failure can be copy/pasted into a diff tool. Without something like this, you can end up going around in circles, especially if the failure only happen in CI.

@tmat
Copy link
Member Author

tmat commented Sep 10, 2025

Can we add a helper method that detects when there's a difference and logs the full "expected" and "actual" items out, so that a test failure can be copy/pasted into a diff tool. Without something like this, you can end up going around in circles, especially if the failure only happen in CI.

Let me copy a helper from Roslyn that does exactly that:

image

Copy link
Member

@drewnoakes drewnoakes left a comment

Choose a reason for hiding this comment

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

Thanks!

@drewnoakes drewnoakes merged commit 0df8f82 into dotnet:main Sep 10, 2025
5 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 18.0 milestone Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants