Extract combined functionality into get_dependencies & modify for get_dependencies_from_latest_tag

Author: ArBridgeman

exasol/toolbox/nox/_dependencies.py

@@ -12,10 +12,7 @@
     licenses,
     packages_to_markdown,
 )
-from exasol.toolbox.util.dependencies.poetry_dependencies import (
-    PoetryDependencies,
-    PoetryToml,
-)
+from exasol.toolbox.util.dependencies.poetry_dependencies import get_dependencies
 
 
 class Audit:
@@ -86,12 +83,8 @@ def run(self, session: Session) -> None:
 
 @nox.session(name="dependency:licenses", python=False)
 def dependency_licenses(session: Session) -> None:
-    """returns the packages and their licenses"""
-    working_directory = Path()
-    poetry_dep = PoetryToml.load_from_toml(working_directory=working_directory)
-    dependencies = PoetryDependencies(
-        groups=poetry_dep.groups, working_directory=working_directory
-    ).direct_dependencies
+    """Return the packages with their licenses"""
+    dependencies = get_dependencies(working_directory=Path())
     package_infos = licenses()
     print(packages_to_markdown(dependencies=dependencies, packages=package_infos))
 

exasol/toolbox/util/dependencies/poetry_dependencies.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import subprocess
+import tempfile
 from pathlib import Path
 from typing import Optional
 
@@ -12,6 +13,7 @@
 from tomlkit import TOMLDocument
 
 from exasol.toolbox.util.dependencies.shared_models import Package
+from exasol.toolbox.util.git import Git
 
 
 class PoetryGroup(BaseModel):
@@ -21,6 +23,7 @@ class PoetryGroup(BaseModel):
     toml_section: Optional[str]
 
 
+PYPROJECT_TOML = "pyproject.toml"
 TRANSITIVE_GROUP = PoetryGroup(name="transitive", toml_section=None)
 
 
@@ -31,7 +34,7 @@ class PoetryToml(BaseModel):
 
     @classmethod
     def load_from_toml(cls, working_directory: Path) -> PoetryToml:
-        file_path = working_directory / "pyproject.toml"
+        file_path = working_directory / PYPROJECT_TOML
         if not file_path.exists():
             raise ValueError(f"File not found: {file_path}")
 
@@ -142,3 +145,21 @@ def all_dependencies(self) -> dict[str, list[Package]]:
                 transitive_dependencies.append(dep)
 
         return direct_dependencies | {TRANSITIVE_GROUP.name: transitive_dependencies}
+
+
+def get_dependencies(working_directory: Path) -> dict[str, list[Package]]:
+    poetry_dep = PoetryToml.load_from_toml(working_directory=working_directory)
+    return PoetryDependencies(
+        groups=poetry_dep.groups, working_directory=working_directory
+    ).direct_dependencies
+
+
+def get_dependencies_from_latest_tag() -> dict[str, list[Package]]:
+    latest_tag = Git.get_latest_tag()
+    with tempfile.TemporaryDirectory() as path:
+        tmpdir = Path(path)
+
+        Git.copy_remote_file_locally(latest_tag, "poetry.lock", tmpdir)
+        Git.copy_remote_file_locally(latest_tag, PYPROJECT_TOML, tmpdir)
+
+        return get_dependencies(working_directory=tmpdir)

test/unit/util/dependencies/poetry_dependencies_test.py

@@ -6,8 +6,11 @@
     PoetryDependencies,
     PoetryGroup,
     PoetryToml,
+    get_dependencies,
+    get_dependencies_from_latest_tag,
 )
 from exasol.toolbox.util.dependencies.shared_models import Package
+from noxconfig import PROJECT_CONFIG
 
 MAIN_GROUP = PoetryGroup(name="main", toml_section="project.dependencies")
 DEV_GROUP = PoetryGroup(name="dev", toml_section="tool.poetry.group.dev.dependencies")
@@ -128,3 +131,19 @@ def test_all_dependencies(create_poetry_project, project_path):
         transitive = result.pop("transitive")
         assert len(transitive) > 0
         assert result == DIRECT_DEPENDENCIES
+
+
+def test_get_dependencies():
+    result = get_dependencies(PROJECT_CONFIG.root)
+
+    # if successful, no errors & should be non-empty dictionary
+    assert isinstance(result, dict)
+    assert result.keys()
+
+
+def test_get_dependencies_from_latest_tag():
+    result = get_dependencies_from_latest_tag()
+
+    # if successful, no errors & should be non-empty dictionary
+    assert isinstance(result, dict)
+    assert result.keys()