Skip to content

Commit

Permalink
Merge pull request dotnet/corefx#42342 from davidsh/win7_disable_tests
Browse files Browse the repository at this point in the history
Disable some networking tests on Windows 7

Commit migrated from dotnet/corefx@f3eea90
  • Loading branch information
davidsh authored Nov 4, 2019
2 parents 0cf6b79 + 5b5a7c8 commit e164c1b
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,12 @@ public void CreateFromStream_ValidBufferSizes_CreatesWebSocket()
[MemberData(nameof(EchoServers))]
public async Task WebSocketProtocol_CreateFromConnectedStream_CanSendReceiveData(Uri echoUri)
{
if (PlatformDetection.IsWindows7)
{
// https://github.com/dotnet/corefx/issues/42339
return;
}

using (var client = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp))
{
bool secure = echoUri.Scheme == "wss";
Expand Down Expand Up @@ -151,6 +157,12 @@ public async Task ReceiveAsync_ServerSplitHeader_ValidDataReceived()
[MemberData(nameof(EchoServersAndBoolean))]
public async Task WebSocketProtocol_CreateFromConnectedStream_CloseAsyncClosesStream(Uri echoUri, bool explicitCloseAsync)
{
if (PlatformDetection.IsWindows7)
{
// https://github.com/dotnet/corefx/issues/42339
return;
}

using (var client = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp))
{
bool secure = echoUri.Scheme == "wss";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,12 @@ public class CertificateValidationRemoteServer
[OuterLoop("Uses external servers")]
public async Task CertificateValidationRemoteServer_EndToEnd_Ok()
{
if (PlatformDetection.IsWindows7)
{
// https://github.com/dotnet/corefx/issues/42339
return;
}

using (var client = new TcpClient(AddressFamily.InterNetwork))
{
await client.ConnectAsync(Configuration.Security.TlsServer.IdnHost, Configuration.Security.TlsServer.Port);
Expand Down

0 comments on commit e164c1b

Please sign in to comment.