add test dependencies_test.py

Author: Jannis-Mittenzwei

.gitignore

@@ -1,6 +1,7 @@
 .lint.json
 .lint.txt
 .security.json
+.packages.json
 
 odbcconfig/odbcinst.ini
 

exasol/toolbox/nox/_dependencies.py

@@ -1,9 +1,9 @@
 from __future__ import annotations
 
-import json
 from collections import defaultdict
 from dataclasses import dataclass
 from inspect import cleandoc
+from json import loads
 from pathlib import Path
 
 import nox
@@ -15,8 +15,8 @@
 class Package:
     name: str
     package_link: str
-    license: str
     version: str
+    license: str
     license_link: str
 
 
@@ -26,31 +26,34 @@ def _dependencies(toml_str: str) -> dict[str, list]:
     dependencies: dict[str, list] = {}
 
     packages = poetry.get("dependencies", {})
-    dependencies["project"] = []
+    if packages:
+        dependencies["project"] = []
     for package in packages:
         dependencies["project"].append(package)
 
     packages = poetry.get("dev", {}).get("dependencies", {})
-    dependencies["dev"] = []
+    if packages:
+        dependencies["dev"] = []
     for package in packages:
-        dependencies["dependencies"].append(package)
+        dependencies["dev"].append(package)
 
     groups = poetry.get("group", {})
     for group in groups:
         packages = groups.get(group, {}).get("dependencies")
-        dependencies[group] = []
+        if packages and not dependencies.get(group, {}):
+            dependencies[group] = []
         for package in packages:
             dependencies[group].append(package)
     return dependencies
 
 
-def _licenses(session: Session) -> None:
+def _licenses(session: Session, filename: str) -> None:
     session.run(
         "poetry",
         "run",
         "pip-licenses",
         "--format=json",
-        "--output-file=packages.json",
+        "--output-file=" + filename,
         "--with-system",
         "--with-urls",
     )
@@ -71,7 +74,6 @@ def select_most_permissive(l: str) -> str:
                 "MPLv2": 3,
                 "LGPLv2": 4,
                 "GPLv2": 5,
-                "GPLv3": 6,
             },
         )
         priority_to_license = defaultdict(
@@ -99,8 +101,8 @@ def select_most_permissive(l: str) -> str:
     return mapping[_license]
 
 
-def _packages_from_json(file: Path) -> list[Package]:
-    packages = json.loads(file.read_text())
+def _packages_from_json(json: str) -> list[Package]:
+    packages = loads(json)
     packages_list = []
     for package in packages:
         packages_list.append(
@@ -115,58 +117,62 @@ def _packages_from_json(file: Path) -> list[Package]:
     return packages_list
 
 
-def dependency(group: str, group_packages: list, packages: list[Package]) -> str:
-    def header(group: str):
-        group = "".join([word.capitalize() for word in group.strip().split()])
-        text = f"## {group} Dependencies\n"
-        text += "---\n"
-        text += "|Package|version|Licence|\n"
-        text += "|---|---|---|\n"
-        return text
-
-    def rows(group_packages: list, packages: list[Package]) -> str:
-        def _normalize_package_name(name: str) -> str:
-            _name = name.lower()
-            while "_" in _name:
-                _name = _name.replace("_", "-")
-            return _name
-
-        text = ""
-        for package in group_packages:
-            consistent = filter(
-                lambda elem: (_normalize_package_name(elem.name) == package),
-                packages,
-            )
-            for content in consistent:
-                text += f"|[{content.name}]({content.package_link})"
-                text += f"|{content.version}"
-                text += f"|[{content.license}]({content.license_link})|\n"
-        text += "\n"
-        return text
-
-    _template = cleandoc(
-        """
-        {header}{rows}
-    """
-    )
-    return _template.format(
-        header=header(group), rows=rows(group_packages, packages)
-    )
-
-
-def packages_markdown(dependencies: dict[str, list], packages: list[Package]) -> str:
+def _packages_to_markdown(
+    dependencies: dict[str, list], packages: list[Package]
+) -> str:
     def heading():
         text = "# Dependecies\n"
-        text += "---\n"
+        text += "---\n\n"
         return text
 
+    def dependency(group: str, group_packages: list, packages: list[Package]) -> str:
+        def _header(_group: str):
+            _group = "".join([word.capitalize() for word in _group.strip().split()])
+            text = f"## {_group} Dependencies\n"
+            text += "---\n"
+            text += "|Package|version|Licence|\n"
+            text += "|---|---|---|\n"
+            return text
+
+        def _rows(_group_packages: list, _packages: list[Package]) -> str:
+            def _normalize_package_name(name: str) -> str:
+                _name = name.lower()
+                while "_" in _name:
+                    _name = _name.replace("_", "-")
+                return _name
+
+            text = ""
+            for package in _group_packages:
+                consistent = filter(
+                    lambda elem: (_normalize_package_name(elem.name) == package),
+                    _packages,
+                )
+                for content in consistent:
+                    text += f"|[{content.name}]({content.package_link})"
+                    text += f"|{content.version}"
+                    if content.license_link:
+                        text += f"|[{content.license}]({content.license_link})|\n"
+                    else:
+                        text += f"|{content.license}|\n"
+            text += "\n"
+            return text
+
+        _template = cleandoc(
+            """
+            {header}{rows}
+        """
+        )
+        return _template.format(
+            header=_header(group), rows=_rows(group_packages, packages)
+        )
+
     template = cleandoc(
         """
         {heading}{rows}
     """
     )
-    rows = ""
 
+    rows = ""
     for group in dependencies:
         rows += dependency(group, dependencies[group], packages)
     return template.format(heading=heading(), rows=rows)
@@ -177,6 +183,7 @@ def dependency_licenses(session: Session) -> None:
     """returns the packages and their licenses"""
     toml = Path("pyproject.toml")
     dependencies = _dependencies(toml.read_text())
-    _licenses(session)
-    package_infos = _packages_from_json(Path("packages.json"))
-    print(packages_markdown(dependencies, package_infos))
+    _licenses(session=session, filename=".packages.json")
+    json = Path(".packages.json").read_text()
+    package_infos = _packages_from_json(json)
+    print(_packages_to_markdown(dependencies=dependencies, packages=package_infos))