Skip to content

chore: Add temporary workaround for ThreadingDiagnoserTests::CompletedWorkItemCountIsAccurate on macos#3101

Open
filzrev wants to merge 2 commits intodotnet:masterfrom
filzrev:chore-fix-flakey-threading-diagnoser-tests
Open

chore: Add temporary workaround for ThreadingDiagnoserTests::CompletedWorkItemCountIsAccurate on macos#3101
filzrev wants to merge 2 commits intodotnet:masterfrom
filzrev:chore-fix-flakey-threading-diagnoser-tests

Conversation

@filzrev
Copy link
Copy Markdown
Contributor

@filzrev filzrev commented Apr 28, 2026

This PR add temporary workaround for #3098

Currently, on macOS environment.
ThreadingDiagnoserTests::CompletedWorkItemCountIsAccurate test failed randomly.
Because some background operation executed during running benchmarks.

At this point, the root cause remains unknown.
So this PR temporary suppress CI failure by reducing assert value of floating point precision.

Note: Temporary modify precision to 0 when it's not running with InProcessToolchain.
It'll be reverted if it's not works as expected

@filzrev filzrev force-pushed the chore-fix-flakey-threading-diagnoser-tests branch 2 times, most recently from 65f9b3f to b85f240 Compare April 28, 2026 20:44
Comment thread tests/BenchmarkDotNet.IntegrationTests/ThreadingDiagnoserTests.cs Outdated
Comment thread tests/BenchmarkDotNet.IntegrationTests/ThreadingDiagnoserTests.cs Outdated
@filzrev filzrev force-pushed the chore-fix-flakey-threading-diagnoser-tests branch 2 times, most recently from 80d7b67 to be3276b Compare April 29, 2026 22:29
@filzrev filzrev force-pushed the chore-fix-flakey-threading-diagnoser-tests branch from be3276b to 61cae2e Compare April 29, 2026 22:30
Comment thread tests/BenchmarkDotNet.IntegrationTests/ThreadingDiagnoserTests.cs Outdated
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.

2 participants