chore(ci_visibility): support Attempt-to-Fix v4 (#13415)

This PR adds:
- Sending commit sha in the test management request;
- Marking failed attempts to fix with
`@test.test_management.attempt_to_fix_passed: false` (previously it was
marked with `true` if the attempt was successful, and not marked at all
if not successful).

## Checklist
- [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))

## Reviewer Checklist
- [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)

Author: vitor-de-araujo

ddtrace/contrib/internal/pytest/_plugin_v2.py

@@ -280,7 +280,7 @@ def pytest_sessionstart(session: pytest.Session) -> None:
             test_impact_analysis="1" if _pytest_version_supports_itr() else None,
             test_management_quarantine="1",
             test_management_disable="1",
-            test_management_attempt_to_fix="2" if _pytest_version_supports_attempt_to_fix() else None,
+            test_management_attempt_to_fix="4" if _pytest_version_supports_attempt_to_fix() else None,
         )
 
         InternalTestSession.discover(

ddtrace/internal/ci_visibility/_api_client.py

@@ -592,6 +592,7 @@ def fetch_test_management_tests(self) -> t.Optional[t.Dict[InternalTestId, TestP
                 "attributes": {
                     "repository_url": self._git_data.repository_url,
                     "commit_message": self._git_data.commit_message,
+                    "sha": self._git_data.commit_sha,
                 },
             }
         }

ddtrace/internal/ci_visibility/api/_test.py

@@ -554,8 +554,8 @@ def attempt_to_fix_finish_retry(
 
             if all_failed:
                 retry_test.set_tag(TEST_HAS_FAILED_ALL_RETRIES, True)
-            elif all_passed:
-                retry_test.set_tag(TEST_ATTEMPT_TO_FIX_PASSED, True)
+
+            retry_test.set_tag(TEST_ATTEMPT_TO_FIX_PASSED, all_passed)
 
         retry_test.finish_test(status, exc_info=exc_info)
 

tests/contrib/pytest/test_pytest_attempt_to_fix.py

@@ -138,7 +138,7 @@ def test_attempt_to_fix_quarantined_test_fail(self):
             assert span.get_tag("test.test_management.is_quarantined") == "true"
             assert span.get_tag("test.status") == "fail"
 
-        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") is None
+        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") == "false"
         assert test_spans[-1].get_tag("test.has_failed_all_retries") == "true"
 
     def test_attempt_to_fix_quarantined_test_flaky(self):
@@ -158,7 +158,7 @@ def test_attempt_to_fix_quarantined_test_flaky(self):
             assert span.get_tag("test.test_management.is_quarantined") == "true"
             assert span.get_tag("test.status") == ("pass" if i == 10 else "fail")
 
-        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") is None
+        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") == "false"
         assert test_spans[-1].get_tag("test.has_failed_all_retries") is None
 
     def test_attempt_to_fix_disabled_test(self):
@@ -221,7 +221,7 @@ def test_attempt_to_fix_active_test_fail(self):
             assert span.get_tag("test.test_management.is_test_disabled") is None
             assert span.get_tag("test.status") == "fail"
 
-        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") is None
+        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") == "false"
         assert test_spans[-1].get_tag("test.has_failed_all_retries") == "true"
 
     def test_attempt_to_fix_active_test_skip(self):
@@ -242,7 +242,7 @@ def test_attempt_to_fix_active_test_skip(self):
             assert span.get_tag("test.test_management.is_test_disabled") is None
             assert span.get_tag("test.status") == "skip"
 
-        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") is None
+        assert test_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") == "false"
         assert test_spans[-1].get_tag("test.has_failed_all_retries") is None
 
     def test_pytest_attempt_to_fix_junit_xml_active(self):
@@ -319,5 +319,5 @@ def test_attempt_to_fix_itr_skipped_test(self):
             assert span.get_tag("test.test_management.is_test_disabled") == "true"
             assert span.get_tag("test.status") == "fail"
 
-        assert attempt_to_fix_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") is None
+        assert attempt_to_fix_spans[-1].get_tag("test.test_management.attempt_to_fix_passed") == "false"
         assert attempt_to_fix_spans[-1].get_tag("test.has_failed_all_retries") == "true"