diff --git a/src/plone/api/addon.py b/src/plone/api/addon.py index 510fe386..816d8378 100644 --- a/src/plone/api/addon.py +++ b/src/plone/api/addon.py @@ -25,7 +25,7 @@ "AddonInformation", "NonInstallableAddons", "get_addons", - "get_addons_ids", + "get_addon_ids", "get_version", "get", "install", @@ -217,11 +217,11 @@ def get_addons(limit: str = "") -> List[AddonInformation]: if limit in ("installed", "upgradable", "available"): addons = [addon for addon in addons if limit in addon.flags] elif limit != "": - raise InvalidParameterError(f"Value {limit} for parameter mode is not valid.") + raise InvalidParameterError(f"Parameter {limit=} is not valid.") return addons -def get_addons_ids(limit: str = "") -> List[str]: +def get_addon_ids(limit: str = "") -> List[str]: """List addons ids in this Plone site. :param limit: Limit list of addons. diff --git a/src/plone/api/tests/test_addon.py b/src/plone/api/tests/test_addon.py index eae1aa01..0e1f8564 100644 --- a/src/plone/api/tests/test_addon.py +++ b/src/plone/api/tests/test_addon.py @@ -56,12 +56,12 @@ def test_api_get_addons_limit_invalid(self): with self.assertRaises(api.exc.InvalidParameterError) as cm: api.addon.get_addons(limit="foobar") self.assertIn( - "Value foobar for parameter mode is not valid.", str(cm.exception) + "Parameter limit='foobar' is not valid.", str(cm.exception) ) - def test_api_get_addons_ids(self): - """Test api.addon.get_addons_ids.""" - result = api.addon.get_addons_ids(limit="installed") + def test_api_get_addon_ids(self): + """Test api.addon.get_addon_ids.""" + result = api.addon.get_addon_ids(limit="installed") self.assertEqual(len(result), 2) self.assertIn(ADDON, result)