Skip to content

ci: Add pytest-cov and Codecov integration for coverage tracking#6507

Draft
Srihari1192 wants to merge 1 commit into
feast-dev:masterfrom
Srihari1192:addQGCodeCov
Draft

ci: Add pytest-cov and Codecov integration for coverage tracking#6507
Srihari1192 wants to merge 1 commit into
feast-dev:masterfrom
Srihari1192:addQGCodeCov

Conversation

@Srihari1192

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Adds end-to-end test coverage tracking for both Python and Go unit tests, reported to Codecov on every PR.

  • .codecov.yml — Codecov config with Python and Go flags, informational-only status checks, and ignores for generated/non-source files
  • .github/workflows/unit_tests.yml — uploads Python coverage after unit tests; adds a new Go job that runs tests with coverage
  • Makefile — adds coverage flags to test-python-unit ; scopes test-go to ./go/...
  • pyproject.toml — adds pytest-cov dependency and coverage configuration scoped to the feast package

Which issue(s) this PR fixes:

Checks

  • I've made sure the tests are passing.
  • My commits are signed off (git commit -s)
  • My PR title follows conventional commits format

Testing Strategy

  • Unit tests
  • Integration tests
  • Manual tests
  • Testing is not required for this change

Misc

@Srihari1192 Srihari1192 force-pushed the addQGCodeCov branch 2 times, most recently from 3d5a651 to f5016c3 Compare June 12, 2026 10:43
@Srihari1192 Srihari1192 changed the title ci: add pytest-cov and Codecov integration for Python and Go coverage tracking ci: add pytest-cov and codecov integration for coverage tracking Jun 12, 2026
@Srihari1192 Srihari1192 changed the title ci: add pytest-cov and codecov integration for coverage tracking ci: Add pytest-cov and Codecov integration for coverage tracking Jun 12, 2026
@codecov-commenter

codecov-commenter commented Jun 12, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.75%. Comparing base (5c9b6fe) to head (ca1de21).
⚠️ Report is 1706 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

❗ There is a different number of reports uploaded between BASE (5c9b6fe) and HEAD (ca1de21). Click for more details.

HEAD has 13 uploads less than BASE
Flag BASE (5c9b6fe) HEAD (ca1de21)
integrationtests 13 0
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #6507       +/-   ##
===========================================
- Coverage   67.65%   45.75%   -21.91%     
===========================================
  Files         181      391      +210     
  Lines       16645    45394    +28749     
  Branches        0     6361     +6361     
===========================================
+ Hits        11262    20771     +9509     
- Misses       5383    23146    +17763     
- Partials        0     1477     +1477     
Flag Coverage Δ
go-feature-server 30.58% <ø> (?)
integrationtests ?
python-unit 47.14% <ø> (?)
see 486 files with indirect coverage changes

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a0f23c...ca1de21. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Srihari <svenkata@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants