refactor: rename check_failing_spans_with_quarantine (#747)

Depends-On: #746

Author: Greesb

mergify_cli/ci/cli.py

@@ -200,7 +200,7 @@ async def _process_junit_files(  # noqa: PLR0913
     # "cicd.test.quarantined" attribute for the required spans.
     try:
         failing_tests_not_quarantined_count = (
-            await quarantine.check_failing_spans_with_quarantine(
+            await quarantine.check_and_update_failing_spans(
                 api_url,
                 token,
                 repository,

mergify_cli/ci/quarantine.py

@@ -22,7 +22,7 @@ class QuarantineFailedError(Exception):
 )
 
 
-async def check_failing_spans_with_quarantine(
+async def check_and_update_failing_spans(
     api_url: str,
     token: str,
     repository: str,

mergify_cli/tests/ci/test_check_failing_spans.py

@@ -5,7 +5,7 @@
 import respx
 
 from mergify_cli.ci.quarantine import QuarantineFailedError
-from mergify_cli.ci.quarantine import check_failing_spans_with_quarantine
+from mergify_cli.ci.quarantine import check_and_update_failing_spans
 
 
 API_MERGIFY_BASE_URL = "https://api.mergify.com"
@@ -29,7 +29,7 @@ async def test_status_code_resp_not_200(
     ]
 
     with pytest.raises(QuarantineFailedError):
-        await check_failing_spans_with_quarantine(
+        await check_and_update_failing_spans(
             API_MERGIFY_BASE_URL,
             "token",
             "foo/bar",
@@ -61,7 +61,7 @@ async def test_no_failing_tests_quarantined(
         ),
     ]
 
-    failed_tests_quarantined_test_count = await check_failing_spans_with_quarantine(
+    failed_tests_quarantined_test_count = await check_and_update_failing_spans(
         API_MERGIFY_BASE_URL,
         "token",
         "foo/bar",
@@ -109,7 +109,7 @@ async def test_some_failing_tests_quarantined(
         ),
     ]
 
-    failed_tests_quarantined_count = await check_failing_spans_with_quarantine(
+    failed_tests_quarantined_count = await check_and_update_failing_spans(
         API_MERGIFY_BASE_URL,
         "token",
         "foo/bar",
@@ -166,7 +166,7 @@ async def test_all_failing_tests_quarantined(
         ),
     ]
 
-    failed_tests_quarantined_count = await check_failing_spans_with_quarantine(
+    failed_tests_quarantined_count = await check_and_update_failing_spans(
         API_MERGIFY_BASE_URL,
         "token",
         "foo/bar",

mergify_cli/tests/ci/test_cli.py

@@ -59,7 +59,7 @@ def test_cli(env: dict[str, str], monkeypatch: pytest.MonkeyPatch) -> None:
         ) as mocked_upload,
         mock.patch.object(
             quarantine,
-            "check_failing_spans_with_quarantine",
+            "check_and_update_failing_spans",
             return_value=0,
         ),
     ):
@@ -203,7 +203,7 @@ def test_upload_error(monkeypatch: pytest.MonkeyPatch) -> None:
         ) as mocked_upload,
         mock.patch.object(
             quarantine,
-            "check_failing_spans_with_quarantine",
+            "check_and_update_failing_spans",
             return_value=0,
         ),
     ):