Again resolve SonarCloud complain: Remove unused parameter token

NatLibFi · Sep 30, 2024 · caf0325 · caf0325
1 parent 4dfa2d1
commit caf0325
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/annif/cli.py b/annif/cli.py
@@ -714,7 +714,7 @@ def run_download(project_ids_pattern, repo_id, token, revision, force, trust_rep
     `--trust-repo` option needs to be used.
     """
 
-    hfh_util.check_is_download_allowed(trust_repo, repo_id, token)
+    hfh_util.check_is_download_allowed(trust_repo, repo_id)
 
     project_ids = hfh_util.get_matching_project_ids_from_hf_hub(
         project_ids_pattern, repo_id, token, revision

diff --git a/annif/hfh_util.py b/annif/hfh_util.py
@@ -24,7 +24,7 @@
 logger = annif.logger
 
 
-def check_is_download_allowed(trust_repo, repo_id, token):
+def check_is_download_allowed(trust_repo, repo_id):
     """Check if downloading from the specified repository is allowed based on the trust
     option and cache status."""
     if trust_repo:

diff --git a/tests/test_hfh_util.py b/tests/test_hfh_util.py
@@ -20,10 +20,9 @@
 def test_download_allowed_trust_repo(mock_is_repo_in_cache, caplog):
     trust_repo = True
     repo_id = "dummy-repo"
-    token = "dummy-token"
 
     with caplog.at_level(logging.WARNING, logger="annif"):
-        annif.hfh_util.check_is_download_allowed(trust_repo, repo_id, token)
+        annif.hfh_util.check_is_download_allowed(trust_repo, repo_id)
     assert (
         'Download allowed from "dummy-repo" because "--trust-repo" flag is used.'
         in caplog.text
@@ -34,10 +33,9 @@ def test_download_allowed_trust_repo(mock_is_repo_in_cache, caplog):
 def test_download_allowed_repo_in_cache(mock_is_repo_in_cache, caplog):
     trust_repo = False
     repo_id = "dummy-repo"
-    token = "dummy-token"
 
     with caplog.at_level(logging.DEBUG, logger="annif"):
-        annif.hfh_util.check_is_download_allowed(trust_repo, repo_id, token)
+        annif.hfh_util.check_is_download_allowed(trust_repo, repo_id)
     assert (
         'Download allowed from "dummy-repo" because repo is already in cache.'
         in caplog.text
@@ -48,10 +46,9 @@ def test_download_allowed_repo_in_cache(mock_is_repo_in_cache, caplog):
 def test_download_not_allowed(mock_is_repo_in_cache):
     trust_repo = False
     repo_id = "dummy-repo"
-    token = "dummy-token"
 
     with pytest.raises(OperationFailedException) as excinfo:
-        annif.hfh_util.check_is_download_allowed(trust_repo, repo_id, token)
+        annif.hfh_util.check_is_download_allowed(trust_repo, repo_id)
     assert (
         str(excinfo.value)
         == 'Cannot download projects from untrusted repo "dummy-repo"'