Skip to content

Commit

Permalink
#75: Lint before test
Browse files Browse the repository at this point in the history
  • Loading branch information
jwallwork23 committed Apr 17, 2024
1 parent 96846d8 commit a9f0b2f
Showing 1 changed file with 17 additions and 11 deletions.
28 changes: 17 additions & 11 deletions .github/workflows/test_suite.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Build Movement
name: 'Run Movement test suite'

on:
# Run test suite whenever main is updated
Expand All @@ -15,7 +15,7 @@ on:

jobs:
build:
name: "Build Movement"
name: 'Test suite'
# The type of runner that the job will run on
runs-on: ubuntu-latest
# The docker container to use.
Expand All @@ -24,28 +24,34 @@ jobs:
options: --user root
steps:
- uses: actions/checkout@v2
- name: Cleanup

- name: 'Cleanup'
if: ${{ always() }}
run: |
cd ..
rm -rf build
- name: Setup python
- name: 'Setup python'
uses: actions/setup-python@v2
with:
python-version: 3.8
- name: Install

- name: 'Install Movement'
run: |
. /home/firedrake/firedrake/bin/activate
python -m pip install -e .
- name: Test
- name: 'Lint'
if: ${{ always() }}
run: |
. /home/firedrake/firedrake/bin/activate
make lint
- name: 'Test Animate'
run: |
. /home/firedrake/firedrake/bin/activate
export GITHUB_ACTIONS_TEST_RUN=1
python $(which firedrake-clean)
python -m coverage erase
python -m coverage run --source=movement -m pytest -v --durations=20 test
python -m coverage report
- name: Lint
if: ${{ always() }}
run: |
. /home/firedrake/firedrake/bin/activate
make lint

0 comments on commit a9f0b2f

Please sign in to comment.