Comment out a couple tests and add notes. Needs test and cleanup.

dotnet · Jul 25, 2024 · f46bebc · f46bebc
1 parent 3a7616e
commit f46bebc
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs b/src/libraries/System.Diagnostics.Process/tests/ProcessTests.cs
@@ -601,14 +601,19 @@ public void TestMaxWorkingSet()
             }
         }
 
+// XXX No longer skip: OSX, FreeBSD
         [Fact]
-        [SkipOnPlatform(TestPlatforms.OSX | TestPlatforms.FreeBSD | TestPlatforms.iOS | TestPlatforms.MacCatalyst | TestPlatforms.tvOS, "Getting MaxWorkingSet is not supported on OSX, BSD, iOS, MacCatalyst, and tvOS.")]
+        [SkipOnPlatform(TestPlatforms.iOS | TestPlatforms.MacCatalyst | TestPlatforms.tvOS, "Getting MaxWorkingSet is not supported on OSX, BSD, iOS, MacCatalyst, and tvOS.")]
         public void MaxWorkingSet_GetNotStarted_ThrowsInvalidOperationException()
         {
             var process = new Process();
             Assert.Throws<InvalidOperationException>(() => process.MaxWorkingSet);
         }
 
+// XXX This appears to be testing for incorrect behavoir which has been fixed.
+// XXX See https://github.com/dotnet/runtime/issues/105422
+// The correct exception for a new process is: InvalidOperationException
+#if 0
         [Fact]
         [PlatformSpecific(TestPlatforms.OSX | TestPlatforms.FreeBSD)]
         public void MaxValueWorkingSet_GetSetMacos_ThrowsPlatformSupportedException()
@@ -617,6 +622,7 @@ public void MaxValueWorkingSet_GetSetMacos_ThrowsPlatformSupportedException()
             Assert.Throws<PlatformNotSupportedException>(() => process.MaxWorkingSet);
             Assert.Throws<PlatformNotSupportedException>(() => process.MaxWorkingSet = (IntPtr)1);
         }
+#endif
 
         [ConditionalFact(typeof(RemoteExecutor), nameof(RemoteExecutor.IsSupported))]
         public void TestMinWorkingSet()
@@ -656,21 +662,27 @@ public void TestMinWorkingSet()
             }
         }
 
+// XXX No longer skip: OSX, FreeBSD
         [Fact]
-        [SkipOnPlatform(TestPlatforms.OSX | TestPlatforms.FreeBSD | TestPlatforms.iOS | TestPlatforms.MacCatalyst | TestPlatforms.tvOS, "Getting MinWorkingSet is not supported on OSX, BSD, iOS, MacCatalyst, and tvOS.")]
+        [SkipOnPlatform(TestPlatforms.iOS | TestPlatforms.MacCatalyst | TestPlatforms.tvOS, "Getting MinWorkingSet is not supported on OSX, BSD, iOS, MacCatalyst, and tvOS.")]
         public void MinWorkingSet_GetNotStarted_ThrowsInvalidOperationException()
         {
             var process = new Process();
             Assert.Throws<InvalidOperationException>(() => process.MinWorkingSet);
         }
 
+// XXX This appears to be testing for incorrect behavoir which has been fixed.
+// XXX See https://github.com/dotnet/runtime/issues/105422
+// The correct exception for a new process is: InvalidOperationException
+#if 0
         [Fact]
         [PlatformSpecific(TestPlatforms.OSX | TestPlatforms.FreeBSD)]
         public void MinWorkingSet_GetMacos_ThrowsPlatformSupportedException()
         {
             var process = new Process();
             Assert.Throws<PlatformNotSupportedException>(() => process.MinWorkingSet);
         }
+#endif
 
         [Fact]
         public void TestModules()