Skip to content

Commit

Permalink
Run tests with py.test rather than ctest
Browse files Browse the repository at this point in the history
py.test should play nicely with codecov,
and this allows us to turn on other checks
such as flake8 later.
  • Loading branch information
benmwebb committed Dec 18, 2020
1 parent 1e56c36 commit 8cceb4d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
8 changes: 3 additions & 5 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,6 @@ jobs:
PYTHONPATH=`pwd`/coverage cmake .. -DCMAKE_BUILD_TYPE=Debug -DPYTHON_EXECUTABLE=$PYTHON_EXE -DPYTHON_INCLUDE_DIR=$PYTHON_INC -DPYTHON_LIBRARY=$PYTHON_LIB -DCMAKE_C_FLAGS="-fprofile-arcs -ftest-coverage"
rm ../src/mdt_config.h
make
ctest --output-on-failure
- name: Collect coverage and send to codecov
run: |
(cd build/coverage && coverage combine && mv .coverage ..)
bash <(curl -s https://codecov.io/bash)
cd ..
PYTHONPATH=build:$PYTHONPATH py.test --cov=pyext --cov-branch --cov-report=xml -v test
- uses: codecov/codecov-action@v1
2 changes: 1 addition & 1 deletion tools/travis/install_deps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ HDF5_HL_SOVER=100

sudo apt-get -qq update
sudo apt-get install -y swig bc
pip install coverage
pip install coverage pytest-cov pytest-flake8

# Install Modeller
modeller_url=https://salilab.org/modeller/${MODELLER_VERSION}
Expand Down

0 comments on commit 8cceb4d

Please sign in to comment.