diff --git a/swh/loader/package/tests/test_utils.py b/swh/loader/package/tests/test_utils.py
--- a/swh/loader/package/tests/test_utils.py
+++ b/swh/loader/package/tests/test_utils.py
@@ -34,22 +34,19 @@
     assert e.value.args[0] == "Fail to query '%s'. Reason: %s" % (url, status_code)
 
 
-@pytest.mark.fs
-def test_download_ok(tmp_path, requests_mock):
-    """Download without issue should provide filename and hashes"""
-    filename = "requests-0.0.1.tar.gz"
-    url = "https://pypi.org/pypi/requests/%s" % filename
-    data = "this is something"
-    requests_mock.get(url, text=data, headers={"content-length": str(len(data))})
+_filename = "requests-0.0.1.tar.gz"
+_data = "this is something"
 
-    actual_filepath, actual_hashes = download(url, dest=str(tmp_path))
+
+def _check_download_ok(url, dest, hashes=None):
+    actual_filepath, actual_hashes = download(url, dest, hashes=hashes)
 
     actual_filename = os.path.basename(actual_filepath)
-    assert actual_filename == filename
-    assert actual_hashes["length"] == len(data)
+    assert actual_filename == _filename
+    assert actual_hashes["length"] == len(_data)
     assert (
         actual_hashes["checksums"]["sha1"] == "fdd1ce606a904b08c816ba84f3125f2af44d92b2"
-    )  # noqa
+    )
     assert (
         actual_hashes["checksums"]["sha256"]
         == "1d9224378d77925d612c9f926eb9fb92850e6551def8328011b6a972323298d5"
@@ -57,34 +54,26 @@
 
 
 @pytest.mark.fs
-def test_download_ok_no_header(tmp_path, requests_mock):
+def test_download_ok(tmp_path, requests_mock):
     """Download without issue should provide filename and hashes"""
-    filename = "requests-0.0.1.tar.gz"
-    url = "https://pypi.org/pypi/requests/%s" % filename
-    data = "this is something"
-    requests_mock.get(url, text=data)  # no header information
+    url = f"https://pypi.org/pypi/requests/{_filename}"
+    requests_mock.get(url, text=_data, headers={"content-length": str(len(_data))})
+    _check_download_ok(url, dest=str(tmp_path))
 
-    actual_filepath, actual_hashes = download(url, dest=str(tmp_path))
 
-    actual_filename = os.path.basename(actual_filepath)
-    assert actual_filename == filename
-    assert actual_hashes["length"] == len(data)
-    assert (
-        actual_hashes["checksums"]["sha1"] == "fdd1ce606a904b08c816ba84f3125f2af44d92b2"
-    )  # noqa
-    assert (
-        actual_hashes["checksums"]["sha256"]
-        == "1d9224378d77925d612c9f926eb9fb92850e6551def8328011b6a972323298d5"
-    )
+@pytest.mark.fs
+def test_download_ok_no_header(tmp_path, requests_mock):
+    """Download without issue should provide filename and hashes"""
+    url = f"https://pypi.org/pypi/requests/{_filename}"
+    requests_mock.get(url, text=_data)  # no header information
+    _check_download_ok(url, dest=str(tmp_path))
 
 
 @pytest.mark.fs
 def test_download_ok_with_hashes(tmp_path, requests_mock):
     """Download without issue should provide filename and hashes"""
-    filename = "requests-0.0.1.tar.gz"
-    url = "https://pypi.org/pypi/requests/%s" % filename
-    data = "this is something"
-    requests_mock.get(url, text=data, headers={"content-length": str(len(data))})
+    url = f"https://pypi.org/pypi/requests/{_filename}"
+    requests_mock.get(url, text=_data, headers={"content-length": str(len(_data))})
 
     # good hashes for such file
     good = {
@@ -92,13 +81,7 @@
         "sha256": "1d9224378d77925d612c9f926eb9fb92850e6551def8328011b6a972323298d5",  # noqa
     }
 
-    actual_filepath, actual_hashes = download(url, dest=str(tmp_path), hashes=good)
-
-    actual_filename = os.path.basename(actual_filepath)
-    assert actual_filename == filename
-    assert actual_hashes["length"] == len(data)
-    assert actual_hashes["checksums"]["sha1"] == good["sha1"]
-    assert actual_hashes["checksums"]["sha256"] == good["sha256"]
+    _check_download_ok(url, dest=str(tmp_path), hashes=good)
 
 
 @pytest.mark.fs
@@ -106,10 +89,8 @@
     """Mismatch hash after download should raise
 
     """
-    filename = "requests-0.0.1.tar.gz"
-    url = "https://pypi.org/pypi/requests/%s" % filename
-    data = "this is something"
-    requests_mock.get(url, text=data, headers={"content-length": str(len(data))})
+    url = f"https://pypi.org/pypi/requests/{_filename}"
+    requests_mock.get(url, text=_data, headers={"content-length": str(len(_data))})
 
     # good hashes for such file
     good = {
@@ -135,28 +116,15 @@
 @pytest.mark.fs
 def test_ftp_download_ok(tmp_path, mocker):
     """Download without issue should provide filename and hashes"""
-    filename = "requests-0.0.1.tar.gz"
-    url = "ftp://pypi.org/pypi/requests/%s" % filename
-    data = b"this is something"
+    url = f"ftp://pypi.org/pypi/requests/{_filename}"
 
     cm = MagicMock()
     cm.getstatus.return_value = 200
-    cm.read.side_effect = [data, b""]
+    cm.read.side_effect = [_data.encode(), b""]
     cm.__enter__.return_value = cm
     mocker.patch("swh.loader.package.utils.urlopen").return_value = cm
 
-    actual_filepath, actual_hashes = download(url, dest=str(tmp_path))
-
-    actual_filename = os.path.basename(actual_filepath)
-    assert actual_filename == filename
-    assert actual_hashes["length"] == len(data)
-    assert (
-        actual_hashes["checksums"]["sha1"] == "fdd1ce606a904b08c816ba84f3125f2af44d92b2"
-    )  # noqa
-    assert (
-        actual_hashes["checksums"]["sha256"]
-        == "1d9224378d77925d612c9f926eb9fb92850e6551def8328011b6a972323298d5"
-    )
+    _check_download_ok(url, dest=str(tmp_path))
 
 
 @pytest.mark.fs
@@ -175,27 +143,14 @@
 def test_download_with_redirection(tmp_path, requests_mock):
     """Download with redirection should use the targeted URL to extract filename"""
     url = "https://example.org/project/requests/download"
-    filename = "requests-0.0.1.tar.gz"
-    redirection_url = f"https://example.org/project/requests/files/{filename}"
-    data = "this is something"
+    redirection_url = f"https://example.org/project/requests/files/{_filename}"
 
     requests_mock.get(url, status_code=302, headers={"location": redirection_url})
     requests_mock.get(
-        redirection_url, text=data, headers={"content-length": str(len(data))}
+        redirection_url, text=_data, headers={"content-length": str(len(_data))}
     )
 
-    actual_filepath, actual_hashes = download(url, dest=str(tmp_path))
-
-    actual_filename = os.path.basename(actual_filepath)
-    assert actual_filename == filename
-    assert actual_hashes["length"] == len(data)
-    assert (
-        actual_hashes["checksums"]["sha1"] == "fdd1ce606a904b08c816ba84f3125f2af44d92b2"
-    )  # noqa
-    assert (
-        actual_hashes["checksums"]["sha256"]
-        == "1d9224378d77925d612c9f926eb9fb92850e6551def8328011b6a972323298d5"
-    )
+    _check_download_ok(url, dest=str(tmp_path))
 
 
 @pytest.mark.fs
@@ -203,31 +158,17 @@
     """Filename should be extracted from content-disposition request header
     when available."""
     url = "https://example.org/download/requests/tar.gz/v0.0.1"
-    filename = "requests-0.0.1.tar.gz"
-
-    data = "this is something"
 
     requests_mock.get(
         url,
-        text=data,
+        text=_data,
         headers={
-            "content-length": str(len(data)),
-            "content-disposition": f"attachment; filename={filename}",
+            "content-length": str(len(_data)),
+            "content-disposition": f"attachment; filename={_filename}",
         },
     )
 
-    actual_filepath, actual_hashes = download(url, dest=str(tmp_path))
-
-    actual_filename = os.path.basename(actual_filepath)
-    assert actual_filename == filename
-    assert actual_hashes["length"] == len(data)
-    assert (
-        actual_hashes["checksums"]["sha1"] == "fdd1ce606a904b08c816ba84f3125f2af44d92b2"
-    )  # noqa
-    assert (
-        actual_hashes["checksums"]["sha256"]
-        == "1d9224378d77925d612c9f926eb9fb92850e6551def8328011b6a972323298d5"
-    )
+    _check_download_ok(url, dest=str(tmp_path))
 
 
 def test_api_info_failure(requests_mock):