Skip to content

Commit

Permalink
Merge pull request #290 from per1234/coverage-task
Browse files Browse the repository at this point in the history
Generate standard code coverage report data file in `python:test` task
  • Loading branch information
per1234 authored Jun 13, 2024
2 parents 494d528 + dc46eb5 commit 9cbdcb8
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 12 deletions.
13 changes: 1 addition & 12 deletions .github/workflows/test-python-poetry-task.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,9 +65,6 @@ jobs:
if: needs.run-determination.outputs.result == 'true'
runs-on: ubuntu-latest

env:
COVERAGE_DATA_FILENAME: coverage.xml

steps:
- name: Checkout repository
uses: actions/checkout@v4
Expand Down Expand Up @@ -98,14 +95,6 @@ jobs:
- name: Display code coverage report
run: task python:coverage-report

# codecov/codecov-action only makes the conversion if the `coverage` package is installed in the global runner
# environment
- name: Convert code coverage report to format required by Codecov
run: |
poetry run \
coverage xml \
-o "${{ github.workspace }}/${{ env.COVERAGE_DATA_FILENAME }}"
# A token is used to avoid intermittent spurious job failures caused by rate limiting.
- name: Set up Codecov upload token
run: |
Expand All @@ -124,5 +113,5 @@ jobs:
uses: codecov/codecov-action@v3
with:
fail_ci_if_error: true
file: ${{ env.COVERAGE_DATA_FILENAME }}
file: coverage.xml
token: ${{ env.CODECOV_TOKEN }}
3 changes: 3 additions & 0 deletions Taskfile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -439,6 +439,9 @@ tasks:
--source="{{.PYTHON_PROJECT_PATH}}" \
--module \
pytest "{{.PYTHON_PROJECT_PATH}}/tests"
- |
poetry run \
coverage xml
# Make a temporary file named according to the passed TEMPLATE variable and print the path passed to stdout
# Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/windows-task/Taskfile.yml
Expand Down

0 comments on commit 9cbdcb8

Please sign in to comment.