test(message_length_limit): validate negative message_length_limit

Author: ttw225

tests/commands/test_check_command.py

@@ -392,6 +392,29 @@ def test_check_command_cli_overrides_config_message_length_limit(
         success_mock.assert_called_once()
 
 
+def test_check_command_with_negative_cli_message_length_limit_raises(config):
+    with pytest.raises(InvalidCommandArgumentError):
+        commands.Check(
+            config=config,
+            arguments={
+                "message": "fix(scope): some commit message",
+                "message_length_limit": -1,
+            },
+        )
+
+
+def test_check_command_with_negative_config_message_length_limit_raises(config):
+    config.settings["message_length_limit"] = -1
+    with pytest.raises(InvalidCommandArgumentError):
+        commands.Check(
+            config=config,
+            arguments={
+                "message": "fix(scope): some commit message",
+                "message_length_limit": None,
+            },
+        )
+
+
 class ValidationCz(BaseCommitizen):
     def questions(self) -> list[CzQuestion]:
         return [

tests/commands/test_commit_command.py

@@ -12,6 +12,7 @@
     CommitMessageLengthExceededError,
     CustomError,
     DryRunExit,
+    InvalidCommandArgumentError,
     NoAnswersError,
     NoCommitBackupError,
     NotAGitProjectError,
@@ -403,3 +404,14 @@ def test_commit_message_length_cli_zero_disables_limit(
     )
     commands.Commit(config, {"message_length_limit": 0})()
     success_mock.assert_called_once()
+
+
+def test_commit_message_length_cli_negative_raises(config):
+    with pytest.raises(InvalidCommandArgumentError):
+        commands.Commit(config, {"message_length_limit": -1})
+
+
+def test_commit_message_length_config_negative_raises_when_cli_unset(config):
+    config.settings["message_length_limit"] = -1
+    with pytest.raises(InvalidCommandArgumentError):
+        commands.Commit(config, {"message_length_limit": None})