Move PTB Config to using BaseConfig for excluded_paths

Author: ArBridgeman

exasol/toolbox/config.py

@@ -88,7 +88,14 @@ def excluded_paths(self) -> tuple[str, ...]:
         source_path, like excluding `dist`, `.eggs`. As such, this property is used to
         exclude such undesired paths.
         """
-        default_excluded_paths = {"dist", ".eggs", "venv", ".poetry"}
+        default_excluded_paths = {
+            "dist",
+            ".eggs",
+            "venv",
+            ".poetry",
+            ".sonar",
+            ".html-documentation",
+        }
         return tuple(default_excluded_paths.union(set(self.addition_to_excluded_paths)))
 
     @computed_field  # type: ignore[misc]

exasol/toolbox/nox/_shared.py

@@ -44,10 +44,13 @@ def python_files(project_root: Path) -> Iterable[str]:
     """
     Returns iterable of python files after removing unwanted paths
     """
-    deny_list = DEFAULT_PATH_FILTERS.union(set(PROJECT_CONFIG.path_filters))
-
+    check_for_config_attribute(config=PROJECT_CONFIG, attribute="excluded_paths")
     files = project_root.glob("**/*.py")
-    return [f"{path}" for path in files if not set(path.parts).intersection(deny_list)]
+    return [
+        f"{path}"
+        for path in files
+        if not set(path.parts).intersection(PROJECT_CONFIG.excluded_paths)
+    ]
 
 
 def _version(session: Session, mode: Mode) -> None:

noxconfig.py

@@ -56,16 +56,16 @@ class Config(BaseConfig):
     source: Path = Path("exasol/toolbox")
     importlinter: Path = Path(__file__).parent / ".import_linter_config"
     version_file: Path = Path(__file__).parent / "exasol" / "toolbox" / "version.py"
-    path_filters: Iterable[str] = (
-        "metrics-schema",
-        "project-template",
-        "idioms",
-        ".github",
-    )
     plugins: Iterable[object] = (UpdateTemplates,)
 
 
 PROJECT_CONFIG = Config(
+    addition_to_excluded_paths=(
+        # The cookiecutter placeholders do not work well with checks.
+        # Instead, the format & linting are checked in the
+        # ``test.integration.project-template``.
+        "project-template",
+    ),
     create_major_version_tags=True,
     # The PTB does not have integration tests run with an Exasol DB,
     # so for running in the CI, we take the first element.