[joanise]

PR Goal?

Convert all out tests that are still based on unittest.TestCase to pytest.

Fixes?

Gives us more of the benefits of using pytest.

• better output capturing
• better test reports
• the full pytest.fixture framework
• pytest.parametrize
• and more

Feedback sought?

Mostly rubber stamping, maybe sanity checking. This PR touches a lot of code, and the diff is very long, but it's just changing how we run tests, nothing in the real code.

Priority?

medium

Tests added?

only changes tests

How to test?

• uv run pytest still runs all the tests -- note that there are now 407 instead of 403 due to some tests having been refactored using pytest.parametrize
• coverage is unchanged
• CI passes

Confidence?

high

Version change?

no

Related PRs?

EveryVoiceTTS/StyleTTS2#40
EveryVoiceTTS/FastSpeech2_lightning#170

Some of this refactoring was done with the assistance of Claude Opus 5.5, fully human reviewed.

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
everyvoice/tests/test_text.py	78% smaller
everyvoice/tests/preprocessed_audio_fixture.py	68% smaller
everyvoice/tests/test_custom_g2p.py	67% smaller
everyvoice/tests/test_preprocessing.py	63% smaller
everyvoice/tests/test_wizard.py	55% smaller
everyvoice/tests/test_model.py	53% smaller
everyvoice/tests/test_wizard_helpers.py	52% smaller
everyvoice/tests/test_utils.py	39% smaller
everyvoice/tests/test_configs.py	36% smaller
everyvoice/tests/test_subsample.py	28% smaller
everyvoice/tests/test_evaluation.py	13% smaller
everyvoice/tests/stubs.py	2% smaller
everyvoice/tests/test_symbol_mapping.py	2% smaller
everyvoice/tests/regression/test-demo-app-lj-full.py	2% smaller
.pre-commit-config.yaml	0% smaller
everyvoice/model/e2e/StyleTTS2_lightning	0% smaller
everyvoice/model/feature_prediction/FastSpeech2_lightning	0% smaller
everyvoice/tests/regression/test-demo-app-mix.py	0% smaller

[github-actions]

CLI load time: 0:00.16
Pull Request HEAD: 92b21472aef867fb778c24675c49880617a710db

Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
import time:      6799 |     153854 | everyvoice.cli
import time:      3330 |     141660 | typer.rich_utils

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.25%. Comparing base (a6e1231) to head (92b2147).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #896   +/-   ##
=======================================
  Coverage   85.25%   85.25%           
=======================================
  Files          51       51           
  Lines        4789     4789           
  Branches      712      712           
=======================================
  Hits         4083     4083           
  Misses        534      534           
  Partials      172      172           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[roedoejet]

I think you mean get rid of unittest* once and for all 😆 but yes, skimmed this and it looks all standard. tests pass on a freshly installed env on my mac.