Flaky test 'InteractiveHostDesktopTests.StackOverflow'

[RikkiGibson]

roslyn/src/Interactive/HostTest/InteractiveHostDesktopTests.cs

Line 71 in 4d0c2c0

public async Task StackOverflow()

Example build: https://dev.azure.com/dnceng/public/_build/results?buildId=749582&view=ms.vss-test-web.build-test-results-tab&runId=23240534&resultId=202015&paneView=debug

Xunit log:

    Microsoft.CodeAnalysis.UnitTests.Interactive.InteractiveHostDesktopTests.StackOverflow [FAIL]
      Assert.Equal() Failure
                                       � (pos 52)
      Expected: úúúckOverflowException.\nHosting process exited with exit code -1úúú
      Actual:   úúúckOverflowException.
                                       � (pos 52)
      Stack Trace:
        /_/src/Interactive/HostTest/InteractiveHostDesktopTests.cs(94,0): at Microsoft.CodeAnalysis.UnitTests.Interactive.InteractiveHostDesktopTests.StackOverflow()
        --- End of stack trace from previous location ---

Runfo Tracking Issue: microsoft.codeanalysis.unittests.interactive.interactivehostdesktoptests.stackoverflow

Build	Definition	Kind	Run Name

Build Result Summary

Day Hit Count	Week Hit Count	Month Hit Count
0	0	0

[RikkiGibson]

Another run containing this failure

https://dev.azure.com/dnceng/public/_build/results?buildId=778472&view=ms.vss-test-web.build-test-results-tab&runId=24376582&resultId=100019&paneView=debug

@tmat would a PR be accepted which accepts either of these strings or perhaps checks a prefix match in the error message?

[tmat]

Yes, accepting either of these seems a fine workaround. Let's keep an issue opened to track investigation of why are we not seeing the rest of the message. It's not critical though that we do not.

[jaredpar]

Can we please move forward with this change? This test is the #1 most flaky test in Roslyn right now. It's failing at least one build per day.