action #61146
closed
[migration] script_run call for `zypper dup -l` does not fail the tests in obvious way when there are conflicts (was: webui doesn't show red border when the test failed)
Added by mlin7442 over 4 years ago.
Updated almost 3 years ago.
Category:
Enhancement to existing tests
Description
Observation¶
openQA test in scenario opensuse-15.2-NET-x86_64-zdup-Leap-15.0-kde@64bit_cirrus fails in
zdup
the error is happened at https://openqa.opensuse.org/tests/1116470#step/zdup/13 however webui doesn't show any red border in the any step of this test
Expected result¶
Red border should appeared.
Note that the problem is most likely how the result has been recorded by os-autoinst and not within the web UI.
It looks like the zypper call is missing -n
(or --non-interactive
).
script_run("(zypper dup -l;echo ZYPPER-DONE) | tee /dev/$serialdev", 0);
- Project changed from openQA Project to openQA Tests
- Subject changed from webui doesn't show red border when the test failed to [migration] script_run call for `zypper dup -l` does not fail the tests in obvious way when there are conflicts (was: webui doesn't show red border when the test failed)
- Category changed from Regressions/Crashes to Enhancement to existing tests
I am not sure if adding -n
is the intended way. About the original issue: the test API behaves as intended. script_run("(zypper dup -l;echo ZYPPER-DONE) | tee /dev/$serialdev", 0);
is not evaluating return code of the script_run
call and moreover with the parameter 0 the call does not wait for evaluation of the return code. This can be improved from test perspective but the underlying code behaves as designed.
- Status changed from New to Rejected
it passed in 695.1 https://openqa.opensuse.org/tests/1316859 as designed.
this issue here is that the zdup didn't get input at due time.
this test case didn't set WORKAROUND_DEPS=1 so it can input '1' and wait the timeout. which trigger to cleanup.
script_run("(zypper dup -l;echo ZYPPER-DONE) | tee /dev/$serialdev", 0);
if ($out =~ $zypper_dup_conflict) {
if (check_var("WORKAROUND_DEPS", '1')) {
record_info 'workaround dependencies';
send_key '1';
send_key 'ret';
}
else {
$self->result('fail');
save_screenshot;
return;
}
with the upper explanation, we'd reject it as not an issue.
Also available in: Atom
PDF