refactor(init): improve test coverage

Author: bearomorphism

commitizen/commands/init.py

@@ -294,9 +294,8 @@ def _gen_pre_commit_cmd(self, hook_types: list[str]) -> str:
         """Generate pre-commit command according to given hook types"""
         if not hook_types:
             raise ValueError("At least 1 hook type should be provided.")
-        return "pre-commit install " + " ".join(
-            f"--hook-type {ty}" for ty in hook_types
-        )
+        hook_str = " ".join(f"--hook-type {ty}" for ty in hook_types)
+        return f"pre-commit install {hook_str}"
 
     def _init_config(self, config_path: str):
         if "toml" in config_path:

tests/commands/test_init_command.py

@@ -292,6 +292,27 @@ def test_init_with_confirmed_tag_format(config, mocker: MockFixture, tmpdir):
 
 def test_init_with_no_existing_tags(config, mocker: MockFixture, tmpdir):
     mocker.patch("commitizen.commands.init.get_tag_names", return_value=[])
+    mocker.patch("commitizen.commands.init.get_latest_tag_name", return_value="v1.0.0")
+    mocker.patch(
+        "questionary.select",
+        side_effect=[
+            FakeQuestion("pyproject.toml"),
+            FakeQuestion("cz_conventional_commits"),
+            FakeQuestion("commitizen"),
+            FakeQuestion("semver"),
+        ],
+    )
+    mocker.patch("questionary.confirm", return_value=FakeQuestion(False))
+    mocker.patch("questionary.text", return_value=FakeQuestion("$version"))
+    mocker.patch("questionary.checkbox", return_value=FakeQuestion(None))
+
+    with tmpdir.as_cwd():
+        commands.Init(config)()
+        with open("pyproject.toml", encoding="utf-8") as toml_file:
+            assert 'version = "0.0.1"' in toml_file.read()
+
+
+def test_init_with_no_existing_latest_tag(config, mocker: MockFixture, tmpdir):
     mocker.patch("commitizen.commands.init.get_latest_tag_name", return_value=None)
     mocker.patch(
         "questionary.select",
@@ -358,3 +379,83 @@ def test_init_with_non_commitizen_version_provider(config, mocker: MockFixture,
             assert (
                 'version = "0.0.1"' not in content
             )  # Version should not be set for non-commitizen providers
+
+
+class TestVersionProviderDefault:
+    def test_default_for_python_poetry(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = True
+        mock_project_info.is_python_poetry = True
+        mock_project_info.is_python_uv = False
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "poetry"
+
+    def test_default_for_python_uv(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = True
+        mock_project_info.is_python_poetry = False
+        mock_project_info.is_python_uv = True
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "uv"
+
+    def test_default_for_python_pep621(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = True
+        mock_project_info.is_python_poetry = False
+        mock_project_info.is_python_uv = False
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "pep621"
+
+    def test_default_for_rust_cargo(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = False
+        mock_project_info.is_rust_cargo = True
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "cargo"
+
+    def test_default_for_npm_package(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = False
+        mock_project_info.is_rust_cargo = False
+        mock_project_info.is_npm_package = True
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "npm"
+
+    def test_default_for_php_composer(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = False
+        mock_project_info.is_rust_cargo = False
+        mock_project_info.is_npm_package = False
+        mock_project_info.is_php_composer = True
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "composer"
+
+    def test_default_fallback_to_commitizen(self, config, mocker: MockFixture):
+        mock_project_info = mocker.Mock()
+        mock_project_info.is_python = False
+        mock_project_info.is_rust_cargo = False
+        mock_project_info.is_npm_package = False
+        mock_project_info.is_php_composer = False
+        mocker.patch(
+            "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
+        )
+        init = commands.Init(config)
+        assert init._version_provider_default_val == "commitizen"