fixup

Author: nickanderson

promise-types/appstreams/appstreams.py

@@ -76,6 +76,15 @@ def _validate_state(self, value):
             accepted_str = "', '".join(accepted)
             raise ValidationError(f"State attribute must be '{accepted_str}'")
 
+    def _validate_module_name(self, name):
+        self._validate_identifier(name, "module name")
+
+    def _validate_stream_name(self, stream):
+        self._validate_identifier(stream, "stream name", required=False)
+
+    def _validate_profile_name(self, profile):
+        self._validate_identifier(profile, "profile name", required=False)
+
     def _validate_identifier(self, value, label, required=True):
         if not required and not value:
             return
@@ -224,8 +233,6 @@ def _get_module_state(self, mpc, module_name):
             return "disabled"
         elif state == mpc.ModuleState_INSTALLED:
             return "installed"
-        elif state == mpc.ModuleState_DEFAULT:
-            return "default"
         return "removed"
 
     def _is_module_installed_with_packages(
@@ -456,7 +463,7 @@ def _remove_module(self, mpc, base, module_name, stream, profile):
 
     def _reset_module(self, mpc, base, module_name):
         """Reset a module to factory state — no stream pinned, no enabled/disabled flag."""
-        if self._get_module_state(mpc, module_name) == "default":
+        if mpc.getModuleState(module_name) == mpc.ModuleState_DEFAULT:
             self.log_verbose(
                 f"Module {module_name} is already in default (reset) state"
             )