Skip to content

Commit

Permalink
Re-enable NameResolution tests on SLES, Debian 9 and Ubuntu 16.04
Browse files Browse the repository at this point in the history
The underlying infrastructure issue was addressed in dotnet/core-eng#13889

Fixes dotnet#55271
Fixes dotnet#56295
  • Loading branch information
karelz committed Aug 16, 2021
1 parent 4723f00 commit d3c14c8
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -108,12 +108,6 @@ public void DnsObsoleteGetHostByName_IPv6String_ReturnsOnlyGivenIP()
[ActiveIssue("https://github.com/dotnet/runtime/issues/51377", TestPlatforms.iOS | TestPlatforms.tvOS | TestPlatforms.MacCatalyst)]
public void DnsObsoleteGetHostByName_EmptyString_ReturnsHostName()
{
if (PlatformDetection.IsSLES || // [ActiveIssue("https://github.com/dotnet/runtime/issues/55271")]
PlatformDetection.IsUbuntu1604 || PlatformDetection.IsDebian9) // [ActiveIssue("https://github.com/dotnet/runtime/issues/56295")]
{
throw new SkipTestException("Test environment is not configured for this test to work.");
}

IPHostEntry entry = Dns.GetHostByName("");

// DNS labels should be compared as case insensitive for ASCII characters. See RFC 4343.
Expand All @@ -125,12 +119,6 @@ public void DnsObsoleteGetHostByName_EmptyString_ReturnsHostName()
[ActiveIssue("https://github.com/dotnet/runtime/issues/51377", TestPlatforms.iOS | TestPlatforms.tvOS | TestPlatforms.MacCatalyst)]
public void DnsObsoleteBeginEndGetHostByName_EmptyString_ReturnsHostName()
{
if (PlatformDetection.IsSLES || // [ActiveIssue("https://github.com/dotnet/runtime/issues/55271")]
PlatformDetection.IsUbuntu1604 || PlatformDetection.IsDebian9) // [ActiveIssue("https://github.com/dotnet/runtime/issues/56295")]
{
throw new SkipTestException("Test environment is not configured for this test to work.");
}

IPHostEntry entry = Dns.EndGetHostByName(Dns.BeginGetHostByName("", null, null));

// DNS labels should be compared as case insensitive for ASCII characters. See RFC 4343.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,7 @@ public async Task Dns_GetHostEntryAsync_IPAddress_Ok()
// [ActiveIssue("https://github.com/dotnet/runtime/issues/1488", TestPlatforms.OSX)]
!PlatformDetection.IsOSX &&
// [ActiveIssue("https://github.com/dotnet/runtime/issues/51377", TestPlatforms.iOS | TestPlatforms.tvOS | TestPlatforms.MacCatalyst)]
!PlatformDetection.IsiOS && !PlatformDetection.IstvOS && !PlatformDetection.IsMacCatalyst &&
// [ActiveIssue("https://github.com/dotnet/runtime/issues/55271")]
!PlatformDetection.IsSLES &&
// [ActiveIssue("https://github.com/dotnet/runtime/issues/56295")]
!PlatformDetection.IsUbuntu1604 && !PlatformDetection.IsDebian9;
!PlatformDetection.IsiOS && !PlatformDetection.IstvOS && !PlatformDetection.IsMacCatalyst;

[ConditionalTheory(nameof(GetHostEntryWorks))]
[InlineData("")]
Expand Down

0 comments on commit d3c14c8

Please sign in to comment.