Log warnings for retries

tests/test_repository.py

@@ -230,7 +230,7 @@ def dictfunc():
     default_repo.upload(package)
 
 
-def test_upload_retry(tmpdir, default_repo, capsys):
+def test_upload_retry(tmpdir, default_repo, caplog):
     """Print retry messages when the upload response indicates a server error."""
     default_repo.disable_progress_bar = True
 
@@ -254,33 +254,27 @@ def test_upload_retry(tmpdir, default_repo, capsys):
     # Upload with default max_redirects of 5
     default_repo.upload(package)
 
-    msg = [
+    assert caplog.messages == [
         (
-            "Uploading fake.whl\n"
             'Received "500: Internal server error"\n'
             f"Package upload appears to have failed. Retry {i} of 5."
         )
         for i in range(1, 6)
     ]
 
-    captured = capsys.readouterr()
-    assert captured.out == "\n".join(msg) + "\n"
+    caplog.clear()
 
     # Upload with custom max_redirects of 3
     default_repo.upload(package, 3)
 
-    msg = [
+    assert caplog.messages == [
         (
-            "Uploading fake.whl\n"
             'Received "500: Internal server error"\n'
             f"Package upload appears to have failed. Retry {i} of 3."
         )
         for i in range(1, 4)
     ]
 
-    captured = capsys.readouterr()
-    assert captured.out == "\n".join(msg) + "\n"
-
 
 @pytest.mark.parametrize(
     "package_meta,repository_url,release_urls",

twine/repository.py

@@ -193,8 +193,8 @@ def upload(
                 return resp
             if 500 <= resp.status_code < 600:
                 number_of_redirects += 1
-                print(
-                    f'[yellow]Received "{resp.status_code}: {resp.reason}"'
+                logger.warning(
+                    f'Received "{resp.status_code}: {resp.reason}"'
                     "\nPackage upload appears to have failed."
                     f" Retry {number_of_redirects} of {max_redirects}."
                 )