On 11/23/20 11:10 PM, Roman Bolshakov wrote:
The test takes 40+ seconds on MBP 2012, MBA 2015. Cirrus completes the test within default timeout, just above 29 seconds but the error margin is narrow, under a second. It'd be good to provide reasonable default timeout to avoid test suite failure if "meson test" is invoked without arguments. Closes https://gitlab.com/libvirt/libvirt/-/issues/58 Signed-off-by: Roman Bolshakov <r.bolshakov@xxxxxxxxx> --- tests/meson.build | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/meson.build b/tests/meson.build index f88410ff33..cb06873ba5 100644 --- a/tests/meson.build +++ b/tests/meson.build @@ -574,7 +574,13 @@ foreach data : tests ], export_dynamic: true, ) - test(data['name'], test_bin, env: tests_env) + if data['name'] == 'qemuxml2argvtest' and host_machine.system() == 'darwin'
If we are doing this, then I'd say do it host arch agnostic.
+ timeout = 90 + else + # default meson timeout + timeout = 30 + endif + test(data['name'], test_bin, env: tests_env, timeout: timeout) endforeach
Michal