Merge pull request #5 from switchbox-data/fix-tests

Fix tests

Author: MeadBarrel

.devcontainer/devcontainer.json

@@ -14,7 +14,9 @@
         "nefrob.vscode-just-syntax"
       ],
       "settings": {
+        "python.defaultInterpreterPath": "/workspaces/data-science-project/.venv/bin/python",
         "python.testing.pytestArgs": ["tests"],
+        "python.testing.pytestPath": "/workspaces/data-science-project/.venv/bin/pytest",
         "python.testing.unittestEnabled": false,
         "python.testing.pytestEnabled": true
       }

.devcontainer/postCreateCommand.sh

@@ -21,5 +21,9 @@ jobs:
           enable-cache: 'true'
       - name: Install deps
         run: uv sync --group dev
+      - name: Configure git identity
+        run: |
+          git config --global user.name "github-actions[bot]"
+          git config --global user.email "[email protected]"
       - name: Run tests
         run: uv run pytest -q

.github/workflows/ci.yml

@@ -152,7 +152,7 @@ def test_python_mkdocs_only(tmp_path: Path) -> None:
     # Check that postCreateCommand.sh contains the expected Python setup commands
     assert_file_contains(dest, ".devcontainer/postCreateCommand.sh", "curl -LsSf https://astral.sh/uv/install.sh | sh")
     assert_file_contains(dest, ".devcontainer/postCreateCommand.sh", "uv sync --group dev")
-    assert_file_contains(dest, ".devcontainer/postCreateCommand.sh", "uv run pre-commit install --install-hooks")
+    assert_file_contains(dest, ".devcontainer/postCreateCommand.sh", "prek install --install-hooks")
     # Check that Justfile contains documentation commands (python_package boolean bug)
     assert_file_contains(dest, "Justfile", "# 📚 DOCUMENTATION")
     assert_file_contains(dest, "Justfile", "docs:")
@@ -181,26 +181,24 @@ def test_python_data_science_notebooks(tmp_path: Path) -> None:
         },
     )
     assert res.returncode == 0, res.stderr
-    assert_exists(
-        dest, "notebooks", "pyproject.toml", "tox.ini", "pydata_proj", "tests", ".github", ".devcontainer"
-    )
+    assert_exists(dest, "notebooks", "pyproject.toml", "tox.ini", "pydata_proj", "tests", ".github", ".devcontainer")
     assert_missing(dest, "docs", "mkdocs.yml")
     # Check that py_example.qmd contains actual content (not template condition)
-    assert_file_contains(dest, "notebooks/py_example.qmd", "title: \"Python Data Analysis Example\"")
+    assert_file_contains(dest, "notebooks/py_example.qmd", 'title: "Python Data Analysis Example"')
     assert_file_contains(dest, "notebooks/py_example.qmd", "import polars as pl")
     # Ensure template condition is not present in final output
     content = (dest / "notebooks/py_example.qmd").read_text()
-    assert "{% if cookiecutter.pydata == \"y\" %}" not in content, "Template condition should be resolved"
+    assert '{% if cookiecutter.pydata == "y" %}' not in content, "Template condition should be resolved"
 
 
-def test_directory_name_in_devcontainer(tmp_path: Path) -> None:
+def test_project_name_in_devcontainer(tmp_path: Path) -> None:
     dest = tmp_path / "my_custom_dir"
     res = run_copier(
         Path(__file__).parents[1],
         dest,
         {
             "author": "Test",
-            "email": "[email protected]", 
+            "email": "[email protected]",
             "author_github_handle": "test",
             "project_name": "different-name",  # This is different from directory name
             "project_features": "[python_package]",
@@ -210,8 +208,8 @@ def test_directory_name_in_devcontainer(tmp_path: Path) -> None:
         },
     )
     assert res.returncode == 0, res.stderr
-    # Check that devcontainer uses actual directory name, not project_name
-    assert_file_contains(dest, ".devcontainer/devcontainer.json", "/workspaces/my_custom_dir/.venv/bin/python")
-    # Should NOT contain the project_name in paths
+    # Check that devcontainer uses project name, not directory name
+    assert_file_contains(dest, ".devcontainer/devcontainer.json", "/workspaces/different-name")
+    # Should NOT contain the directory name in paths
     content = (dest / ".devcontainer/devcontainer.json").read_text()
-    assert "/workspaces/different-name/" not in content
+    assert "/workspaces/my_custom_dir/" not in content