Switch loading of content to be explicit so clearer

Author: ArBridgeman

exasol/toolbox/nox/_dependencies.py

@@ -88,9 +88,9 @@ def run(self, session: Session) -> None:
 def dependency_licenses(session: Session) -> None:
     """returns the packages and their licenses"""
     working_directory = Path()
-    groups = PoetryToml(working_directory=working_directory).groups
+    poetry_dep = PoetryToml.load_from_toml(working_directory=working_directory)
     dependencies = PoetryDependencies(
-        groups=groups, working_directory=working_directory
+        groups=poetry_dep.groups, working_directory=working_directory
     ).direct_dependencies
     package_infos = licenses()
     print(packages_to_markdown(dependencies=dependencies, packages=package_infos))

exasol/toolbox/util/dependencies/poetry_dependencies.py

@@ -27,23 +27,27 @@ class Config:
 
 
 class PoetryToml(BaseModel):
-    working_directory: Path
+    content: TOMLDocument
+
+    class Config:
+        frozen = True
+        arbitrary_types_allowed = True
 
-    @model_validator(mode="before")
-    def read_content(cls, values):
-        file_path = values["working_directory"] / "pyproject.toml"
+    @classmethod
+    def load_from_toml(cls, working_directory: Path) -> PoetryToml:
+        file_path = working_directory / "pyproject.toml"
         if not file_path.exists():
             raise ValueError(f"File not found: {file_path}")
 
         try:
             text = file_path.read_text()
-            cls._content: TOMLDocument = tomlkit.loads(text)
+            content = tomlkit.loads(text)
+            return cls(content=content)
         except Exception as e:
             raise ValueError(f"Error reading file: {str(e)}")
-        return values
 
     def get_section_dict(self, section: str) -> Optional[dict]:
-        current = self._content.copy()
+        current = self.content.copy()
         for section in section.split("."):
             if section not in current:
                 return None

test/unit/util/dependencies/poetry_dependencies_test.py

@@ -59,7 +59,7 @@ def create_poetry_project(cwd, project_name, project_path):
 
 @pytest.fixture(scope="module")
 def pyproject_toml(project_path, create_poetry_project):
-    return PoetryToml(working_directory=project_path)
+    return PoetryToml.load_from_toml(working_directory=project_path)
 
 
 class TestPoetryToml: