Skip to content

Commit 206acdb

Browse files
Add coverage report to GHA for PRs (#14)
* chore: add coverage report * chore: add thresholds * chore: break test * chore: switch providers * chore: fix integration * chore: remove build ephemera * chore: typo * fix: fail on test fail * fix: see outputs * fix: see outputs * fix: see outputs * fix: fail on test fail * feat: externalize coverage report * fix: typo * fix: pin reviewdog version * fix: intentionall broken test * chore: poetry update * Update tests/test_users.py Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
1 parent 6fe7c48 commit 206acdb

File tree

4 files changed

+182
-145
lines changed

4 files changed

+182
-145
lines changed

.github/workflows/ci.yml

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
poetry env use "3.12"
2828
poetry install
2929
- name: Setup reviewdog
30-
uses: reviewdog/action-setup@v1
30+
uses: reviewdog/action-setup@e04ffabe3898a0af8d0fb1af00c188831c4b5893
3131
- name: Run black
3232
env:
3333
REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -43,4 +43,20 @@ jobs:
4343
poetry run ruff check nodestream_github tests --output-format rdjson | reviewdog -f=rdjson -name="ruff-check" -level=warning -reporter=github-pr-review
4444
- name: Run Tests
4545
run: |
46-
poetry run pytest
46+
poetry run pytest --cov-report=html --junitxml=pytest.xml --cov-report=term-missing:skip-covered --cov=nodestream_github tests/ | tee pytest-coverage.txt
47+
- uses: actions/upload-artifact@v4
48+
with:
49+
name: coverage-info
50+
path: htmlcov/
51+
- name: Pytest coverage comment
52+
id: coverageComment
53+
uses: MishaKav/pytest-coverage-comment@v1
54+
with:
55+
pytest-coverage-path: ./pytest-coverage.txt
56+
junitxml-path: ./pytest.xml
57+
- name: fail on test errors
58+
if: ${{ steps.coverageComment.outputs.errors > 0 || steps.coverageComment.outputs.failures > 0 }}
59+
uses: actions/github-script@v3
60+
with:
61+
script: |
62+
core.setFailed('Pytest Failures or Errors')

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -161,3 +161,5 @@ cython_debug/
161161
**/coverage.json
162162
**/coverage.xml
163163
**/htmlcov/*
164+
pytest-coverage.txt
165+
pytest.xml

0 commit comments

Comments
 (0)