fix(BaseConfig): raise error if Path is not set

Author: bearomorphism

commitizen/config/base_config.py

@@ -26,16 +26,13 @@ def settings(self) -> Settings:
         return self._settings
 
     @property
-    def path(self) -> Path | None:
+    def path(self) -> Path:
+        if self._path is None:
+            raise ValueError("Path is not set")
         return self._path
 
     @path.setter
     def path(self, path: str | Path) -> None:
-        """
-        mypy does not like this until 1.16
-        See https://github.com/python/mypy/pull/18510
-        TODO: remove "type: ignore" from the call sites when 1.16 is available
-        """
         self._path = Path(path)
 
     def set_key(self, key: str, value: Any) -> Self:

tests/config/test_base_config.py

@@ -0,0 +1,9 @@
+import pytest
+
+from commitizen.config.base_config import BaseConfig
+
+
+def test_base_config_init():
+    config = BaseConfig()
+    with pytest.raises(ValueError):
+        config.path