From 9b2e2a830a4e2e67c920aa200329533baba5c363 Mon Sep 17 00:00:00 2001 From: Steve Pfister Date: Thu, 31 Mar 2022 19:11:09 -0400 Subject: [PATCH] Add RemoteExecutor check in TestWaitForExitValidation (#67397) https://github.com/dotnet/runtime/pull/64860 added a TestWaitForExitValidation test that would fail when run on platforms that did not support RemoteExecutor. This change adds a `ConditionalTheory` to make sure that doesn't happen. --- src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs b/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs index 90dc6a8b4ba8f6..cc5a5894bf7fe9 100644 --- a/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs +++ b/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs @@ -74,7 +74,7 @@ private void AssertNonZeroAllZeroDarwin(long value) } } - [Theory] + [ConditionalTheory(typeof(RemoteExecutor), nameof(RemoteExecutor.IsSupported))] [InlineData(-2)] [InlineData((long)int.MaxValue + 1)] public void TestWaitForExitValidation(long milliseconds)