ci 4

Author: bjones1

.github/workflows/python-package.yml

@@ -41,13 +41,13 @@ jobs:
               run: |
                   python -m pip install --upgrade pip
                   python -m pip install flake8 pytest
-                  pip install .
+                  # Install as editable so that tests run in cwd, instead of in wherever Python puts system lib. This is important because tests are run on the local (not system lib) files. Therefore, the npm run build produces its files locally, not in system libs; if installed in system libs, then the components won't be able to find these files.
+                  pip install -e .
                   if [ -f requirements-dev.txt ]; then pip install -r requirements-dev.txt; fi
             - name: Test with pytest
               id: pytest
               run: |
-                  pwd
-                  pytest -s -v -k test_dynamic
+                pytest -s -v -k test_dynamic
 
             - uses: act10ns/slack@v1
               with:

runestone/conftest.py

@@ -32,17 +32,15 @@
 # Run this once, before all tests, to update the webpacked JS.
 @pytest.fixture(scope="session", autouse=True)
 def run_webpack():
-    p = subprocess.run(["pwd"], text=True, shell=True, capture_output=True)
-    print(p.stderr + p.stdout)
-    assert not p.returncode
-
-
+    import os
+    print(os.getcwd())
 
     # Note that Windows requires ``shell=True``, since the command to execute is ``npm.cmd``. Use the ``--`` to pass following args to the script (webpack), per the `npm docs <https://docs.npmjs.com/cli/v7/commands/npm-run-script>`_. Use ``--env test`` to tell webpack to do a test build of the Runestone Components (see `RAND_FUNC <RAND_FUNC>`).
     p = subprocess.run(["npm", "run", "build", "--", "--env", "test"], text=True, shell=IS_WINDOWS, capture_output=True)
     print(p.stderr + p.stdout)
     assert not p.returncode
 
+    print(os.getcwd())
     p = subprocess.run(["ls", "-la", "runestone/dist"], text=True, shell=IS_WINDOWS, capture_output=True)
     print(p.stderr + p.stdout)
     assert not p.returncode