Skip to content

Conversation

@ponyisi
Copy link
Collaborator

@ponyisi ponyisi commented Oct 27, 2025

Enable uploading of coverage data to codecov.io

@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.42%. Comparing base (eb568b0) to head (739b90b).
⚠️ Report is 554 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1191      +/-   ##
===========================================
- Coverage    92.72%   85.42%   -7.30%     
===========================================
  Files           80       93      +13     
  Lines         2309     3163     +854     
  Branches       154      355     +201     
===========================================
+ Hits          2141     2702     +561     
- Misses         143      387     +244     
- Partials        25       74      +49     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enables code coverage reporting to codecov.io by adding .coveragerc configuration files across multiple services and updating the CI workflow to generate and upload coverage reports.

  • Adds standardized .coveragerc files to all service directories to configure coverage collection
  • Updates CI workflow to generate XML coverage reports and upload them to codecov
  • Enables test requirement for did_finder_rucio service

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
transformer_sidecar/.coveragerc Adds coverage configuration for transformer_sidecar with source tracking and exclusion rules
servicex_app/.coveragerc Adds coverage configuration for servicex_app with source tracking and exclusion rules
did_finder_xrootd/.coveragerc Adds coverage configuration for did_finder_xrootd with source tracking and exclusion rules
did_finder_rucio/.coveragerc Updates existing coverage configuration to add report settings and exclusion rules
did_finder_cernopendata/.coveragerc Adds coverage configuration for did_finder_cernopendata with source tracking and exclusion rules
code_generator_raw_uproot/.coveragerc Adds coverage configuration for code_generator_raw_uproot with source tracking and exclusion rules
code_generator_python/.coveragerc Adds coverage configuration for code_generator_python with source tracking and exclusion rules
code_generator_funcadl_xAOD/.coveragerc Updates source path from servicex to xaod_code_generator/
code_generator_funcadl_uproot/.coveragerc Adds coverage configuration for code_generator_funcadl_uproot with source tracking and exclusion rules
code_generator_TopCPToolkit/.coveragerc Adds coverage configuration for code_generator_TopCPToolkit with source tracking and exclusion rules
.github/workflows/deploy-config.json Enables test requirement for did_finder_rucio
.github/workflows/ci_servicex.yaml Adds coverage XML generation and codecov upload with explicit file specification
Comments suppressed due to low confidence (4)

did_finder_xrootd/.coveragerc:1

  • Corrected syntax: 'main' should not have periods around it. The correct pattern is if __name__ == \"__main__\":
    did_finder_rucio/.coveragerc:1
  • Corrected syntax: 'main' should not have periods around it. The correct pattern is if __name__ == \"__main__\":
    did_finder_cernopendata/.coveragerc:1
  • Corrected syntax: 'main' should not have periods around it. The correct pattern is if __name__ == \"__main__\":
    code_generator_TopCPToolkit/.coveragerc:1
  • Corrected syntax: 'main' should not have periods around it. The correct pattern is if __name__ == \"__main__\":

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ponyisi
Copy link
Collaborator Author

ponyisi commented Oct 27, 2025

For some reason the x509_secrets container has no test, but otherwise it looks like we have everything in with this.

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.

1 participant