chore(ci_visibility): ensure exception dictionary is cleared after each test (#14344)

The solution in PR #14328 was incomplete. This PR ensures all exceptions
are cleared up at the end of each test.

I'm not adding a release note because this is a continuation of #14328,
which already has a release note.

- [x] PR author has checked that all the criteria below are met
- The PR description includes an overview of the change
- The PR description articulates the motivation for the change
- The change includes tests OR the PR description describes a testing
strategy
- The PR description notes risks associated with the change, if any
- Newly-added code is easy to change
- The change follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

- [x] Reviewer has checked that all the criteria below are met
- Title is accurate
- All changes are related to the pull request's stated goal
- Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes
- Testing strategy adequately addresses listed risks
- Newly-added code is easy to change
- Release note makes sense to a user of the library
- If necessary, author has acknowledged and discussed the performance
implications of this PR as reported in the benchmarks PR comment
- Backport labels are set in a manner that is consistent with the
[release branch maintenance
policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

(cherry picked from commit a5aee01)

Author: vitor-de-araujo

ddtrace/contrib/internal/pytest/_plugin_v2.py

@@ -485,16 +485,21 @@ def _pytest_runtest_protocol_post_yield(item, nextitem, coverage_collector):
     suite_id = test_id.parent_id
     module_id = suite_id.parent_id
 
+    reports_dict = reports_by_item.pop(item, None)
+
     if not InternalTest.is_finished(test_id):
         log.debug("Test %s was not finished normally during pytest_runtest_protocol, finishing it now", test_id)
-        reports_dict = reports_by_item.pop(item, None)
         if reports_dict:
             test_outcome = _process_reports_dict(item, reports_dict)
             InternalTest.finish(test_id, test_outcome.status, test_outcome.skip_reason, test_outcome.exc_info)
         else:
             log.debug("Test %s has no entry in reports_by_item", test_id)
             InternalTest.finish(test_id)
 
+    if reports_dict:
+        for report in reports_dict.values():
+            excinfo_by_report.pop(report, None)
+
     if coverage_collector is not None:
         _handle_collected_coverage(test_id, coverage_collector)
 

tests/contrib/pytest/test_pytest.py

@@ -9,6 +9,8 @@
 import ddtrace
 from ddtrace.constants import _SAMPLING_PRIORITY_KEY
 from ddtrace.constants import ERROR_MSG
+from ddtrace.contrib.internal.pytest._utils import excinfo_by_report
+from ddtrace.contrib.internal.pytest._utils import reports_by_item
 from ddtrace.contrib.internal.pytest.constants import XFAIL_REASON
 from ddtrace.contrib.internal.pytest.patch import get_version
 from ddtrace.contrib.internal.pytest.plugin import is_enabled
@@ -4275,6 +4277,21 @@ def test_log_capture():
         assert "I/O operation on closed file" not in result.stderr.str()
         assert result.ret == 0
 
+    def test_pytest_clears_excinfo_dict_after_use(self):
+        reports_by_item_count_before = len(reports_by_item)
+        excinfo_by_report_count_before = len(excinfo_by_report)
+
+        self.testdir.makepyfile(
+            """
+            def test_one():
+                assert False
+            """
+        )
+
+        self.inline_run("--ddtrace")
+        assert len(reports_by_item) == reports_by_item_count_before
+        assert len(excinfo_by_report) == excinfo_by_report_count_before
+
 
 def test_pytest_coverage_data_format_handling_none_value():
     """Test that coverage data format issues are handled correctly with proper logging for None value."""