Skip to content

Restructure coverage handling with dedicated cover-linux job#291

Merged
blink1073 merged 8 commits into
Calysto:mainfrom
blink1073:fix-coverage-handling
Mar 14, 2026
Merged

Restructure coverage handling with dedicated cover-linux job#291
blink1073 merged 8 commits into
Calysto:mainfrom
blink1073:fix-coverage-handling

Conversation

@blink1073
Copy link
Copy Markdown
Contributor

@blink1073 blink1073 commented Mar 14, 2026

Summary

  • test-linux and test-other-ubuntu no longer collect or upload coverage (run just test only)
  • Add cover-linux job (ubuntu-latest, Python 3.10) as the dedicated Linux coverage runner; uploads directly to Codecov with flags: linux
  • test-other-systems uploads coverage to Codecov only for windows-latest with flags: windows
  • Add [tool.coverage.run] relative_files = true so coverage XML paths resolve correctly across platforms
  • Set after_n_builds: 2 in codecov.yml to wait for both uploads (linux + windows) before notifying
  • Add cover-linux to tests_check branch protection needs
  • Add tests for PDF class and get_kernel_json with OCTAVE_KERNEL_JSON env var

Test plan

  • Verify cover-linux runs and uploads coverage to Codecov with flags: linux
  • Verify test-other-systems uploads coverage only for windows-latest with flags: windows
  • Verify test-linux and test-other-ubuntu run tests without uploading coverage
  • Verify Codecov notifies after both uploads complete
  • Verify new TestPDF and TestGetKernelJson tests pass

Comment thread .github/workflows/test.yml Dismissed
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.54%. Comparing base (87c234d) to head (e0947fa).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #291   +/-   ##
=======================================
  Coverage   91.54%   91.54%           
=======================================
  Files           6        6           
  Lines         426      426           
=======================================
  Hits          390      390           
  Misses         36       36           
Flag Coverage Δ
linux 91.31% <ø> (?)
linux-py3.10 ?
linux-py3.11 ?
linux-py3.12 ?
linux-py3.13 ?
linux-py3.14 ?
linux-py3.14t ?
macos-latest ?
ubuntu-flatpak ?
ubuntu-snap ?
windows 91.54% <ø> (?)
windows-latest ?

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

Files with missing lines Coverage Δ
octave_kernel/kernel.py 92.47% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@blink1073 blink1073 enabled auto-merge (squash) March 14, 2026 18:16
@blink1073 blink1073 merged commit b5c7289 into Calysto:main Mar 14, 2026
33 checks passed
@blink1073 blink1073 deleted the fix-coverage-handling branch March 14, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants