close
Skip to content

refactor: Deprecated ImportTestCase in favor of PytestImportHelper#6680

Draft
semohr wants to merge 16 commits into
masterfrom
pytest_import_helper
Draft

refactor: Deprecated ImportTestCase in favor of PytestImportHelper#6680
semohr wants to merge 16 commits into
masterfrom
pytest_import_helper

Conversation

@semohr
Copy link
Copy Markdown
Contributor

@semohr semohr commented May 29, 2026

Description

This PR introduces PytestImportHelper which replaces ImportTestCase. I tried to propagate all updates already, this PR touches:

  • test_scrub
  • test_filefilter
  • test_replaygain
  • test_chroma
  • test_import: Major refactor to use pytest

I know this is quite a big PR, the changes are very related tho. The only thing missing to allow us to remove the ImportTestCase is to migrate all tests using AutotagImportTestCase.

TODOs:

  • Changelog Not needed as this is an internal refactor only

This is related to the multi-step efforts to improve logging in beets #6553

@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

⚠️ Unsupported file format

Upload processing failed due to unsupported file format. Please review the parser error message:
Error parsing JUnit XML in /home/runner/work/beets/beets/.reports/pytest.xml at 1:82538

Caused by:
RuntimeError: Error converting computed name to ValidatedString

Caused by:
    string is too long</code></pre>

For more help, visit our troubleshooting guide.

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