Skip to content

Commit

Permalink
Fix testing + coverage
Browse files Browse the repository at this point in the history
* Drop `py312` to combat #16
* Install `--editable`, so that Python uses the correct source files for coverage
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <133706+stdedos@users.noreply.github.com>
  • Loading branch information
stdedos committed Oct 18, 2023
1 parent 55e4218 commit c208abb
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
2 changes: 2 additions & 0 deletions .codecov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
ignore:
- tests/input/**/*.py
5 changes: 4 additions & 1 deletion .github/workflows/run-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,9 @@ jobs:
steps:
- uses: actions/checkout@v3

- name: Slugify GITHUB_REPOSITORY
run: echo "GITHUB_REPOSITORY_SLUG=${GITHUB_REPOSITORY////_}" >> $GITHUB_ENV

- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v4
with:
Expand Down Expand Up @@ -71,5 +74,5 @@ jobs:
uses: actions/upload-artifact@v3
if: ${{ !cancelled() }}
with:
name: test-artifacts_${{ matrix.os }}_${{ matrix.python-version }}
name: ${{ env.GITHUB_REPOSITORY_SLUG }}_test-artifacts_${{ github.event_name }}_${{ github.event.pull_request.number || github.sha }}_${{ matrix.os }}_py${{ matrix.python-version }}
path: test_artifacts/
4 changes: 2 additions & 2 deletions tox.ini
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[tox]
envlist = py36,py37,py38,py39,py310,py311,py312
envlist = py36,py37,py38,py39,py310,py311
skipsdist = True
passenv =
FORCE_COLOR
Expand All @@ -9,5 +9,5 @@ deps =
pytest
pytest-cov
commands =
pip install --upgrade .
pip install --upgrade --editable .
pytest --cov --cov-append {env:PYTEST_CI_ARGS:} {tty:--color=yes} {posargs:tests}

0 comments on commit c208abb

Please sign in to comment.