feat: add support for scoop apps environment variables

Author: cuinixam

src/py_app_dev/core/scoop_wrapper.py

@@ -100,7 +100,7 @@ def to_file(self, scoop_file: Path) -> None:
 
 
 @dataclass
-class InstalledApp:
+class InstalledScoopApp:
     #: App name
     name: str
     #: App version
@@ -111,6 +111,10 @@ class InstalledApp:
     bin_dirs: List[Path]
     #: List of directories relative to the app path
     env_add_path: List[Path]
+    #: App scoop manifest file
+    manifest_file: Path
+    #: Environment variables defined in the manifest
+    env_vars: Dict[str, Any] = field(default_factory=dict)
 
     def get_bin_paths(self) -> List[Path]:
         """Return the list of absolute bin paths."""
@@ -126,11 +130,9 @@ def get_all_required_paths(self) -> List[Path]:
         unique_paths = list(dict.fromkeys(all_paths))
         return unique_paths
 
-
-@dataclass
-class InstalledScoopApp(InstalledApp):
-    #: App scoop manifest file
-    manifest_file: Path
+    def parse_env_vars(self, env_set: Dict[str, str]) -> None:
+        """Parse and populate environment variables from the manifest."""
+        self.env_vars = {key: value.replace("$dir", str(self.path)) for key, value in env_set.items()}
 
 
 class ScoopWrapper:
@@ -202,14 +204,16 @@ def parse_manifest_file(self, manifest_file: Path) -> InstalledScoopApp:
             tool_version: str = manifest_data.get("version", None)
             bin_dirs: List[Path] = self.parse_bin_dirs(manifest_data.get("bin", []))
             env_add_path: List[Path] = self.parse_env_path_dirs(manifest_data.get("env_add_path", []))
-            return InstalledScoopApp(
+            installed_app = InstalledScoopApp(
                 name=tool_name,
                 version=tool_version,
                 path=app_directory,
                 manifest_file=manifest_file,
                 bin_dirs=bin_dirs,
                 env_add_path=env_add_path,
             )
+            installed_app.parse_env_vars(manifest_data.get("env_set", {}))
+            return installed_app
 
     def get_installed_apps(self) -> List[InstalledScoopApp]:
         installed_tools: List[InstalledScoopApp] = []

tests/test_scoop_wrapper.py

@@ -2,7 +2,7 @@
 import sys
 import textwrap
 from pathlib import Path
-from typing import List, Optional, Tuple, Type
+from typing import Any, Dict, List, Optional, Tuple, Type
 from unittest.mock import Mock, patch
 
 import pytest
@@ -468,3 +468,35 @@ def test_get_tools_to_be_installed(
 def test_semver_compare(v1: str, v2: str, expected: int) -> None:
     result = _semver_compare(v1, v2)
     assert result == expected, f"Comparison of '{v1}' and '{v2}' failed: Expected {expected}, Got {result}"
+
+
+@pytest.mark.parametrize(
+    "env_set, expected_env_vars",
+    [
+        ({"MY_COMP_ROOT": "$dir"}, {"MY_COMP_ROOT": "{path}"}),
+        ({"MY_LIB": "my_lib", "MY_COMP_BIN": "$dir/bin"}, {"MY_LIB": "my_lib", "MY_COMP_BIN": "{path}/bin"}),
+    ],
+)
+def test_env_vars_parsing(scoop_dir: Path, env_set: Dict[str, Any], expected_env_vars: Dict[str, Any]) -> None:
+    scoop_wrapper = create_scoop_wrapper(scoop_dir / "scoop.ps1")
+
+    # Create a mock manifest file with env_set
+    manifest_file = scoop_dir / "apps" / "app_with_env" / "1.0" / "manifest.json"
+    manifest_file.parent.mkdir(parents=True)
+    manifest_file.write_text(
+        json.dumps(
+            {
+                "version": "1.0",
+                "env_set": env_set,
+            }
+        )
+    )
+
+    # Parse the manifest file
+    installed_app = scoop_wrapper.parse_manifest_file(manifest_file)
+
+    # Format expected_env_vars with the actual path
+    formatted_expected_env_vars = {key: value.format(path=str(installed_app.path)) for key, value in expected_env_vars.items()}
+
+    # Assert the environment variables are correctly parsed
+    assert installed_app.env_vars == formatted_expected_env_vars