Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update workflow to generate test coverage statistics #26

Merged
merged 3 commits into from
Sep 26, 2024
Merged

Update workflow to generate test coverage statistics #26

merged 3 commits into from
Sep 26, 2024

Conversation

AustinCullar
Copy link
Owner

@AustinCullar AustinCullar commented Sep 26, 2024 •
edited
Loading

Update workflow to generate test coverage statistics using coverage with pytest.

This work also included:

  • Moving test json api strings to a python file in src/tests so that they can be imported
  • Updating the requirements.txt file to include the coverage tool
  • Renamed the workflow to 'Astro Testing' and the file itself to astro-testing.yml

@AustinCullar
- Modify workflow
d4ffc12 
    - Rename to 'Astro Testing'
    - Run `pytest` under `coverage`
    - Add step to generate coverage report
    - remote `--exit-zero` option from `flake8` to cause lint errors to fail the build

- Moved test comment api response to a python file and imported it into the tests to avoid file relative path errors
@AustinCullar AustinCullar added the enhancement New feature or request label Sep 26, 2024
@AustinCullar AustinCullar self-assigned this Sep 26, 2024
@AustinCullar AustinCullar linked an issue Sep 26, 2024 that may be closed by this pull request
Add test coverage statistics #20
Closed
@AustinCullar
- Exclude test_api_responses.py from flake8 tests
4143f4c 
There's likely a more elegant way to exclude these files, but I'll
leave that for a later issue to stay within scope.
@AustinCullar
- Add coverage to requirements.txt
82478ba
@AustinCullar AustinCullar changed the title - Modify workflow Update workflow to generate test coverage statistics Sep 26, 2024
@AustinCullar AustinCullar merged commit b53768b into main Sep 26, 2024
1 check passed
@AustinCullar AustinCullar deleted the 20-add-test-coverage-statistics branch September 26, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@AustinCullar AustinCullar

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add test coverage statistics
1 participant
@AustinCullar