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.

Sign up for GitHub

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

Jump to bottom

Unstable test: RunAsyncWithYieldingDelegateNestedInRunOverhead #45

Closed
AArnott opened this issue Jan 14, 2017 · 0 comments
Closed

Unstable test: RunAsyncWithYieldingDelegateNestedInRunOverhead #45

AArnott opened this issue Jan 14, 2017 · 0 comments
Assignees
@AArnott
Labels
test bug

Comments

@AArnott
Copy link
Member

AArnott commented Jan 14, 2017

This test has been observed to fail:

    RunAsyncWithYieldingDelegateNestedInRunOverhead [FAIL]
      Assert.Equal() Failure
      Expected: 1
      Actual:   0
      Stack Trace:
        src\Microsoft.VisualStudio.Threading.Tests.Shared\JoinableTaskTests.cs(3030,0): at Microsoft.VisualStudio.Threading.Tests.JoinableTaskTests.RunAsyncWithYieldingDelegateNestedInRunOverhead()
AArnott added a commit that referenced this issue Jan 14, 2017
@AArnott
Suppress 2 tests that are unstable in cloud builds
e3a6a79 
#44 #45
@AArnott AArnott self-assigned this May 23, 2017
AArnott added a commit that referenced this issue May 23, 2017
@AArnott
Fix unreliability in RunAsyncWithYieldingDelegateNestedInRunOverhead …
11820a4 
…test

Somehow we were getting a WaitCount of 0 at the end of the test, which I can only explain by suggesting that the 1000 iteration loop on a background thread completed before the main thread even had to sleep once. This seems hardly expected, but it's the only explanation I can think of. So this commit changes the test to *ensure* it waits at least once.

Fixes #45
@AArnott AArnott closed this as completed in 5ebdc7a Jun 3, 2017
AArnott added a commit that referenced this issue Apr 11, 2022
@AArnott
Merge pull request #45 from AArnott/moveSlnTopLevel
5bfc8ae 
Move sln file to repo root
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AArnott AArnott

Labels
test bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@AArnott