Actions
action #38606
closedproceed_on_failure option in script_output is broken
Description
apparently calling script_output('blah', proceed_on_failure => 1); ends up in the following:
[2018-07-19T13:50:15.0373 CEST] [debug] <<< testapi::wait_serial(regexp='SCRIPT_FINISHED25dbl-\\d+-', record_output=1, timeout='proceed_on_failure')
Argument "proceed_on_failure" isn't numeric in multiplication (*) at /usr/lib/os-autoinst/bmwqemu.pm line 316.
https://github.com/os-autoinst/os-autoinst/pull/954
Argument "proceed_on_failure" isn't numeric in multiplication (*) at /usr/lib/os-autoinst/bmwqemu.pm line 316.
Updated by asmorodskyi over 6 years ago
I found root cause, it is really simple to fix the issue :
you need just call function in documented way - script_output('blah',30 , proceed_on_failure => 1);
Updated by mkittler about 6 years ago
So the issue can be considered resolved/rejected?
Updated by okurz about 6 years ago
- Status changed from New to Rejected
- Assignee set to okurz
yes, the "wait" parameter is simply a mandatory one. We can't fix perl's crazy approach of function parameter validation ;)
Actions