Skip to content

Fix integration tests#6724

Merged
snejus merged 8 commits into
masterfrom
fix-integration-tests
Jun 13, 2026
Merged

Fix integration tests#6724
snejus merged 8 commits into
masterfrom
fix-integration-tests

Conversation

@snejus

@snejus snejus commented Jun 8, 2026

Copy link
Copy Markdown
Member

This PR tightens the integration test workflow so CI runs the dependency-heavy checks that are usually skipped in local environments.

What changed

  • Install the system packages and Poetry extras needed by integration-only coverage, including ffmpeg/ffprobe, ImageMagick/Pillow-backed art resizing, chroma, docs, replaygain, reflink, lyrics, and related plugin dependencies.
  • Run poe docs before poe test in the integration workflow and set LYRICS_UPDATED=1 so lyrics tests are included in CI.
  • Centralize CI detection in beets.test.helper.RUNNING_IN_CI and reuse shared import/program checks across test collection and helpers.
  • Keep dependency-heavy tests strict in the beetbox GitHub Actions environment, while skipping them locally when optional tools/modules are unavailable.
  • Mark ffprobe-, artresizer-, chroma-, pandoc-, and completion-related cases with focused availability gates so local test runs fail less often for missing optional dependencies.
  • Update docs link-check ignores and stale documentation URLs so poe check-docs-links is less noisy in CI.

@snejus snejus requested a review from a team as a code owner June 8, 2026 23:17
Copilot AI review requested due to automatic review settings June 8, 2026 23:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

grug see PR change GitHub Actions integration test workflow trigger. it stop being only manual/weekly. it now run for pull request and push to master.

Changes:

  • change integration_test.yaml trigger to run on pull_request
  • add push trigger for master
  • remove workflow_dispatch and weekly schedule trigger

Comment thread .github/workflows/integration_test.yaml Outdated
@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.52%. Comparing base (eaba5aa) to head (01792ec).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6724      +/-   ##
==========================================
- Coverage   74.70%   74.52%   -0.19%     
==========================================
  Files         162      162              
  Lines       20822    20822              
  Branches     3298     3298              
==========================================
- Hits        15556    15517      -39     
- Misses       4507     4549      +42     
+ Partials      759      756       -3     
Files with missing lines Coverage Δ
docs/conf.py 88.88% <ø> (ø)

... and 1 file with indirect coverage changes

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

@snejus snejus force-pushed the fix-integration-tests branch 5 times, most recently from 19ebea8 to ec91a8f Compare June 9, 2026 03:00
@snejus snejus marked this pull request as draft June 11, 2026 00:07
@snejus snejus force-pushed the fix-integration-tests branch 6 times, most recently from 948d697 to 40c7151 Compare June 11, 2026 16:09
@snejus snejus marked this pull request as ready for review June 11, 2026 20:41
@github-actions

Copy link
Copy Markdown

Thank you for the PR! The changelog has not been updated, so here is a friendly reminder to check if you need to add an entry.

@snejus snejus force-pushed the fix-integration-tests branch from 4a29785 to e769b5a Compare June 11, 2026 20:46
@snejus snejus force-pushed the fix-integration-tests branch from e769b5a to 01792ec Compare June 13, 2026 00:46
@snejus snejus merged commit ad03b35 into master Jun 13, 2026
22 checks passed
@snejus snejus deleted the fix-integration-tests branch June 13, 2026 01:02
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