dont raise NotAnIdentifier function

snakefmt/exceptions.py

@@ -10,8 +10,10 @@ class EmptyContextError(Exception):
     pass
 
 
-class NotAnIdentifierError(Exception):
-    pass
+def NotAnIdentifierError(line_nb: str, identifier: str, keyword_line: str):
+    raise SyntaxError(
+        f"{line_nb}'{identifier}' in '{keyword_line}' is not a valid identifier"
+    )
 
 
 def ColonError(line_nb: str, identifier: str, keyword_line: str):

snakefmt/parser/syntax.py

@@ -268,10 +268,7 @@ def validate_userule_syntax(self, snakefile: TokenIterator):
     def validate_rulelike_syntax(self, snakefile: TokenIterator):
         if not is_colon(self.token):
             if self.token.type != tokenize.NAME:
-                raise NotAnIdentifierError(
-                    f"{self.line_nb}'{self.token.string}' in '{self.keyword_line}' is "
-                    f"not a valid identifier"
-                )
+                NotAnIdentifierError(self.line_nb, self.token.string, self.keyword_line)
             self.keyword_line += f" {self.token.string}"
             self.token = next(snakefile)
         if not is_colon(self.token):

tests/test_parser.py

@@ -12,7 +12,6 @@
     InvalidParameterSyntax,
     InvalidPython,
     NoParametersError,
-    NotAnIdentifierError,
     TooManyParameters,
 )
 from snakefmt.formatter import TAB
@@ -63,9 +62,7 @@ def test_keyword_cannot_be_named(self):
             setup_formatter('workdir a: "/to/dir"')
 
     def test_invalid_name_for_keyword(self):
-        with pytest.raises(
-            NotAnIdentifierError, match=".*checkpoint.*valid identifier"
-        ):
+        with pytest.raises(SyntaxError, match=".*checkpoint.*valid identifier"):
             setup_formatter("checkpoint (): \n" '\tinput: "a"')
 
     def test_explicitly_unrecognised_keyword(self):