fixup! Add tests for sampling profiler

lkollar · lkollar · commit a019e6a38360 · 2025-07-03T21:35:49.000+01:00
diff --git a/Lib/test/test_sample_profiler.py b/Lib/test/test_sample_profiler.py
@@ -19,6 +19,15 @@
 
 from test.support.os_helper import unlink
 
+PROCESS_VM_READV_SUPPORTED = False
+
+try:
+    from _remote_debugging import PROCESS_VM_READV_SUPPORTED
+except ImportError:
+    raise unittest.SkipTest(
+        "Test only runs when _remote_debugging is available"
+    )
+
 
 class MockFrameInfo:
     """Mock FrameInfo for testing since the real one isn't accessible."""
@@ -256,11 +265,13 @@ def test_pstats_collector_export(self):
         self.assertEqual(func1_stats[3], 2.0)  # ct (cumulative time)
 
 
+@unittest.skipIf(
+    sys.platform == "linux" and not PROCESS_VM_READV_SUPPORTED,
+    "Test only runs on Linux with process_vm_readv support",
+)
 class TestSampleProfilerIntegration(unittest.TestCase):
     @classmethod
     def setUpClass(cls):
-        cls.python_exe = sys.executable
-
         cls.test_script = '''
 import time
 import os
@@ -472,6 +483,10 @@ def test_sampling_all_threads(self):
         # We're not testing output format here
 
 
+@unittest.skipIf(
+    sys.platform == "linux" and not PROCESS_VM_READV_SUPPORTED,
+    "Test only runs on Linux with process_vm_readv support",
+)
 class TestSampleProfilerErrorHandling(unittest.TestCase):
     def test_invalid_pid(self):
         with self.assertRaises((OSError, RuntimeError)):
