Skip to content

Commit

Permalink
Disabling two socket tests (dotnet#93916)
Browse files Browse the repository at this point in the history
* Disable two socket tests

* Revert close to dispose

* Disable tests against one issue
  • Loading branch information
liveans authored Oct 29, 2023
1 parent e307926 commit b6b00ec
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,12 @@ public async Task Connect_AfterDisconnect_Fails()
[InlineData("[::ffff:1.1.1.1]", true, false)]
public async Task ConnectGetsCanceledByDispose(string addressString, bool useDns, bool owning)
{
if (UsesSync && !PlatformDetection.IsWindows)
{
// [ActiveIssue("https://github.com/dotnet/runtime/issues/94149", TestPlatforms.Linux)]
return;
}

// Aborting sync operations for non-owning handles is not supported on Unix.
if (!owning && UsesSync && !PlatformDetection.IsWindows)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,7 @@ await listener.RunWithCallbackAsync(e => events.Enqueue((e, e.ActivityId)), asyn
}, connectMethod, useDnsEndPoint.ToString()).Dispose();
}

[ActiveIssue("https://github.com/dotnet/runtime/issues/94149", TestPlatforms.Linux)]
[OuterLoop]
[ConditionalTheory(typeof(RemoteExecutor), nameof(RemoteExecutor.IsSupported))]
[SkipOnPlatform(TestPlatforms.OSX | TestPlatforms.FreeBSD, "Same as Connect.ConnectGetsCanceledByDispose")]
Expand Down

0 comments on commit b6b00ec

Please sign in to comment.