Skip to content

Commit fa38444

Browse files
committed
tests: Reduce time of "All options" test
Using REQUIRE from catch2 in a hot loop is terrible for performance.
1 parent d92e0f1 commit fa38444

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/common/src/tests/tests.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,10 @@ TEST_CASE("All options", "[common_plugin]") {
5959

6060
for (int ch = 0; ch < channels; ch++) {
6161
for (int j = 0; j < sampleFrames; j++) {
62-
REQUIRE(outputs[ch][j] != -1.f);
62+
if (outputs[ch][j] != -1.f) {
63+
CAPTURE(ch, j);
64+
FAIL("No output written");
65+
}
6366
}
6467
}
6568
}

0 commit comments

Comments
 (0)