Skip to content

chore: Add test coverage reports #707

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
Merged
vdusek merged 18 commits into from
Dec 12, 2025
Merged

chore: Add test coverage reports #707

vdusek merged 18 commits into from
Dec 12, 2025

Conversation

@vdusek
Copy link
Contributor

@vdusek vdusek commented Dec 11, 2025 •
edited
Loading

  • Add a code coverage badge to the README.
  • Update the pre_release and run_code_checks test pipeline invocations.
  • Remove the unit test call from the release workflow. Since this workflow is triggered only manually, the caller should ensure the package is OK (and also often the flakiness factor).
  • Temporarily skip four newly failing integration tests to unblock coverage - these will be investigated right after this.
  • Update pytest invocations in the Makefile.
  • Collect coverage from both unit and integration tests and combine them into a single report.
  • PRs for Crawlee and Client will follow.
  • CODECOV_TOKEN is a repository-level secret.
  • Relates to: Add code coverage badge to README crawlee-python#767

@vdusek vdusek requested a review from janbuchar December 11, 2025 20:44
@vdusek vdusek self-assigned this Dec 11, 2025
@vdusek vdusek added the t-tooling Issues with this label are in the ownership of the tooling team. label Dec 11, 2025
@github-actions github-actions bot added this to the 129th sprint - Tooling team milestone Dec 11, 2025
@vdusek vdusek force-pushed the add-code-cov-badge branch from c34746a to ffe29d1 Compare December 11, 2025 20:46
@vdusek vdusek added the adhoc Ad-hoc unplanned task added during the sprint. label Dec 11, 2025
@codecov
Copy link

codecov bot commented Dec 12, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label Dec 12, 2025
@vdusek vdusek changed the title chore: Add Codecov coverage reports chore: Add test coverage reports Dec 12, 2025
janbuchar
janbuchar reviewed Dec 12, 2025
View reviewed changes
@vdusek vdusek merged commit ea0a3c0 into master Dec 12, 2025
26 of 28 checks passed
@vdusek vdusek deleted the add-code-cov-badge branch December 12, 2025 12:47
@vdusek vdusek mentioned this pull request Dec 12, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@janbuchar janbuchar janbuchar approved these changes

Assignees

@vdusek vdusek

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Milestone

129th sprint - Tooling team

Development

Successfully merging this pull request may close these issues.

3 participants

@vdusek @janbuchar