New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 3 users
Status: Assigned
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Sign in to add a comment
CommonFormats/AudioProcessingTest take too long
Project Member Reported by, Jul 8 2014 Back to list
There seems to be around 47 formats and each tests takes like 20 seconds...
This is 16mins of the total 26mins duration of modules_unittests.

Can this be reduced?

Can the formats be tested faster? in parallel?
Can the test be moved into modules_tests?
Project Member Comment 1 by, Jul 9 2014
Labels: Area-SignalProcessing
Moving it to modules_tests sounds like a good first step at least since its character is more of integration test rather than unit test.
We should then definitely try to reduce the process time. Thanks for creating the bug
Project Member Comment 2 by, Oct 30 2014
Labels: EngTriaged IceBox
Project Member Comment 3 by, May 26 2015
Project Member Comment 4 by, Jun 9 2015
Owner: ----
Status: Available
Project Member Comment 5 by, Nov 18 2015
Project Member Comment 6 by, Nov 18 2015
 Issue 4713  has been merged into this issue.
Project Member Comment 7 by, Nov 18 2015
Labels: -IceBox
They're now tested in parallel, I think the reasonable path here is to reduce per-test time if that's feasible, or reduce the number of combinations tested.

Also I wanna thaw this (-IceBox) because we've done some work on reducing trybot/buildbot times and this is currently a Major Blocker, yessir!
Project Member Comment 8 by, Nov 19 2015
Labels: IceBox
Adding back Icebox, until we start the work.
Project Member Comment 9 by, Nov 25 2015
Components: Audio
Labels: fixit
Project Member Comment 10 by, Oct 5 2016
Components: -SignalProcessing
Project Member Comment 11 by, Nov 16 2016
Status: Assigned
Project Member Comment 12 by, Nov 10
Henrik, is this still relevant?
Project Member Comment 13 by, Nov 17
Yes, this is still relevant. There are 48 tests in this suite, and they take 180 seconds in total to run (on my Linux Z620).
Sign in to add a comment