Dev.ej/studio tests data

[joanise]

PR Goal?

Make it so that errors from ffmpeg are visible in the logs where there is a test failure.

Also add missing permissions to make workflow more secure.

Fixes?

broken w2v unit testing in the EveryVoice CI workflow

Feedback sought?

mostly rubber stamping

Priority?

normal

Tests added?

just fixing existing tests

How to test?

see that CI passes in EV, both tests and matrix-tests

Confidence?

high

Version change?

no

Related PRs?

EveryVoiceTTS/EveryVoice#767

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
aligner/tests/test_cli.py	22% smaller
.github/workflows/test.yml	0% smaller

[roedoejet]

So I guess it was an ffmpeg issue that crashed CI? Thanks @joanise - looks good.

[github-actions]

CLI load time: 0:00.16
Pull Request HEAD: e4311d7e9f37844f8936d8d054095887db17f2c1
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.86%. Comparing base (d4b491f) to head (e4311d7).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   96.86%   96.86%           
=======================================
  Files           5        5           
  Lines         223      223           
=======================================
  Hits          216      216           
  Misses          7        7           

☔ 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.