Rename to assert_complete_download

Author: VeckoTheGecko

src/virtualship/cli/_fetch.py

@@ -85,14 +85,13 @@ def get_existing_download(data_folder: Path, aoi_hash: str) -> Path | None:
             continue
 
         if hash == aoi_hash:
-            check_complete_download(download_path)
+            assert_complete_download(download_path)
             return download_path
 
     return None
 
 
-def check_complete_download(download_path: Path) -> bool:
-    """Check if a download is complete."""
+def assert_complete_download(download_path: Path) -> None:
     download_metadata = download_path / DOWNLOAD_METADATA
     try:
         with open(download_metadata) as file:
@@ -102,7 +101,7 @@ def check_complete_download(download_path: Path) -> bool:
             f"Download at {download_path} was found, but looks to be incomplete "
             f"(likely due to interupting it mid-download). Please delete this folder and retry."
         ) from e
-    return True
+    return
 
 
 def complete_download(download_path: Path) -> None:

tests/cli/test_fetch.py

@@ -7,7 +7,7 @@
     DOWNLOAD_METADATA,
     DownloadMetadata,
     IncompleteDownloadError,
-    check_complete_download,
+    assert_complete_download,
     complete_download,
     create_hash,
     filename_to_hash,
@@ -42,27 +42,27 @@ def test_complete_download(tmp_path):
 
     complete_download(tmp_path)
 
-    assert check_complete_download(tmp_path)
+    assert_complete_download(tmp_path)
 
 
-def test_check_complete_download_complete(tmp_path):
+def test_assert_complete_download_complete(tmp_path):
     # Setup
     DownloadMetadata(download_complete=True).to_yaml(tmp_path / DOWNLOAD_METADATA)
 
-    assert check_complete_download(tmp_path)
+    assert_complete_download(tmp_path)
 
 
-def test_check_complete_download_incomplete(tmp_path):
+def test_assert_complete_download_incomplete(tmp_path):
     # Setup
     DownloadMetadata(download_complete=False).to_yaml(tmp_path / DOWNLOAD_METADATA)
 
     with pytest.raises(IncompleteDownloadError):
-        check_complete_download(tmp_path)
+        assert_complete_download(tmp_path)
 
 
-def test_check_complete_download_missing(tmp_path):
+def test_assert_complete_download_missing(tmp_path):
     with pytest.raises(IncompleteDownloadError):
-        assert not check_complete_download(tmp_path)
+        assert_complete_download(tmp_path)
 
 
 @pytest.fixture
@@ -76,7 +76,7 @@ def existing_data_folder(tmp_path, monkeypatch):
     ]
     data_folder = tmp_path
     monkeypatch.setattr(
-        "virtualship.cli._fetch.check_complete_download", lambda x: True
+        "virtualship.cli._fetch.assert_complete_download", lambda x: None
     )
     for f in folders:
         (data_folder / f).mkdir()