Refactor a lot of test assertions to pytest style

[joanise]

PR Goal?

Moving further along in the pytest conversion, this PR takes advantage of the nicer assertions and fixtures that pytest provides.

The idea going forward is to always write pytest-style tests, with assert <expr> everywhere, and no longer use TestCase at all. This PR does not go 100% there, but it moves significantly in that direction. For the submodules, however, it's a 100% completed conversion.

Feedback sought?

sanity checking I guess. It's a long diff, but it only touches tests files.

Priority?

normal

How to test?

• look at CI results
• look at coverage, it should not drop since the refactored tests are still the same
• manually: compare running pytest in the parent branch and running pytest in this one.

Confidence?

high

Version change?

no

Related PRs?

EveryVoiceTTS/wav2vec2aligner#30
EveryVoiceTTS/FastSpeech2_lightning#139

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
everyvoice/tests/test_doctests.py	87% smaller
everyvoice/tests/test_dataloader.py	75% smaller
everyvoice/tests/test_model.py	69% smaller
everyvoice/tests/test_subsample.py	69% smaller
everyvoice/tests/test_utils.py	60% smaller
everyvoice/tests/test_cli.py	57% smaller
everyvoice/tests/test_wizard.py	56% smaller
everyvoice/tests/test_wizard_helpers.py	54% smaller
everyvoice/tests/test_preprocessing.py	53% smaller
everyvoice/tests/test_custom_g2p.py	48% smaller
everyvoice/tests/stubs.py	47% smaller
everyvoice/tests/test_text.py	35% smaller
everyvoice/tests/test_evaluation.py	34% smaller
everyvoice/tests/test_configs.py	32% smaller
everyvoice/tests/regression/test-demo-app-mix.py	5% smaller
everyvoice/model/aligner/wav2vec2aligner	0% smaller
everyvoice/model/feature_prediction/FastSpeech2_lightning	0% smaller
everyvoice/tests/preprocessed_audio_fixture.py	0% smaller
pyproject.toml	Unsupported file format

[joanise]

Note: I will clean up the commit history before merging.

[github-actions]

CLI load time: 0:00.32
Pull Request HEAD: d095deebcb7516ead5c22b37ae333a8384440cc0
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
import time:      1797 |     194775 | everyvoice.cli

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.41%. Comparing base (1b85524) to head (d095dee).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #794   +/-   ##
=======================================
  Coverage   83.41%   83.41%           
=======================================
  Files          47       47           
  Lines        4203     4203           
  Branches      615      615           
=======================================
  Hits         3506     3506           
  Misses        567      567           
  Partials      130      130           

☔ View full report in Codecov by Sentry.
📢 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]

lgtm - thanks @joanise !