diff --git a/codecov_cli/helpers/request.py b/codecov_cli/helpers/request.py index 27bd3be08..3641d6dd9 100644 --- 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 index 7069068b8..769e3f3a0 100644 --- 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 index ff24329d1..b727708b3 100644 --- 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(