exit parent test immediately

Author: rhoban13

src/pytest_subtests/plugin.py

@@ -242,9 +242,6 @@ def __exit__(
         __tracebackhide__ = True
         try:
             if exc_val is not None:
-                if self.request.session.shouldfail:
-                    return False
-
                 exc_info = ExceptionInfo.from_exception(exc_val)
             else:
                 exc_info = None
@@ -275,7 +272,9 @@ def __exit__(
                 node=self.request.node, call=call_info, report=sub_report
             )
 
-        return True
+        if exc_val is not None:
+            if self.request.session.shouldfail:
+                pytest.exit(reason=f"subtest failed")
 
 
 def make_call_info(

tests/test_subtests.py

@@ -593,11 +593,13 @@ def test_foo(subtests):
                 assert False
 
             with subtests.test("sub2"):
-                assert False
+                assert True
+
+            assert False, "This would fail the parent, but shouldn't be reached"
         """
     )
     result = pytester.runpytest("--exitfirst")
-    assert result.parseoutcomes()["failed"] == 1
+    assert result.parseoutcomes()["failed"] == 1  # sub1 failed
     result.stdout.fnmatch_lines(
         [
             "*[[]sub1[]] SUBFAIL test_exitfirst.py::test_foo - assert False*",
@@ -606,3 +608,4 @@ def test_foo(subtests):
         consecutive=True,
     )
     result.stdout.no_fnmatch_line("*sub2*")  # sub2 not executed.
+    result.stdout.no_fnmatch_line("*This would fail the parent*")  # parent test neither passed nor failed