Skip to content

Commit

Permalink
[wasm][mt] Fix System.Threading.Tasks.Parallel build (#97505)
Browse files Browse the repository at this point in the history
* [wasm][mt] Fix System.Threading.Tasks.Parallel build

Fixes #91583 and #91579

* Enable 5 more tests

* Feedback

* Fix typo
  • Loading branch information
radekdoulik authored Jan 26, 2024
1 parent 0d81a5a commit c105e7d
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 7 deletions.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>$(NetCoreAppCurrent)</TargetFramework>
<TargetFrameworks>$(NetCoreAppCurrent);$(NetCoreAppCurrent)-browser</TargetFrameworks>
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<UseCompilerGeneratedDocXmlFile>false</UseCompilerGeneratedDocXmlFile>
</PropertyGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -885,7 +885,6 @@ public static void TestParallelForPaths_Exceptions()
}

[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/91582", typeof(PlatformDetection), nameof(PlatformDetection.IsWasmThreadingSupported))]
public static void TestParallelScheduler()
{
ParallelOptions parallelOptions = new ParallelOptions();
Expand Down Expand Up @@ -993,7 +992,6 @@ public static void TestParallelScheduler()
}

[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/91541", typeof(PlatformDetection), nameof(PlatformDetection.IsWasmThreadingSupported))]
public static void TestInvokeDOPAndCancel()
{
ParallelOptions parallelOptions = null;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,6 @@ private static void OrderablePartitionerForEachPLRTest(

// Perform tests on various combinations of Stop()/Break()
[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/91579", typeof(PlatformDetection), nameof(PlatformDetection.IsWasmThreadingSupported))]
public static void SimultaneousStopBreakTests()
{
//
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,6 @@ static void oneMoveNext(int length, bool isOrderable)
/// </summary>
/// <param name="length"></param>
[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/97396", typeof(PlatformDetection), nameof(PlatformDetection.IsWasmThreadingSupported))]
public static void IterationsWithDependency()
{
static void iterationsWithDependency(int length, int dependencyIndex)
Expand Down Expand Up @@ -167,7 +166,6 @@ public static void PFEDisposeEnum()
/// Exception is expected and the enumerators are disposed
/// </summary>
[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/91581", typeof(PlatformDetection), nameof(PlatformDetection.IsWasmThreadingSupported))]
public static void ExceptionOnMoveNext()
{
static void exceptionOnMoveNext(int length, int indexToThrow, bool isOrderable)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ namespace System.Threading.Tasks.Tests
public static class RangePartitionerTests
{
[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/97440", typeof(PlatformDetection), nameof(PlatformDetection.IsWasmThreadingSupported))]
public static void RunPartitionerStaticTest_SingleChunking()
{
CountdownEvent cde = new CountdownEvent(2);
Expand Down

0 comments on commit c105e7d

Please sign in to comment.