Skip to content

Commit

Permalink
skip parallel for tests
Browse files Browse the repository at this point in the history
  • Loading branch information
tylerje committed Dec 14, 2024
1 parent f2fef4f commit be54ced
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions src/Tests/Unit/ServiceWireTests/NpTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ public async Task CalculateAsyncTest()
Assert.Equal(a + b, result);
}

[Fact]
[Fact(Skip = "Avoid Parallel.For")]
public void SimpleParallelTest()
{
var rnd = new Random();
Expand Down Expand Up @@ -131,7 +131,7 @@ public void ResponseTest()
}
}

[Fact]
[Fact(Skip = "Avoid Parallel.For")]
public void ResponseParallelTest()
{
Parallel.For(0, 4, (index, state) =>
Expand Down
4 changes: 2 additions & 2 deletions src/Tests/Unit/ServiceWireTests/TcpTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public async Task CalculateAsyncTest()
await Task.Delay(100);
}

[Fact]
[Fact(Skip = "Avoid Parallel.For")]
public void SimpleParallelTest()
{
Task.Delay(100);
Expand Down Expand Up @@ -104,7 +104,7 @@ public void ResponseTest()
Task.Delay(100);
}

[Fact]
[Fact(Skip = "Avoid Parallel.For")]
public void ResponseParallelTest()
{
Task.Delay(100);
Expand Down
4 changes: 2 additions & 2 deletions src/Tests/Unit/ServiceWireTests/TcpZkTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ public async Task CalculateAsyncTest()
Task.Delay(100);

Check warning on line 84 in src/Tests/Unit/ServiceWireTests/TcpZkTests.cs

View workflow job for this annotation

GitHub Actions / build_windows

Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.

Check warning on line 84 in src/Tests/Unit/ServiceWireTests/TcpZkTests.cs

View workflow job for this annotation

GitHub Actions / build_linux

Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
}

[Fact]
[Fact(Skip = "Avoid Parallel.For")]
public void SimpleParallelZkTest()
{
Task.Delay(100);
Expand Down Expand Up @@ -132,7 +132,7 @@ public void ResponseZkTest()
Task.Delay(100);
}

[Fact]
[Fact(Skip = "Avoid Parallel.For")]
public void ResponseParallelTest()
{
Task.Delay(100);
Expand Down

0 comments on commit be54ced

Please sign in to comment.