ci: add Codecov test results upload to enable Test Analytics

[bcdonadio]

Summary

This PR fixes the Codecov test results upload issue by adding the codecov/test-results-action@v1 to the CI workflow. Previously, test results were not being uploaded to Codecov Test Analytics despite JUnit XML files being generated correctly.

Changes Made

• Added codecov/test-results-action@v1 step in .github/workflows/ci.yml (after line 222) to upload test results to Codecov Test Analytics
• Fixed codecov/codecov-action@v4 configuration by removing ./junit.xml from the files parameter (line 218), as this action is designed for coverage reports only
• Maintained existing dorny/test-reporter@v1 integration for GitHub Checks visibility

Root Cause

The workflow was incorrectly passing junit.xml to codecov/codecov-action@v4, which is designed for coverage reports (coverage.xml), not test results. While pytest was generating JUnit XML correctly, no action was uploading these test results to Codecov Test Analytics.

Benefits

This change enables several Codecov Test Analytics features:

• ✅ Test failure tracking and trend analysis
• ✅ Flaky test detection
• ✅ Test performance metrics and historical data
• ✅ Better visibility into test suite health

Test Results Distribution

After this change, test results will be available in three places:

1. Codecov Test Analytics (new) - via codecov/test-results-action@v1
2. GitHub Checks (existing) - via dorny/test-reporter@v1
3. Coverage Reports (existing) - via codecov/codecov-action@v4

Test Plan

• Run make check locally - all checks passed
• Verify workflow YAML syntax is valid
• Wait for CI to complete successfully
• Verify test results appear in Codecov Test Analytics dashboard
• Confirm GitHub Checks still show test results as before
• Ensure coverage reports continue uploading correctly

References

• Codecov Test Analytics Documentation
• Related to the investigation of why codecov/test-results-action@v1 wasn't uploading

🤖 Generated with Claude Code

[Copilot]

Pull Request Overview

This PR separates coverage and test results uploads to Codecov by using two distinct actions. Previously, both coverage.xml and junit.xml were uploaded together through the codecov-action. Now, coverage is uploaded via codecov/codecov-action and test results are uploaded separately via codecov/test-results-action.

• Split Codecov uploads into separate steps for coverage and test results
• Introduced codecov/test-results-action@v1 for JUnit test results
• Removed junit.xml from the original codecov-action file list

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".