Validate full model name in the deprecate model flow (#427)

Earlier, the model name was not being validated for the deprecate model
action. It was getting validated for all other actions including the
unassign and deregister actions (which are variations of the deprecate
action).

Resolves
iterative/dvc#9821 (comment)

gto/constants.py

@@ -72,7 +72,7 @@ def assert_name_is_valid(value):
     if not check_string_is_valid(value, regex=name_re):
         raise ValidationError(
             f"Invalid value '{value}'. Only letters, numbers, '_', '-', '/' are allowed."
-            "Value must be of len >= 2 and must start and end with a letter or a number."
+            " Value must be of len >= 2 and must start and end with a letter or a number."
         )
 
 
@@ -81,7 +81,7 @@ def assert_fullname_is_valid(value):
         # fix error message to be regex-specific
         raise ValidationError(
             f"Invalid value '{value}'. Only letters, numbers, '_', '-', '/' are allowed."
-            "Value must be of len >= 2 and must start and end with a letter or a number."
+            " Value must be of len >= 2 and must start and end with a letter or a number."
         )
 
 

gto/registry.py

@@ -421,6 +421,8 @@ def deprecate(
         author: Optional[str] = None,
         author_email: Optional[str] = None,
     ) -> Optional[Deprecation]:
+        """Deprecate artifact"""
+        assert_fullname_is_valid(name)
         if force:
             if simple:
                 raise WrongArgs("Can't use 'force' with 'simple=True'")