Skip to content

Commit

Permalink
Running integration tests as a seperate step
Browse files Browse the repository at this point in the history
Signed-off-by: Pradyot Ranjan <99216956+pradyotRanjan@users.noreply.github.com>
  • Loading branch information
prady0t committed May 26, 2024
1 parent 64f5524 commit eb586f0
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions .github/workflows/test_on_push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,6 @@ jobs:
matrix:
os: [ubuntu-latest, macos-12, macos-14, windows-latest]
python-version: ["3.9", "3.10", "3.11", "3.12"]
# We check coverage on Ubuntu with Python 3.12, so we skip unit tests for it here
exclude:
- os: ubuntu-latest
python-version: "3.12"
name: Tests (${{ matrix.os }} / Python ${{ matrix.python-version }})

steps:
Expand Down Expand Up @@ -109,15 +105,17 @@ jobs:
if: matrix.os != 'windows-latest'
run: python -m nox -s pybamm-requires

- name: Run unit/coverage and integration tests
- name: Run unit/coverage tests for ${{ matrix.os }} with Python ${{ matrix.python-version }}
shell: bash
run: |
if [ ${{ matrix.python-version }} == "3.12" && ${{ matrix.os }} == "ubuntu-latest" ]; then
python -m nox -s coverage
else
python -m nox -s unit
fi
python -m nox -s integration
- name: Run integration tests for ${{ matrix.os }} with Python ${{ matrix.python-version }}
run: python -m nox -s integration

- name: Upload coverage report
if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.12'
Expand Down

0 comments on commit eb586f0

Please sign in to comment.