Skip to content

Commit

Permalink
Fix CI issues related to files that cannot be loaded (#1555)
Browse files Browse the repository at this point in the history
* Ensure python path has base directory when running tests

* Update workflows

* Fix notebook test
  • Loading branch information
saulfield authored Aug 13, 2022
1 parent 197fabc commit ab8ca3c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-base-tests-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
run: |
. ${{env.venv_dir}}/bin/activate
make build-all-scenarios
PYTHONHASHSEED=42 pytest -v \
PYTHONPATH=$PWD PYTHONHASHSEED=42 pytest -v \
--doctest-modules \
--forked \
--dist=no \
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-base-tests-mac.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
export SUMO_HOME="/usr/local/opt/sumo/share/sumo"
open -g -a XQuartz.app
make build-all-scenarios
PYTHONHASHSEED=42 pytest -v \
PYTHONPATH=$PWD PYTHONHASHSEED=42 pytest -v \
--doctest-modules \
-n auto \
--ignore-glob="**/waymo_map.py" \
Expand Down
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
test: build-all-scenarios
# sstudio uses hash(...) as part of some of its type IDs. To make the tests
# repeatable we fix the seed.
PYTHONHASHSEED=42 pytest -v \
PYTHONPATH=$(PWD) PYTHONHASHSEED=42 pytest -v \
--cov=smarts \
--doctest-modules \
--forked \
Expand All @@ -21,7 +21,7 @@ test: build-all-scenarios
.PHONY: sanity-test
sanity-test: build-sanity-scenarios
./tests/test_setup.py
PYTHONHASHSEED=42 pytest -v \
PYTHONPATH=$(PWD) PYTHONHASHSEED=42 pytest -v \
--doctest-modules \
--forked \
--dist=loadscope \
Expand Down

0 comments on commit ab8ca3c

Please sign in to comment.