Skip to content

[wasm][aot] Enable tests that are passing now#53977

Merged
lewing merged 2 commits into
dotnet:mainfrom
radical:aot-enable-exception-fix
Jun 11, 2021
Merged

[wasm][aot] Enable tests that are passing now#53977
lewing merged 2 commits into
dotnet:mainfrom
radical:aot-enable-exception-fix

Conversation

@radical

@radical radical commented Jun 10, 2021

Copy link
Copy Markdown
Member

These don't reproduce on main anymore, after the exceptions fix.

System.Reflection.Tests Fixes #51673
System.Transactions.Locals.Tests Fixes #50969
System.Threading.Tasks.Parallel.Tests Fixes #51677

@radical radical added the arch-wasm WebAssembly architecture label Jun 10, 2021
@ghost

ghost commented Jun 10, 2021

Copy link
Copy Markdown

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

These don't reproduce on main anymore, after the exceptions fix.

System.Reflection.Tests Fixes #51673
System.Transactions.Locals.Tests Fixes #50969
System.Threading.Tasks.Parallel.Tests Fixes #51677

Author: radical
Assignees: -
Labels:

arch-wasm

Milestone: -

@ghost

ghost commented Jun 10, 2021

Copy link
Copy Markdown

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

radical added 2 commits June 10, 2021 12:29
These don't reproduce on `main` anymore, after the exceptions fix.

`System.Transactions.Locals.Tests` Fixes dotnet#50969
`System.Threading.Tasks.Parallel.Tests` Fixes dotnet#51677
This hasn't been crashing for quite some time.
The individual test failures are already disabled with
dotnet#51244, and
dotnet#51911 .

Fixes dotnet#51245
@radical radical force-pushed the aot-enable-exception-fix branch from ad600cb to 38bd58a Compare June 10, 2021 16:31
@lewing lewing merged commit e6daad2 into dotnet:main Jun 11, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jul 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-wasm WebAssembly architecture

Projects

None yet

2 participants