[devtools][tests][4/N] Report disabled inspector tests as executed

Applies the same disabled-test treatment as the prior diffs in this stack to the devtools inspector tests. Some test runners preserve stale failure state when tests report through unittest skip results, so this replaces the conditionally disabled coverage with a local decorator that keeps the tests discoverable, logs their disabled reason, and produces an executed result.

Adds a disable_if decorator that mirrors unittest.skipIf (evaluating the condition at decoration time) and converts the three Windows-gated test cases to use it.

Differential Revision: [D106736354](https://our.internmc.facebook.com/intern/diff/D106736354/)


ghstack-source-id: 387629542
Pull-Request: pytorch#19874

Author: ssjia

devtools/inspector/tests/inspector_test.py

@@ -7,6 +7,7 @@
 # pyre-unsafe
 
 import copy
+import functools
 import os
 import random
 import statistics
@@ -90,6 +91,28 @@ def forward(self, indices: torch.Tensor, values: torch.Tensor) -> torch.Tensor:
 ETRECORD_PATH = "unittest_etrecord_path"
 
 
+def disable_if(condition, reason):
+    """Disable a test when condition is true, still reporting it as executed.
+
+    Conditional analogue of unittest.skipIf that keeps disabled tests visible in
+    logs instead of producing a skipped result, which some test runners handle
+    inconsistently.
+    """
+
+    def decorator(fn):
+        if not condition:
+            return fn
+
+        @functools.wraps(fn)
+        def wrapper(*args, **kwargs):
+            print(f"DISABLED_TEST: {fn.__qualname__}: {reason}")
+            return None
+
+        return wrapper
+
+    return decorator
+
+
 # TODO: write an E2E test: create an inspector instance, mock just the file reads, and then verify the external correctness
 class TestInspector(unittest.TestCase):
     def test_perf_data(self) -> None:
@@ -1504,7 +1527,7 @@ def test_calculate_numeric_gap_with_edge_dialect_exported_program_name(self):
             self.assertIsInstance(df, pd.DataFrame)
             self.assertEqual(len(df), 1)
 
-    @unittest.skipIf(sys.platform.startswith("win"), "Skipping on Windows")
+    @disable_if(sys.platform.startswith("win"), "Skipping on Windows")
     def test_transformer_block_xnnpack_numeric_gap_within_tolerance(self):
         """
         Test that the numeric gap between AOT and runtime intermediate outputs
@@ -1693,7 +1716,7 @@ def forward(
                         f"Stack trace for {op_name} doesn't contain file info",
                     )
 
-    @unittest.skipIf(sys.platform.startswith("win"), "Skipping on Windows")
+    @disable_if(sys.platform.startswith("win"), "Skipping on Windows")
     def test_intermediate_tensor_comparison_with_torch_export(self):
         """Test intermediate tensor comparison using torch.export.export and to_edge_transform_and_lower.
 
@@ -1840,7 +1863,7 @@ def _gen_random_runtime_output(
     ) -> List[Union[None, List[torch.Tensor], bool, float, int, str, torch.Tensor]]:
         return [torch.randn(RAW_DATA_SIZE)]
 
-    @unittest.skipIf(sys.platform.startswith("win"), "Skipping on Windows")
+    @disable_if(sys.platform.startswith("win"), "Skipping on Windows")
     def test_disable_debug_handle_validation_with_symbolic_shapes(self):
         """
         Test that demonstrates the issue with symbolic shape related nodes losing from_node info