fix: add log for url on retry failure (#613)

codecov · Feb 5, 2025 · 0728eb0 · 0728eb0
1 parent 613107a
commit 0728eb0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/codecov_cli/helpers/request.py b/codecov_cli/helpers/request.py
@@ -79,7 +79,7 @@ def wrapper(*args, **kwargs):
                 )
                 sleep(backoff_time(retry))
                 retry += 1
-        raise Exception("Request failed after too many retries")
+        raise Exception(f"Request failed after too many retries. URL: {kwargs.get('url', args[0] if args else 'Unknown')}")
 
     return wrapper
 

diff --git a/tests/helpers/test_request.py b/tests/helpers/test_request.py
@@ -151,7 +151,7 @@ def test_request_retry_too_many_errors(mocker):
     )
     with pytest.raises(Exception) as exp:
         _ = send_post_request("my_url")
-    assert str(exp.value) == "Request failed after too many retries"
+    assert str(exp.value) == "Request failed after too many retries. URL: my_url"
 
 
 def test_user_agent(mocker):

diff --git a/tests/helpers/test_upload_sender.py b/tests/helpers/test_upload_sender.py
@@ -366,7 +366,6 @@ def test_upload_sender_result_fail_post_500s(
         matcher = re.compile(
             rf"(warning.*((Response status code was {error_code})|(Request failed\. Retrying)).*(\n)?){{6}}"
         )
-
         assert matcher.match(capsys.readouterr().err) is not None
 
     def test_upload_sender_result_fail_put_400(