Skip to content

Commit

Permalink
try running tests with valgrind in the CI
Browse files Browse the repository at this point in the history
Fixes #563
  • Loading branch information
jacobwilliams committed May 28, 2024
1 parent c19763f commit 8d62554
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-20.04]
gcc_v: [7,8,9,10] # Version of GFortran we want to use.
python-version: [3.9]
gcc_v: [7,8,9,10,11] # Version of GFortran we want to use.
python-version: [3.10]
env:
FC: gfortran-${{matrix.gcc_v}}
GCC_V: ${{matrix.gcc_v}}
Expand Down Expand Up @@ -46,7 +46,7 @@ jobs:
if: contains( matrix.os, 'ubuntu')
run: |
python -m pip install --upgrade pip
pip install ford FoBiS.py pygooglechart
pip install ford FoBiS.py pygooglechart fpm
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: Install GFortran Linux
Expand All @@ -60,14 +60,23 @@ jobs:
--slave /usr/bin/gcov gcov /usr/bin/gcov-${GCC_V}
- name: Compile_with_build
if: matrix.gcc_v != 7
if: matrix.gcc_v != 7 && matrix.gcc_v != 11
run: |
GFORTRAN=gfortran-${{matrix.gcc_v}}
GCOV=gcov-${{matrix.gcc_v}}
# build with build.sh, run unit tests
./build.sh --skip-documentation
./build.sh --skip-documentation --enable-unicode
- name: Compile_with_FPM
if: matrix.gcc_v == 11
run: |
GFORTRAN=gfortran-${{matrix.gcc_v}}
GCOV=gcov-${{matrix.gcc_v}}
apt-get update
sudo apt update && sudo apt install -y valgrind
fpm test --runner "valgrind --leak-check=full --show-leak-kinds=all"
- name: Compile_with_cmake
# CMake build with unit tests, no documentation, with coverage analysis
# No unicode so that coverage combined with the build script will cover unicode
Expand Down

0 comments on commit 8d62554

Please sign in to comment.