Skip to content

Commit

Permalink
refactor(tests): get coverage comments in PRs
Browse files Browse the repository at this point in the history
  • Loading branch information
thekaveman committed Sep 14, 2023
1 parent 1be61dd commit 751537e
Show file tree
Hide file tree
Showing 6 changed files with 57 additions and 42 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/.python-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3.9
3.11
41 changes: 41 additions & 0 deletions .github/workflows/pytest.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
name: Pytest

on:
pull_request:
push:
branches:
- "main"

jobs:
pytest:
runs-on: ubuntu-latest
permissions:
# Gives the action the necessary permissions for publishing new
# comments in pull requests.
pull-requests: write
# Gives the action the necessary permissions for pushing data to the
# python-coverage-comment-action branch, and for editing existing
# comments (to avoid publishing multiple comments in the same PR)
contents: write
steps:
- name: Check out code
uses: actions/checkout@v4

- uses: actions/setup-python@v4
with:
python-version-file: .github/workflows/.python-version
cache: pip
cache-dependency-path: "**/pyproject.toml"

- name: Install Python dependencies
run: pip install -e .[test]

- name: Run tests
run: ./bin/coverage.sh

- name: Coverage comment
uses: py-cov-action/python-coverage-comment-action@v3
with:
GITHUB_TOKEN: ${{ github.token }}
MINIMUM_GREEN: 90
MINIMUM_ORANGE: 80
33 changes: 0 additions & 33 deletions .github/workflows/tests-pytest.yml

This file was deleted.

11 changes: 11 additions & 0 deletions bin/coverage.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
#!/usr/bin/env bash
set -eu

# run normal pytests
coverage run -m pytest

# clean out old coverage results
rm -rf ./tests/coverage

# regenerate coverate report
coverage html --directory ./tests/coverage
4 changes: 4 additions & 0 deletions bin/web.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
#!/usr/bin/env bash
set -eu

python -m http.server -d ./tests/coverage
8 changes: 0 additions & 8 deletions tests/run.sh

This file was deleted.

0 comments on commit 751537e

Please sign in to comment.