[Git][pulseaudio/pulseaudio][master] tests: remove check2 timeouts in favor of meson

PulseAudio Marge Bot (@pulseaudio-merge-bot) gitlab at gitlab.freedesktop.org
Wed Jan 31 19:38:17 UTC 2024



PulseAudio Marge Bot pushed to branch master at PulseAudio / pulseaudio


Commits:
26ccd116 by matoro at 2024-01-30T12:39:47-05:00
tests: remove check2 timeouts in favor of meson

Meson already handles timeouts, configurable on the command line with
--timeout-multiplier.  They are set to 300s for all tests.

Part-of: <https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/807>

- - - - -


20 changed files:

- src/tests/alsa-mixer-path-test.c
- src/tests/connect-stress.c
- src/tests/cpu-mix-test.c
- src/tests/cpu-remap-test.c
- src/tests/cpu-sconv-test.c
- src/tests/cpu-volume-test.c
- src/tests/extended-test.c
- src/tests/interpol-test.c
- src/tests/lfe-filter-test.c
- src/tests/lo-latency-test.c
- src/tests/lock-autospawn-test.c
- src/tests/meson.build
- src/tests/mult-s16-test.c
- src/tests/once-test.c
- src/tests/passthrough-test.c
- src/tests/rtpoll-test.c
- src/tests/sync-playback.c
- src/tests/thread-mainloop-test.c
- src/tests/thread-test.c
- src/tests/volume-test.c


Changes:

=====================================
src/tests/alsa-mixer-path-test.c
=====================================
@@ -107,7 +107,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Alsa-mixer-path");
     tc = tcase_create("alsa-mixer-path");
     tcase_add_test(tc, mixer_path_test);
-    tcase_set_timeout(tc, 30);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/connect-stress.c
=====================================
@@ -223,7 +223,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Connect Stress");
     tc = tcase_create("connectstress");
     tcase_add_test(tc, connect_stress_test);
-    tcase_set_timeout(tc, 20 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/cpu-mix-test.c
=====================================
@@ -212,7 +212,6 @@ int main(int argc, char *argv[]) {
 #if defined (__arm__) && defined (__linux__) && defined (HAVE_NEON)
     tcase_add_test(tc, mix_neon_test);
 #endif
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/cpu-remap-test.c
=====================================
@@ -524,7 +524,6 @@ int main(int argc, char *argv[]) {
 #if defined (__arm__) && defined (__linux__) && defined (HAVE_NEON)
     tcase_add_test(tc, remap_neon_test);
 #endif
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     tc = tcase_create("rearrange");
@@ -532,7 +531,6 @@ int main(int argc, char *argv[]) {
 #if defined (__arm__) && defined (__linux__) && defined (HAVE_NEON)
     tcase_add_test(tc, rearrange_neon_test);
 #endif
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/cpu-sconv-test.c
=====================================
@@ -251,7 +251,6 @@ int main(int argc, char *argv[]) {
 #if defined (__arm__) && defined (__linux__) && defined (HAVE_NEON)
     tcase_add_test(tc, sconv_neon_test);
 #endif
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/cpu-volume-test.c
=====================================
@@ -235,7 +235,6 @@ int main(int argc, char *argv[]) {
     tcase_add_test(tc, svolume_arm_test);
 #endif
     tcase_add_test(tc, svolume_orc_test);
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/extended-test.c
=====================================
@@ -211,7 +211,6 @@ int main(int argc, char *argv[]) {
     tc = tcase_create("extended");
     tcase_add_test(tc, extended_test);
     /* 4s of audio, 0.5s grace time */
-    tcase_set_timeout(tc, 4.5);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/interpol-test.c
=====================================
@@ -282,7 +282,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Interpol");
     tc = tcase_create("interpol");
     tcase_add_test(tc, interpol_test);
-    tcase_set_timeout(tc, 5 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/lfe-filter-test.c
=====================================
@@ -184,7 +184,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("lfe-filter");
     tc = tcase_create("lfe-filter");
     tcase_add_test(tc, lfe_filter_test);
-    tcase_set_timeout(tc, 10);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/lo-latency-test.c
=====================================
@@ -175,7 +175,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Loopback latency");
     tc = tcase_create("loopback latency");
     tcase_add_test(tc, loopback_test);
-    tcase_set_timeout(tc, 5 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/lock-autospawn-test.c
=====================================
@@ -114,10 +114,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Lock Auto Spawn");
     tc = tcase_create("lockautospawn");
     tcase_add_test(tc, lockautospawn_test);
-    /* the default timeout is too small,
-     * set it to a reasonable large one.
-     */
-    tcase_set_timeout(tc, 60 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/meson.build
=====================================
@@ -208,6 +208,7 @@ endif
 
 test_env = environment()
 test_env.set('MAKE_CHECK', '1')
+test_env.set('CK_DEFAULT_TIMEOUT', '0')
 
 foreach t : default_tests + norun_tests
   name = t[0]


=====================================
src/tests/mult-s16-test.c
=====================================
@@ -102,7 +102,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Mult-s16");
     tc = tcase_create("mult-s16");
     tcase_add_test(tc, mult_s16_test);
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/once-test.c
=====================================
@@ -132,10 +132,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Once");
     tc = tcase_create("once");
     tcase_add_test(tc, once_test);
-    /* the default timeout is too small,
-     * set it to a reasonable large one.
-     */
-    tcase_set_timeout(tc, 60 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/passthrough-test.c
=====================================
@@ -335,7 +335,6 @@ int main(int argc, char *argv[]) {
     tcase_add_test(tc, passthrough_playback_test);
     sink_num++;
     tcase_add_test(tc, passthrough_volume_test);
-    tcase_set_timeout(tc, 5);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/rtpoll-test.c
=====================================
@@ -91,10 +91,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("RT Poll");
     tc = tcase_create("rtpoll");
     tcase_add_test(tc, rtpoll_test);
-    /* the default timeout is too small,
-     * set it to a reasonable large one.
-     */
-    tcase_set_timeout(tc, 60 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/sync-playback.c
=====================================
@@ -208,7 +208,6 @@ int main(int argc, char *argv[]) {
     tc = tcase_create("syncplayback");
     tcase_add_test(tc, sync_playback_test);
     /* 4s of audio, 0.5s grace time */
-    tcase_set_timeout(tc, 4.5);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/thread-mainloop-test.c
=====================================
@@ -102,10 +102,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Thread MainLoop");
     tc = tcase_create("threadmainloop");
     tcase_add_test(tc, thread_mainloop_test);
-    /* the default timeout is too small,
-     * set it to a reasonable large one.
-     */
-    tcase_set_timeout(tc, 60 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/thread-test.c
=====================================
@@ -153,7 +153,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Thread");
     tc = tcase_create("thread");
     tcase_add_test(tc, thread_test);
-    tcase_set_timeout(tc, 60 * 60);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);


=====================================
src/tests/volume-test.c
=====================================
@@ -161,7 +161,6 @@ int main(int argc, char *argv[]) {
     s = suite_create("Volume");
     tc = tcase_create("volume");
     tcase_add_test(tc, volume_test);
-    tcase_set_timeout(tc, 120);
     suite_add_tcase(s, tc);
 
     sr = srunner_create(s);



View it on GitLab: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/commit/26ccd1167a6188fb28745f3f5c9940657f64343c

-- 
View it on GitLab: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/commit/26ccd1167a6188fb28745f3f5c9940657f64343c
You're receiving this email because of your account on gitlab.freedesktop.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/pulseaudio-commits/attachments/20240131/99f891a4/attachment-0001.htm>


More information about the pulseaudio-commits mailing list