Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package os-autoinst for openSUSE:Factory checked in at 2025-09-18 21:09:14 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/os-autoinst (Old) and /work/SRC/openSUSE:Factory/.os-autoinst.new.27445 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "os-autoinst" Thu Sep 18 21:09:14 2025 rev:537 rq:1305588 version:5.1758095301.2731a7d Changes: -------- --- /work/SRC/openSUSE:Factory/os-autoinst/os-autoinst.changes 2025-09-14 18:51:22.725121916 +0200 +++ /work/SRC/openSUSE:Factory/.os-autoinst.new.27445/os-autoinst.changes 2025-09-18 21:09:39.820890067 +0200 @@ -1,0 +2,12 @@ +Wed Sep 17 15:14:18 UTC 2025 - [email protected] + +- Update to version 5.1758095301.2731a7d: + * testapi: allow passing max_interval through (assert_)script_run + * t/consoles-s3270.t: fix with IPC::Run 20250809.0 + * backend: Simplify json_fails collection with map + * backend: Simplify check_socket with early-return + * backend: Simplify do_capture by moving loop one level up + * backend: Extract method for do_capture loop body + * backend: Use more member variables in baseclass + +------------------------------------------------------------------- Old: ---- os-autoinst-5.1757691610.477636b.obscpio New: ---- os-autoinst-5.1758095301.2731a7d.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ os-autoinst-devel-test.spec ++++++ --- /var/tmp/diff_new_pack.uV4skL/_old 2025-09-18 21:09:40.612923347 +0200 +++ /var/tmp/diff_new_pack.uV4skL/_new 2025-09-18 21:09:40.612923347 +0200 @@ -18,7 +18,7 @@ %define short_name os-autoinst-devel Name: %{short_name}-test -Version: 5.1757691610.477636b +Version: 5.1758095301.2731a7d Release: 0 Summary: Test package for %{short_name} License: GPL-2.0-or-later ++++++ os-autoinst-openvswitch-test.spec ++++++ --- /var/tmp/diff_new_pack.uV4skL/_old 2025-09-18 21:09:40.640924524 +0200 +++ /var/tmp/diff_new_pack.uV4skL/_new 2025-09-18 21:09:40.640924524 +0200 @@ -19,7 +19,7 @@ %define name_ext -test %define short_name os-autoinst-openvswitch Name: %{short_name}%{?name_ext} -Version: 5.1757691610.477636b +Version: 5.1758095301.2731a7d Release: 0 Summary: test package for %{short_name} License: GPL-2.0-or-later ++++++ os-autoinst-test.spec ++++++ --- /var/tmp/diff_new_pack.uV4skL/_old 2025-09-18 21:09:40.668925701 +0200 +++ /var/tmp/diff_new_pack.uV4skL/_new 2025-09-18 21:09:40.672925868 +0200 @@ -19,7 +19,7 @@ %define name_ext -test %define short_name os-autoinst Name: %{short_name}%{?name_ext} -Version: 5.1757691610.477636b +Version: 5.1758095301.2731a7d Release: 0 Summary: test package for os-autoinst License: GPL-2.0-or-later ++++++ os-autoinst.spec ++++++ --- /var/tmp/diff_new_pack.uV4skL/_old 2025-09-18 21:09:40.700927045 +0200 +++ /var/tmp/diff_new_pack.uV4skL/_new 2025-09-18 21:09:40.700927045 +0200 @@ -17,7 +17,7 @@ Name: os-autoinst -Version: 5.1757691610.477636b +Version: 5.1758095301.2731a7d Release: 0 Summary: OS-level test automation License: GPL-2.0-or-later ++++++ os-autoinst-5.1757691610.477636b.obscpio -> os-autoinst-5.1758095301.2731a7d.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/os-autoinst-5.1757691610.477636b/distribution.pm new/os-autoinst-5.1758095301.2731a7d/distribution.pm --- old/os-autoinst-5.1757691610.477636b/distribution.pm 2025-09-12 17:40:10.000000000 +0200 +++ new/os-autoinst-5.1758095301.2731a7d/distribution.pm 2025-09-17 09:48:21.000000000 +0200 @@ -112,25 +112,26 @@ { timeout => $bmwqemu::default_timeout, output => '', - quiet => undef + quiet => undef, + max_interval => testapi::DEFAULT_MAX_INTERVAL }, ['timeout'], @args); if (testapi::is_serial_terminal) { testapi::wait_serial($self->{serial_term_prompt}, no_regex => 1, quiet => $args{quiet}); } - testapi::type_string "$cmd"; + testapi::type_string "$cmd", max_interval => $args{max_interval}; if ($args{timeout} > 0) { die "Terminator '&' found in script_run call. script_run can not check script success. Use 'background_script_run' instead." if $cmd =~ qr/(?<!\\)&$/; my $str = testapi::hashed_string("SR" . $cmd . $args{timeout}); my $marker = "; echo $str-\$?-" . ($args{output} ? "Comment: $args{output}" : ''); if (testapi::is_serial_terminal) { - testapi::type_string($marker); + testapi::type_string($marker, max_interval => $args{max_interval}); testapi::wait_serial($cmd . $marker, no_regex => 1, quiet => $args{quiet}, buffer_size => length($cmd) + 128); - testapi::type_string("\n"); + testapi::type_string("\n", max_interval => $args{max_interval}); } else { - testapi::type_string "$marker > /dev/$testapi::serialdev\n"; + testapi::type_string "$marker > /dev/$testapi::serialdev\n", max_interval => $args{max_interval}; } my $res = testapi::wait_serial(qr/$str-\d+-/, timeout => $args{timeout}, quiet => $args{quiet}); return unless $res; diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/os-autoinst-5.1757691610.477636b/t/03-testapi.t new/os-autoinst-5.1758095301.2731a7d/t/03-testapi.t --- old/os-autoinst-5.1757691610.477636b/t/03-testapi.t 2025-09-12 17:40:10.000000000 +0200 +++ new/os-autoinst-5.1758095301.2731a7d/t/03-testapi.t 2025-09-17 09:48:21.000000000 +0200 @@ -365,6 +365,26 @@ $mock_testapi->redefine(wait_serial => 'XXXfoo-SCRIPT_FINISHEDXXX'); throws_ok { background_script_run('foo') } qr/PID marker not found/, 'dies without PID marker'; + $cmds = []; + $mock_testapi->redefine(hashed_string => 'XXX'); + is(assert_script_run('true', max_interval => 1), undef, 'nothing happens on success (slow typing)'); + is_deeply($cmds, [ + { + text => 'true', + cmd => 'backend_type_string', + max_interval => 1 + }, + { + text => '; echo XXX-$?-', + cmd => 'backend_type_string', + max_interval => 1 + }, + { + text => "\n", + cmd => 'backend_type_string', + max_interval => 1 + } + ]); $mock_testapi->unmock('wait_serial'); }; @@ -568,15 +588,18 @@ is_deeply($cmds, [ { text => 'curl --form upload=@/var/log/messages --form upname=basetest-messages http://localhost:4243/LookAtMeImAToken/uploadlog/messages', - cmd => 'backend_type_string' + cmd => 'backend_type_string', + max_interval => 250 }, { text => '; echo XXX-$?-', - cmd => 'backend_type_string' + cmd => 'backend_type_string', + max_interval => 250 }, { text => "\n", - cmd => 'backend_type_string' + cmd => 'backend_type_string', + max_interval => 250 } ]); $cmds = []; @@ -584,15 +607,18 @@ is_deeply($cmds, [ { text => 'curl --form upload=@/var/log/messages --form upname=basetest-messages http://localhost:4243/LookAtMeImAToken/uploadlog/messages', - cmd => 'backend_type_string' + cmd => 'backend_type_string', + max_interval => 250 }, { text => '; echo XXX-$?-', - cmd => 'backend_type_string' + cmd => 'backend_type_string', + max_interval => 250 }, { text => "\n", - cmd => 'backend_type_string' + cmd => 'backend_type_string', + max_interval => 250 } ]); delete $bmwqemu::vars{AUTOINST_URL_HOSTNAME}; diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/os-autoinst-5.1757691610.477636b/t/27-consoles-s3270.t new/os-autoinst-5.1758095301.2731a7d/t/27-consoles-s3270.t --- old/os-autoinst-5.1757691610.477636b/t/27-consoles-s3270.t 2025-09-12 17:40:10.000000000 +0200 +++ new/os-autoinst-5.1758095301.2731a7d/t/27-consoles-s3270.t 2025-09-17 09:48:21.000000000 +0200 @@ -33,7 +33,7 @@ $s3270_console->{backend} = backend::baseclass->new(); subtest 's3270_console start' => sub { - my $bless_obj = bless({KIDS => [{VAL => '', PID => '0', NUM => 1, TYPE => 'cmd', RESULT => 1, OPS => []}]}, 'IPC::Run'); + my $bless_obj = bless({KIDS => [{VAL => '', PID => '0', NUM => 1, TYPE => 'cmd', RESULT => 1, OPS => []}], PTYS => {}, PIPES => [], TIMERS => []}, 'IPC::Run'); $ipc_run_mock->redefine(start => sub ($self, $in, $out, $err) { $$out = "success\nconnet($bmwqemu::vars{ZVM_HOST})\nstart to execute process\nok"; return $bless_obj; diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/os-autoinst-5.1757691610.477636b/testapi.pm new/os-autoinst-5.1758095301.2731a7d/testapi.pm --- old/os-autoinst-5.1757691610.477636b/testapi.pm 2025-09-12 17:40:10.000000000 +0200 +++ new/os-autoinst-5.1758095301.2731a7d/testapi.pm 2025-09-17 09:48:21.000000000 +0200 @@ -28,6 +28,7 @@ require bmwqemu; use constant OPENQA_LIBPATH => '/usr/share/openqa/lib'; +use constant DEFAULT_MAX_INTERVAL => 250; our @EXPORT = qw( @@ -936,11 +937,12 @@ # not change default timeout. timeout => 90, fail_message => '', - quiet => testapi::get_var('_QUIET_SCRIPT_CALLS') + quiet => testapi::get_var('_QUIET_SCRIPT_CALLS'), + max_interval => DEFAULT_MAX_INTERVAL }, ['timeout', 'fail_message'], @_); bmwqemu::log_call(cmd => $cmd, %args); - my $ret = $distri->script_run($cmd, timeout => $args{timeout}, quiet => $args{quiet}); + my $ret = $distri->script_run($cmd, timeout => $args{timeout}, quiet => $args{quiet}, max_interval => $args{max_interval}); _handle_script_run_ret($ret, $cmd, %args); return; } @@ -978,7 +980,8 @@ { timeout => $bmwqemu::default_timeout, output => '', - quiet => testapi::get_var('_QUIET_SCRIPT_CALLS') + quiet => testapi::get_var('_QUIET_SCRIPT_CALLS'), + max_interval => DEFAULT_MAX_INTERVAL }, ['timeout'], @_); bmwqemu::log_call(cmd => $cmd, %args); @@ -1390,7 +1393,7 @@ return; } - my $max_interval = $args{max_interval} // 250; + my $max_interval = $args{max_interval} // DEFAULT_MAX_INTERVAL; my $wait = $args{wait_screen_change} // 0; my $wait_still = $args{wait_still_screen} // 0; my $wait_timeout = $args{timeout} // 30; ++++++ os-autoinst.obsinfo ++++++ --- /var/tmp/diff_new_pack.uV4skL/_old 2025-09-18 21:09:45.609133283 +0200 +++ /var/tmp/diff_new_pack.uV4skL/_new 2025-09-18 21:09:45.645134796 +0200 @@ -1,5 +1,5 @@ name: os-autoinst -version: 5.1757691610.477636b -mtime: 1757691610 -commit: 477636bfa4116ad209af6a12c2d033836d4ce77a +version: 5.1758095301.2731a7d +mtime: 1758095301 +commit: 2731a7d4f785d851f0f8888cab07e6e99f73f565
