Skip to content

Commit

Permalink
Merge pull request #90 from rjleveque/pytest
Browse files Browse the repository at this point in the history
WIP: Using pytest instead of nose requires renaming to regression_test.py
  • Loading branch information
rjleveque authored May 13, 2024
2 parents 9d69dcb + 1c5adda commit 5f178e4
Show file tree
Hide file tree
Showing 4 changed files with 54 additions and 0 deletions.
54 changes: 54 additions & 0 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
name: Test Classic

on:
push:
branches: [ "master" ]
pull_request:
branches: [ "master" ]

permissions:
contents: read

env:
CLAW: ${{ github.workspace }}

jobs:
build:
runs-on: ubuntu-latest
steps:
- name: Set up Python 3.10
uses: actions/setup-python@v5
with:
python-version: "3.10"
- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install gfortran
python -m pip install --upgrade pip
pip install flake8 meson-python ninja pytest numpy
- name: Checkout clawpack
uses: actions/[email protected]
with:
repository: clawpack/clawpack
submodules: true
- name: Checkout classic branch
uses: actions/[email protected]
with:
repository: clawpack/classic
path: classic
- name: Install clawpack
run: |
pip install --user -e $CLAW
- name: Lint with flake8
run: |
cd ${CLAW}/geoclaw
# stop the build if there are Python syntax errors or undefined names
flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
- name: Test with pytest
run: |
cd ${CLAW}/classic
pytest

0 comments on commit 5f178e4

Please sign in to comment.