Fix dependency issue blocking pdm test

khanlab · Sep 22, 2023 · 8f377fe · 8f377fe
1 parent 3e424a8
commit 8f377fe
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/snakebids/project_template/{% if build_system != 'poetry' %}pyproject.toml{% endif %}.jinja b/snakebids/project_template/{% if build_system != 'poetry' %}pyproject.toml{% endif %}.jinja
@@ -30,6 +30,10 @@ requires-python = ">=3.8,<3.12"
 dependencies = [
     "snakemake >= {{ snakemake_version }}",
     "snakebids >= {{ snakebids_version }}",
+    {# Explicitely specify numpy version for py38 until snakebids 0.10.x to
+       work around specification bug causing numpys too high for py38 to be
+       locked by pdm -#}
+    "numpy <=1.24.4; python_version < \"3.9\"",
 ]
 
 [project.scripts]

diff --git a/snakebids/tests/test_template.py b/snakebids/tests/test_template.py
@@ -240,6 +240,12 @@ def test_template_dry_runs_successfully(tmp_path: Path, build: BuildSystems, ven
             app_name,
         ],
         capture_output=True,
-        check=True,
+        check=False,
     )
+    try:
+        cmd.check_returncode()
+    except Exception as err:
+        print(cmd.stdout)
+        print(cmd.stderr, file=sys.stderr)
+        raise err
     assert "All set" in cmd.stdout.decode()