Skip to content

Commit

Permalink
Re-enable NameResolution tests on SLES, Debian 9 and Ubuntu 16.04 (#5…
Browse files Browse the repository at this point in the history
…7509)

The underlying infrastructure issue was addressed in dotnet/core-eng#13889

Fixes #55271
Fixes #56295
  • Loading branch information
karelz authored Aug 17, 2021
1 parent 14b34eb commit 570efbb
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 570efbb

Please sign in to comment.