fix #13537: Add support for ExceptionGroup with only Skipped exceptions in teardown

Author: gomri15

src/_pytest/reports.py

@@ -274,26 +274,30 @@ def _format_exception_group_all_skipped_longrepr(
     excinfo: ExceptionInfo[BaseException],
     exceptions: list[Skipped],
 ) -> tuple[str, int, str]:
+    r = excinfo._getreprcrash()
+    assert r is not None, (
+        "There should always be a traceback entry for skipping a test."
+    )
     if any(getattr(skip, "_use_item_location", False) for skip in exceptions):
         path, line = item.reportinfo()[:2]
         assert line is not None
         loc = (os.fspath(path), line + 1)
         default_msg = "skipped"
+        # longrepr = (*loc, r.message)
     else:
-        r = excinfo._getreprcrash()
         assert r is not None
         loc = (str(r.path), r.lineno)
         default_msg = r.message
 
-        # reason(s): order-preserving de-dupe, same fields as single-skip
-        msgs: list[str] = []
-        for exception in exceptions:
-            m = exception.msg or exception.args[0]
-            if m and m not in msgs:
-                msgs.append(m)
+    # reason(s): order-preserving de-dupe, same fields as single-skip
+    msgs: list[str] = []
+    for exception in exceptions:
+        m = exception.msg or exception.args[0]
+        if m and m not in msgs:
+            msgs.append(m)
 
-        reason = "; ".join(msgs) if msgs else default_msg
-        longrepr = (*loc, reason)
+    reason = "; ".join(msgs) if msgs else default_msg
+    longrepr = (*loc, reason)
     return longrepr
 
 

testing/test_reports.py

@@ -450,16 +450,56 @@ def fixA():
             @pytest.fixture
             def fixB():
                 yield
-                pytest.skip(reason="A")
+                pytest.skip(reason="B")
             def test_skip(
                 fixA,
                 fixB
                 ):
                 assert True
             """
         )
-        result = pytester.runpytest("-vv")
+        result = pytester.runpytest("-v")
         result.assert_outcomes(passed=1, skipped=1)
+        out = result.stdout.str()
+        # Both reasons should appear
+        assert "A" in out and "B" in out
+        assert "ERROR at teardown" not in out
+
+    def test_exception_group_skips_use_item_location(self, pytester: Pytester):
+        """
+        Regression for #13537:
+        If any skip inside an ExceptionGroup has _use_item_location=True,
+        the report location should point to the test item, not the fixture teardown.
+        """
+        pytester.makepyfile(
+            test_it="""
+            import pytest
+
+            @pytest.fixture
+            def fix_item_loc():
+                yield
+                # Create skip.Exception and set flag to use item location
+                exc = pytest.skip.Exception("A")
+                exc._use_item_location = True
+                raise exc
+
+            @pytest.fixture
+            def fix_normal():
+                yield
+                raise pytest.skip.Exception("B")
+
+            def test_both(fix_item_loc, fix_normal):
+                assert True
+            """
+        )
+        result = pytester.runpytest("-v")
+        result.assert_outcomes(passed=1, skipped=1)
+
+        out = result.stdout.str()
+        # Both reasons should appear
+        assert "A" in out and "B" in out
+        # Crucially, the skip should be attributed to the test item, not teardown
+        assert "test_both" in out
 
 
 class TestHooks: