Reformat documentation string so clear intent, move to outward file, extend tests

Author: ArBridgeman

exasol/toolbox/BaseConfig/__init__.py

@@ -0,0 +1,65 @@
+from typing import (
+    Annotated,
+)
+
+from pydantic import (
+    AfterValidator,
+    BaseModel,
+    ConfigDict,
+    Field,
+    computed_field,
+)
+
+from exasol.toolbox.util.version import Version
+
+
+def valid_version_string(version_string: str) -> str:
+    Version.from_string(version_string)
+    return version_string
+
+
+ValidVersionStr = Annotated[str, AfterValidator(valid_version_string)]
+
+
+class BaseConfig(BaseModel):
+    """
+    Basic configuration for projects using the PTB
+
+    This configuration class defines the necessary attributes for using the PTB's
+    various nox sessions and GitHub CI workflows. Defaults are provided via the
+    attributes, which makes it easy for a specific project to modify these values
+    as needed. There are properties provided which automate some configuration aspects
+    so that the project-specific modifications are reduced.
+
+    pydantic has been used so that altered attributes are quickly validated.
+    This allows for immediate feedback, instead of waiting for various failed CI
+    runs.
+    """
+
+    python_versions: tuple[ValidVersionStr, ...] = Field(
+        default=(
+            "3.9",
+            "3.10",
+            "3.11",
+            "3.12",
+            "3.13",
+        ),
+        description="Python versions to use in running CI workflows",
+    )
+
+    exasol_versions: tuple[ValidVersionStr, ...] = Field(
+        default=("7.1.9",),
+        description="Exasol versions to use in running CI workflows for integration tests using the DB",
+    )
+    model_config = ConfigDict(frozen=True, arbitrary_types_allowed=True)
+
+    @computed_field  # type: ignore[misc]
+    @property
+    def min_py_version(self) -> str:
+        """
+        Minimum Python version declared from the `python_versions` list
+
+        This is used in specific testing scenarios where it would be either
+        costly to run the tests for all `python_versions` or we need a single metric.
+        """
+        return str(min([Version.from_string(v) for v in self.python_versions]))

exasol/toolbox/config.py

@@ -3,10 +3,9 @@
 from __future__ import annotations
 
 from collections.abc import Iterable
-from dataclasses import dataclass
 from pathlib import Path
 
-from exasol.toolbox.BaseConfig import BaseConfig
+from exasol.toolbox.config import BaseConfig
 from exasol.toolbox.nox.plugin import hookimpl
 from exasol.toolbox.tools.replace_version import update_github_yml
 

noxconfig.py

@@ -1,23 +1,24 @@
 from __future__ import annotations
 
-from dataclasses import dataclass
 from pathlib import Path
 from typing import Iterable
-from exasol.toolbox.BaseConfig import BaseConfig
+
+from exasol.toolbox.config import BaseConfig
 
 
 class Config(BaseConfig):
     root: Path = Path(__file__).parent
     doc: Path = Path(__file__).parent / "doc"
     source: Path = Path("exasol/{{cookiecutter.package_name}}")
     version_file: Path = (
-        Path(__file__).parent
-        / "exasol"
-        / "{{cookiecutter.package_name}}"
-        / "version.py"
+            Path(__file__).parent
+            / "exasol"
+            / "{{cookiecutter.package_name}}"
+            / "version.py"
     )
     path_filters: Iterable[str] = ()
-    pyupgrade_args: Iterable[str] = ("--py{{cookiecutter.python_version_min | replace('.', '')}}-plus",)
+    pyupgrade_args: Iterable[str] = (
+    "--py{{cookiecutter.python_version_min | replace('.', '')}}-plus",)
     plugins: Iterable[object] = ()
 
 

project-template/{{cookiecutter.repo_name}}/noxconfig.py

@@ -0,0 +1,61 @@
+import pytest
+from pydantic_core._pydantic_core import ValidationError
+
+from exasol.toolbox.config import (
+    BaseConfig,
+    valid_version_string,
+)
+
+
+class TestBaseConfig:
+    @staticmethod
+    def test_works_as_defined():
+        BaseConfig()
+
+    @staticmethod
+    @pytest.mark.parametrize(
+        "wrong_input,expected_message",
+        [
+            pytest.param(
+                {"python_versions": ["1.2.3.1"]},
+                "Version has an invalid format",
+                id="python_versions",
+            ),
+            pytest.param(
+                {"exasol_versions": ["1.2.3.1"]},
+                "Version has an invalid format",
+                id="exasol_versions",
+            ),
+        ],
+    )
+    def test_raises_exception_when_incorrect_modification(
+        wrong_input: dict, expected_message: str
+    ):
+        with pytest.raises(ValidationError, match=expected_message):
+            BaseConfig(**wrong_input)
+
+
+class TestValidVersionString:
+    @staticmethod
+    def test_work_as_expected():
+        version_string = "1.2.3"
+        result = valid_version_string(version_string=version_string)
+        assert result == version_string
+
+    @staticmethod
+    def test_raises_exception_when_not_valid():
+        with pytest.raises(ValueError):
+            valid_version_string("$.2.3")
+
+
+class BaseConfigExpansion(BaseConfig):
+    expansion1: str = "test1"
+
+
+def test_expansion_validation():
+    BaseConfigExpansion()
+
+
+def test_min_py_version():
+    conf = BaseConfig(python_versions=("5.5.5", "1.1.1", "9.9.9"))
+    assert conf.min_py_version == "1.1.1"