Merge branch 'main' into th/rebuild-packaging

thomasrockhu-codecov · web-flow · commit b2c63c871bbd · 2025-02-05T18:18:22.000-03:00
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(

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ def wrapper(args, *kwargs):`
`79`	`79`	`)`
`80`	`80`	`sleep(backoff_time(retry))`
`81`	`81`	`retry += 1`
`82`		`- raise Exception("Request failed after too many retries")`
	`82`	`+ raise Exception(f"Request failed after too many retries. URL: {kwargs.get('url', args[0] if args else 'Unknown')}")`
`83`	`83`
`84`	`84`	`return wrapper`
`85`	`85`
Original file line number	Diff line number	Diff line change
`@@ -151,7 +151,7 @@ def test_request_retry_too_many_errors(mocker):`
`151`	`151`	`)`
`152`	`152`	`with pytest.raises(Exception) as exp:`
`153`	`153`	`_ = send_post_request("my_url")`
`154`		`- assert str(exp.value) == "Request failed after too many retries"`
	`154`	`+ assert str(exp.value) == "Request failed after too many retries. URL: my_url"`
`155`	`155`
`156`	`156`
`157`	`157`	`def test_user_agent(mocker):`
Original file line number	Diff line number	Diff line change
`@@ -366,7 +366,6 @@ def test_upload_sender_result_fail_post_500s(`
`366`	`366`	`matcher = re.compile(`
`367`	`367`	`rf"(warning.((Response status code was {error_code})\|(Request failed\. Retrying)).(\n)?){{6}}"`
`368`	`368`	`)`
`369`		`-`
`370`	`369`	`assert matcher.match(capsys.readouterr().err) is not None`
`371`	`370`
`372`	`371`	`def test_upload_sender_result_fail_put_400(`