Merge branch 'main' into dependabot/github_actions/softprops/action-g…

…h-release-2.0.8
repository-service-tuf · Jul 27, 2024 · 1c42624 · 1c42624
2 parents 0dfe400 + 47dd369
commit 1c42624
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 15 deletions.
diff --git a/Pipfile.lock b/Pipfile.lock
diff --git a/repository_service_tuf/cli/admin/sign.py b/repository_service_tuf/cli/admin/sign.py
@@ -36,10 +36,7 @@
 
 
 def _parse_pending_data(pending_roles_resp: Dict[str, Any]) -> Dict[str, Any]:
-    data = pending_roles_resp.get("data")
-    if data is None:
-        error = "'data' field missing from api server response/file input"
-        raise click.ClickException(error)
+    data = pending_roles_resp.get("data", {})
 
     pending_roles: Dict[str, Dict[str, Any]] = data.get("metadata", {})
     if len(pending_roles) == 0:

diff --git a/requirements-dev.txt b/requirements-dev.txt
@@ -45,7 +45,7 @@ pyflakes==3.2.0; python_version >= '3.8'
 pygments==2.18.0; python_version >= '3.8'
 pyproject-api==1.7.1; python_version >= '3.8'
 pyproject-hooks==1.1.0; python_version >= '3.7'
-pytest==8.3.1; python_version >= '3.8'
+pytest==8.3.2; python_version >= '3.8'
 pytest-cov==5.0.0; python_version >= '3.8'
 pytest-xdist==3.6.1; python_version >= '3.8'
 pyyaml==6.0.1; python_version >= '3.6'

diff --git a/tests/unit/cli/admin/test_sign.py b/tests/unit/cli/admin/test_sign.py
@@ -370,13 +370,6 @@ def test__parse_pending_data_missing_metadata(self):
 
         assert "No metadata available for signing" in str(e)
 
-    def test__parse_pending_data_missing_data(self):
-        with pytest.raises(click.ClickException) as e:
-            sign._parse_pending_data({})
-
-        err = "'data' field missing from api server response/file input"
-        assert err in str(e)
-
     def test__get_pending_roles_request(self, monkeypatch):
         fake_settings = pretend.stub(SERVER=None)
         fake_json = pretend.stub()