Skip to content

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

Merged
ntkathole merged 1 commit into
feast-dev:masterfrom
Srihari1192:addQGCodeCov
Jul 16, 2026
Merged

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

Conversation

@Srihari1192

@Srihari1192 Srihari1192 commented Jun 12, 2026 •
edited
Loading

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 •
edited
Loading

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 46.02%. Comparing base (5c9b6fe) to head (17322bf).
⚠️ Report is 1751 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 (17322bf). Click for more details.

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

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #6507       +/-   ##
===========================================
- Coverage   67.65%   46.02%   -21.64%     
===========================================
  Files         181      402      +221     
  Lines       16645    47656    +31011     
  Branches        0     6741     +6741     
===========================================
+ Hits        11262    21936    +10674     
- Misses       5383    24196    +18813     
- Partials        0     1524     +1524
Flag Coverage Δ
go-feature-server 30.58% <ø> (?)
integrationtests ?
python-unit 47.37% <ø> (?)
see 497 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 fd68d65...17322bf. 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.

@Srihari1192
Srihari1192 marked this pull request as ready for review June 15, 2026 05:23
@ntkathole
ntkathole merged commit bf8a5ad into feast-dev:master Jul 16, 2026
27 checks passed
@ntkathole ntkathole mentioned this pull request Jul 19, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ntkathole ntkathole ntkathole approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Srihari1192 @codecov-commenter @ntkathole