Skip to content

Improve CI test result and coverage reporting - #74

Merged
uuf6429 merged 2 commits into
minkphp:mainfrom
aik099:ci-improvements
Oct 11, 2026
Merged

uuf6429 merged 2 commits into
minkphp:mainfrom
aik099:ci-improvements

Conversation

@aik099

@aik099 aik099 commented Oct 10, 2026

Copy link
Copy Markdown
Member

Changes:

  • print browser version used by Selenium image (in "Show browser version" step)
  • collect and upload coverage only on default PHP version
  • upload test results from all jobs
  • replace deprecated codecov/test-results-action with codecov/codecov-action@v5 (report_type: test_results)

Coverage is collected only where the driver has code that runs differently. The driver has browser (default capabilities, browser name aliases) and Selenium version (W3C mode) specific code, but no PHP version specific code. So coverage is collected on all browser/Selenium combinations (default PHP version), but not on other PHP versions.

Test results are uploaded from all jobs, so Codecov can detect flaky tests on more test runs.

Same changes in MinkSelenium2Driver: minkphp/MinkSelenium2Driver#416

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.86%. Comparing base (efa6de0) to head (c22a886).

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #74   +/-   ##
=========================================
  Coverage     88.86%   88.86%           
  Complexity      197      197           
=========================================
  Files             1        1           
  Lines           494      494           
=========================================
  Hits            439      439           
  Misses           55       55           
Flag Coverage Δ
chrome 88.25% <ø> (ø)
chromium 87.85% <ø> (ø)
edge 88.25% <ø> (ø)
firefox 88.05% <ø> (ø)
selenium-2.53.1 87.04% <ø> (ø)
selenium-3 88.05% <ø> (ø)
selenium-4 88.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MqZBAjVsaHhnSadusMUzuu
Comment thread .github/workflows/ci.yml
@uuf6429
uuf6429 merged commit 908358a into minkphp:main Oct 11, 2026
17 of 19 checks passed
@aik099
aik099 deleted the ci-improvements branch October 11, 2026 14:45
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