Skip to content

Commit

Permalink
#546 Fix unit test runs that relied on implicitly finding venv
Browse files Browse the repository at this point in the history
  • Loading branch information
PhilMiller authored and mattw-nws committed Aug 8, 2023
1 parent cd422ad commit 407f147
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/test_and_validate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,7 @@ jobs:

- name: run_bmi_python_tests
run: |
. .venv/bin/activate
cd ./cmake_build/test/
./test_bmi_python
cd ../../
Expand Down Expand Up @@ -353,7 +354,9 @@ jobs:
additional_python_requirements: 'extern/test_bmi_py/requirements.txt'

- name: Run Unit Tests
run: ./cmake_build/test/test_bmi_multi
run: |
. .venv/bin/activate
./cmake_build/test/test_bmi_multi
timeout-minutes: 15

- name: Clean Up Unit Test Build
Expand Down

0 comments on commit 407f147

Please sign in to comment.