Skip to content
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

[tests] Try a newer xunit #22860

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[tests] Try a newer xunit #22860

wants to merge 2 commits into from

Conversation

Redth
Copy link
Member

@Redth Redth commented Jun 5, 2024

XUnit 2.8.0 seemed to have issues in CI for us around GC related tests.

We dropped down to an older 2.6.6 version which helped. In testing newer versions (eg: 2.7.1) we hit some errors while running in the IDE which were fixed in #22835

Now trying a slightly newer version of Xunit again to see where we stand.

@Redth Redth requested a review from a team as a code owner June 5, 2024 17:11
@jsuarezruiz jsuarezruiz added the area-testing Unit tests, device tests label Jun 6, 2024
eng/Versions.props Outdated Show resolved Hide resolved
@Redth
Copy link
Member Author

Redth commented Jun 12, 2024

/rebase

Redth and others added 2 commits June 12, 2024 00:31
Let's try latest again and see if the previous fix which excludes the default ref from arcade helps fix the pipeline test failures we saw too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-testing Unit tests, device tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants