Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

fix: coverage error #28

Merged
merged 1 commit into from
Aug 17, 2023
Merged

fix: coverage error #28

merged 1 commit into from
Aug 17, 2023

Conversation

UponTheSky
Copy link
Owner

@UponTheSky UponTheSky commented Aug 16, 2023

How did I resolve this problem?

  1. pdm PYTHONPATH: unless you specify the actual directory path, it imports __pypackage__ in an unexpected manner, so you have to make sure you run pytest tests rather than merely calling pytest.

  2. generating junitxml: it is very hard to find the CLI reference: https://docs.pytest.org/en/7.4.x/reference/reference.html#command-line-flags

  3. GA permissions: see this issue - Error: Resource not accessible by integration dorny/test-reporter#168 (comment)

@UponTheSky UponTheSky linked an issue Aug 16, 2023 that may be closed by this pull request
@UponTheSky UponTheSky force-pushed the 27-fix-cicd-test-coverage branch 4 times, most recently from 419297f to 5ae427e Compare August 16, 2023 11:17
@UponTheSky UponTheSky self-assigned this Aug 16, 2023
@UponTheSky UponTheSky added the bug Something isn't working label Aug 16, 2023
@UponTheSky UponTheSky force-pushed the 27-fix-cicd-test-coverage branch 13 times, most recently from 974a60a to 0253afe Compare August 16, 2023 23:53
@UponTheSky UponTheSky merged commit d0d9e00 into main Aug 17, 2023
2 checks passed
@UponTheSky UponTheSky deleted the 27-fix-cicd-test-coverage branch August 17, 2023 00:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix cicd test coverage
1 participant