Validate full model name in the deprecate model flow

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'")

tests/test_cli.py

@@ -348,6 +348,16 @@ def test_assign(repo_with_commit: str):
     )
 
 
+def test_deprecate_artifact(repo_with_commit: str):
+    _check_failing_cmd(
+        "deprecate",
+        ["-r", repo_with_commit, "a4!"],
+        "❌ Invalid value 'a4!'."
+        " Only letters, numbers, '_', '-', '/' are allowed."
+        " Value must be of len >= 2 and must start and end with a letter or a number.\n",
+    )
+
+
 GTO_EXCEPTION_MESSAGE = "Test GTOException Message"