updates

Author: eleanorjboyd

python_files/tests/unittestadapter/test_discovery.py

@@ -191,7 +191,8 @@ def test_empty_discovery() -> None:
     actual = discover_tests(start_dir, pattern, None)
 
     assert actual["status"] == "success"
-    assert "tests" in actual
+    # When no tests are found, the tests key should not be present in the payload
+    assert "tests" not in actual
     assert "error" not in actual
 
 

python_files/unittestadapter/discovery.py

@@ -91,9 +91,10 @@ def discover_tests(
     except Exception:
         error.append(traceback.format_exc())
 
-    # Still include the tests in the payload even if there are errors so that the TS
-    # side can determine if it is from run or discovery.
-    payload["tests"] = tests if tests is not None else None
+    # Only include tests in the payload if tests were discovered.
+    # If no tests found (tests is None), omit the tests key per the docstring contract.
+    if tests is not None:
+        payload["tests"] = tests
 
     if len(error):
         payload["status"] = "error"

python_files/unittestadapter/pvsc_utils.py

@@ -281,6 +281,10 @@ def build_test_tree(
             }  # concatenate class name and function test name
             current_node["children"].append(test_node)
 
+    # If no tests were discovered, return None instead of an empty root node
+    if not root["children"]:
+        return None, error
+
     return root, error