Skip to content

Disable System.IO.Tests.FileStream_SafeFileHandle.ThrowWhenHandlePositionIsChanged_async test on Browser wasm - #40920

Merged
danmoseley merged 2 commits into
dotnet:masterfrom
MaximLipnin:wasm_disable_test_in_system.io.filename
Aug 19, 2020
Merged

Disable System.IO.Tests.FileStream_SafeFileHandle.ThrowWhenHandlePositionIsChanged_async test on Browser wasm#40920
danmoseley merged 2 commits into
dotnet:masterfrom
MaximLipnin:wasm_disable_test_in_system.io.filename

Conversation

@MaximLipnin

Copy link
Copy Markdown
Contributor

The System.IO.Tests.FileStream_SafeFileHandle.ThrowWhenHandlePositionIsChanged_async test sometimes (flaky?) fails in CI on browser wasm (according to #40886) with PlatformNotSupportedException: Cannot wait on monitors on this runtime.
In this case we are used to marking the failing test with platform detection for the threading feature.

Closes #40886

@ghost

ghost commented Aug 17, 2020

Copy link
Copy Markdown

Tagging subscribers to this area: @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

@steveisok

Copy link
Copy Markdown
Member

Do we have a sense of why it's flaky?

@MaximLipnin

MaximLipnin commented Aug 18, 2020

Copy link
Copy Markdown
Contributor Author

Do we have a sense of why it's flaky?

Tried to find out in which conditions it could fail but I got it only once - it was during the first run on clean build. Other times I didn't manage to reproduce it.

@MaximLipnin

Copy link
Copy Markdown
Contributor Author

OSX/iOS lanes fail with #41006

@danmoseley

Copy link
Copy Markdown
Contributor

Merging as this is sufficiently validated and is breaking other PR validation jobs right now.

@danmoseley
danmoseley merged commit 9401b2f into dotnet:master Aug 19, 2020
@MaximLipnin
MaximLipnin deleted the wasm_disable_test_in_system.io.filename branch August 20, 2020 07:18
@Anipik

Anipik commented Aug 20, 2020

Copy link
Copy Markdown
Contributor

/backport to release/5.0

@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/5.0: https://github.com/dotnet/runtime/actions/runs/217066409

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

wasm test failure: System.IO.Tests.FileStream_SafeFileHandle.ThrowWhenHandlePositionIsChanged_async

5 participants