Skip to content

Commit 530ec12

Browse files
committed
Rework retries test
1 parent ad1259a commit 530ec12

File tree

1 file changed

+12
-22
lines changed

1 file changed

+12
-22
lines changed

tests/test_repository.py

Lines changed: 12 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -254,35 +254,25 @@ def test_upload_retry(tmpdir, default_repo, capsys):
254254
metadata_dictionary=lambda: {"name": "fake"},
255255
)
256256

257+
def assert_retries(output, total):
258+
retries = [line for line in output.splitlines() if line.startswith("Received")]
259+
assert retries == [
260+
(
261+
'Received "500: Internal server error" '
262+
f"Package upload appears to have failed. Retry {i} of {total}"
263+
)
264+
for i in range(1, total + 1)
265+
]
266+
257267
# Upload with default max_redirects of 5
258268
default_repo.upload(package)
259269

260-
msg = [
261-
(
262-
"Uploading fake.whl\n"
263-
'Received "500: Internal server error" '
264-
f"Package upload appears to have failed. Retry {i} of 5"
265-
)
266-
for i in range(1, 6)
267-
]
268-
269-
captured = capsys.readouterr()
270-
assert captured.out == "\n".join(msg) + "\n"
270+
assert_retries(capsys.readouterr().out, 5)
271271

272272
# Upload with custom max_redirects of 3
273273
default_repo.upload(package, 3)
274274

275-
msg = [
276-
(
277-
"Uploading fake.whl\n"
278-
'Received "500: Internal server error" '
279-
f"Package upload appears to have failed. Retry {i} of 3"
280-
)
281-
for i in range(1, 4)
282-
]
283-
284-
captured = capsys.readouterr()
285-
assert captured.out == "\n".join(msg) + "\n"
275+
assert_retries(capsys.readouterr().out, 3)
286276

287277

288278
@pytest.mark.parametrize(

0 commit comments

Comments
 (0)