Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Commit c3b67d5

Browse files
committed
Disable inheritance test on linux #3034
1 parent 23a4e11 commit c3b67d5

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

test/Kestrel.Transport.FunctionalTests/HandleInheritanceTests.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ namespace Microsoft.AspNetCore.Server.Kestrel.FunctionalTests
1717
public class HandleInheritanceTests : TestApplicationErrorLoggerLoggedTest
1818
{
1919
[ConditionalFact]
20-
[OSSkipCondition(OperatingSystems.MacOSX, SkipReason = "No fix available for Mac https://github.com/aspnet/KestrelHttpServer/pull/2944#issuecomment-426397600")]
20+
[OSSkipCondition(OperatingSystems.Linux, SkipReason = "Fixed in 3.0 https://github.com/aspnet/KestrelHttpServer/issues/3040")]
21+
[OSSkipCondition(OperatingSystems.MacOSX, SkipReason = "Fixed in 3.0 https://github.com/aspnet/KestrelHttpServer/issues/3040")]
2122
public async Task SpawnChildProcess_DoesNotInheritListenHandle()
2223
{
2324
var hostBuilder = TransportSelector.GetWebHostBuilder()
@@ -38,7 +39,7 @@ public async Task SpawnChildProcess_DoesNotInheritListenHandle()
3839

3940
var processInfo = new ProcessStartInfo
4041
{
41-
FileName = RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? "cmd.exe" : "vi",
42+
FileName = "cmd.exe",
4243
CreateNoWindow = true,
4344
};
4445
using (var process = Process.Start(processInfo))

0 commit comments

Comments
 (0)