Skip to content
Permalink
Browse files

workflow: Do not use ctest, invoke pytest directly instead

There is no advantage to using ctest, which directly invokes pytest anyway, except that it becomes harder to diagnose failures.
  • Loading branch information
rdb committed Mar 20, 2020
1 parent e1659b8 commit b84df2db1b1af654a0c7d91db8b64a684c311d69
Showing with 5 additions and 5 deletions.
  1. +5 −5 .github/workflows/ci.yml
@@ -206,7 +206,7 @@ jobs:
run: |
PYTHON_EXECUTABLE=$(grep 'Python_EXECUTABLE:FILEPATH' CMakeCache.txt | sed 's/.*=//')
$PYTHON_EXECUTABLE -m pip install pytest
ctest -V -C ${{ matrix.config }}
$PYTHON_EXECUTABLE -m pytest -v ../tests
# END B

- name: Setup Python (Python 3.5)
@@ -235,7 +235,7 @@ jobs:
run: |
PYTHON_EXECUTABLE=$(grep 'Python_EXECUTABLE:FILEPATH' CMakeCache.txt | sed 's/.*=//')
$PYTHON_EXECUTABLE -m pip install pytest
ctest -V -C ${{ matrix.config }}
$PYTHON_EXECUTABLE -m pytest -v ../tests
# END B

- name: Setup Python (Python 3.6)
@@ -265,7 +265,7 @@ jobs:
PYTHON_EXECUTABLE=$(grep 'Python_EXECUTABLE:FILEPATH' CMakeCache.txt | sed 's/.*=//')
$PYTHON_EXECUTABLE -m pip install pytest pytest-cov
export COVERAGE_FILE=.coverage.$RANDOM LLVM_PROFILE_FILE=$PWD/pid-%p.profraw
ctest -V -C ${{ matrix.config }}
$PYTHON_EXECUTABLE -m pytest -v ../tests
# END B

- name: Setup Python (Python 3.7)
@@ -295,7 +295,7 @@ jobs:
PYTHON_EXECUTABLE=$(grep 'Python_EXECUTABLE:FILEPATH' CMakeCache.txt | sed 's/.*=//')
$PYTHON_EXECUTABLE -m pip install pytest pytest-cov
export COVERAGE_FILE=.coverage.$RANDOM LLVM_PROFILE_FILE=$PWD/pid-%p.profraw
ctest -V -C ${{ matrix.config }}
$PYTHON_EXECUTABLE -m pytest -v ../tests
# END B

- name: Setup Python (Python 3.8)
@@ -325,7 +325,7 @@ jobs:
PYTHON_EXECUTABLE=$(grep 'Python_EXECUTABLE:FILEPATH' CMakeCache.txt | sed 's/.*=//')
$PYTHON_EXECUTABLE -m pip install pytest pytest-cov
export COVERAGE_FILE=.coverage.$RANDOM LLVM_PROFILE_FILE=$PWD/pid-%p.profraw
ctest -V -C ${{ matrix.config }}
$PYTHON_EXECUTABLE -m pytest -v ../tests
# END B

- name: Upload coverage reports

0 comments on commit b84df2d

Please sign in to comment.
You can’t perform that action at this time.