test(add-tests-to-improve-code-coverage-for-commit.py): Add missing test cases to cover previously untested code paths in the commit command:

- Test ValueError with CzException context in _handle_questionary_prompt
- Test manual_edit when editor executable is not found
- Test multiline question handling when result is None with/without default values

Author: anasalaqeel

tests/commands/test_commit_command.py

@@ -327,6 +327,24 @@ def test_commit_when_customized_expected_raised(config, mocker: MockFixture, cap
     assert "This is the root custom err" in str(excinfo.value)
 
 
+@pytest.mark.usefixtures("staging_is_clean")
+def test_commit_when_customized_error_in_handle_questionary_prompt(
+    config, mocker: MockFixture
+):
+    """Test ValueError with CzException context in _handle_questionary_prompt."""
+    _err = ValueError("questionary error")
+    _err.__context__ = CzException("Custom validation failed")
+
+    handle_prompt_mock = mocker.patch(
+        "commitizen.commands.commit._handle_questionary_prompt"
+    )
+    handle_prompt_mock.side_effect = _err
+
+    with pytest.raises(ValueError):
+        commit_cmd = commands.Commit(config, {})
+        commit_cmd()
+
+
 @pytest.mark.usefixtures("staging_is_clean")
 def test_commit_when_non_customized_expected_raised(
     config, mocker: MockFixture, capsys
@@ -449,6 +467,18 @@ def test_manual_edit(editor, config, mocker: MockFixture, tmp_path):
         assert edited_message == test_message.strip()
 
 
+@pytest.mark.usefixtures("staging_is_clean")
+def test_manual_edit_editor_not_found(config, mocker: MockFixture):
+    """Test manual_edit raises error when editor executable is not found."""
+    mocker.patch("commitizen.git.get_core_editor", return_value="nonexistent_editor")
+    mocker.patch("shutil.which", return_value=None)
+
+    commit_cmd = commands.Commit(config, {"edit": True})
+
+    with pytest.raises(RuntimeError, match="Editor 'nonexistent_editor' not found."):
+        commit_cmd.manual_edit("test message")
+
+
 @skip_below_py_3_13
 def test_commit_command_shows_description_when_use_help_option(
     mocker: MockFixture, capsys, file_regression

tests/commands/test_commit_multiline.py

@@ -150,6 +150,45 @@ def failing_then_succeeding_filter(text):
         assert mock_text.call_count == 2  # Prompt shown twice
         mock_error.assert_called_once()  # Error message displayed once
 
+    @patch("commitizen.out.info")
+    @patch("questionary.text")
+    def test_result_none_uses_default(self, mock_text, _mock_info, mock_cz_style):
+        """Test that when result is None, default value is used."""
+        mock_instance = Mock()
+        mock_instance.unsafe_ask.return_value = None
+        mock_text.return_value = mock_instance
+
+        question = {
+            "type": "input",
+            "name": "scope",
+            "message": "Scope",
+            "multiline": True,
+            "default": "default_value",
+        }
+
+        result = _handle_multiline_question(question, mock_cz_style)
+
+        assert result == {"scope": "default_value"}
+
+    @patch("commitizen.out.info")
+    @patch("questionary.text")
+    def test_result_none_no_default(self, mock_text, _mock_info, mock_cz_style):
+        """Test that when result is None and no default, empty string is used."""
+        mock_instance = Mock()
+        mock_instance.unsafe_ask.return_value = None
+        mock_text.return_value = mock_instance
+
+        question = {
+            "type": "input",
+            "name": "scope",
+            "message": "Scope",
+            "multiline": True,
+        }
+
+        result = _handle_multiline_question(question, mock_cz_style)
+
+        assert result == {"scope": ""}
+
 
 class TestKeyBindings:
     """Test key binding setup."""