refactor(ProjectInfo): extract property default_version_provider

Author: bearomorphism

commitizen/commands/init.py

@@ -72,6 +72,22 @@ def is_php_composer(self) -> bool:
     def is_pre_commit_installed(self) -> bool:
         return bool(shutil.which("pre-commit"))
 
+    @property
+    def default_version_provider(self) -> str:
+        if self.is_python:
+            if self.is_python_poetry:
+                return "poetry"
+            if self.is_python_uv:
+                return "uv"
+            return "pep621"
+        if self.is_rust_cargo:
+            return "cargo"
+        if self.is_npm_package:
+            return "npm"
+        if self.is_php_composer:
+            return "composer"
+        return "commitizen"
+
 
 class Init:
     _PRE_COMMIT_CONFIG_FILENAME = ".pre-commit-config.yaml"
@@ -226,26 +242,10 @@ def _ask_version_provider(self) -> str:
                 "Choose the source of the version:",
                 choices=choices,
                 style=self.cz.style,
-                default=self._version_provider_default_val,
+                default=self.project_info.default_version_provider,
             ).unsafe_ask()
         )
 
-    @property
-    def _version_provider_default_val(self) -> str:
-        if self.project_info.is_python:
-            if self.project_info.is_python_poetry:
-                return "poetry"
-            if self.project_info.is_python_uv:
-                return "uv"
-            return "pep621"
-        if self.project_info.is_rust_cargo:
-            return "cargo"
-        if self.project_info.is_npm_package:
-            return "npm"
-        if self.project_info.is_php_composer:
-            return "composer"
-        return "commitizen"
-
     def _ask_version_scheme(self) -> str:
         """Ask for setting: version_scheme"""
 

tests/commands/test_init_command.py

@@ -421,7 +421,7 @@ def test_default_for_python_poetry(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "poetry"
+        assert init.project_info.default_version_provider == "poetry"
 
     def test_default_for_python_uv(self, config, mocker: MockFixture):
         mock_project_info = mocker.Mock()
@@ -432,7 +432,7 @@ def test_default_for_python_uv(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "uv"
+        assert init.project_info.default_version_provider == "uv"
 
     def test_default_for_python_pep621(self, config, mocker: MockFixture):
         mock_project_info = mocker.Mock()
@@ -443,7 +443,7 @@ def test_default_for_python_pep621(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "pep621"
+        assert init.project_info.default_version_provider == "pep621"
 
     def test_default_for_rust_cargo(self, config, mocker: MockFixture):
         mock_project_info = mocker.Mock()
@@ -453,7 +453,7 @@ def test_default_for_rust_cargo(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "cargo"
+        assert init.project_info.default_version_provider == "cargo"
 
     def test_default_for_npm_package(self, config, mocker: MockFixture):
         mock_project_info = mocker.Mock()
@@ -464,7 +464,7 @@ def test_default_for_npm_package(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "npm"
+        assert init.project_info.default_version_provider == "npm"
 
     def test_default_for_php_composer(self, config, mocker: MockFixture):
         mock_project_info = mocker.Mock()
@@ -476,7 +476,7 @@ def test_default_for_php_composer(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "composer"
+        assert init.project_info.default_version_provider == "composer"
 
     def test_default_fallback_to_commitizen(self, config, mocker: MockFixture):
         mock_project_info = mocker.Mock()
@@ -488,4 +488,4 @@ def test_default_fallback_to_commitizen(self, config, mocker: MockFixture):
             "commitizen.commands.init.ProjectInfo", return_value=mock_project_info
         )
         init = commands.Init(config)
-        assert init._version_provider_default_val == "commitizen"
+        assert init.project_info.default_version_provider == "commitizen"