Skip to content

Commit ecc1886

Browse files
committed
Remove unneeded return.
`skipTest` works by raising an exception, so it automatically interrupts the current code flow, there's no need for an explicit `return`.
1 parent 9ae0876 commit ecc1886

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

tests/test_download.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ def test_download_once(self) -> None:
3333
code = download_file(self.url, self.downloaded_file, ri, max_retry=5)
3434
if code != 200:
3535
self.skipTest(f"Could not dowload from {self.url}, cannot test")
36-
return
3736

3837
self.assertTrue(self.downloaded_file.exists())
3938
mtime = self.downloaded_file.stat().st_mtime
@@ -55,7 +54,6 @@ def test_no_overwrite_identical_file(self) -> None:
5554
code = download_file(self.url, self.downloaded_file, ri, max_retry=5)
5655
if code != 200:
5756
self.skipTest(f"Could not download from {self.url}, cannot test")
58-
return
5957

6058
mtime = self.downloaded_file.stat().st_mtime
6159

0 commit comments

Comments
 (0)