Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package openQA for openSUSE:Factory checked in at 2025-12-09 12:52:52 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/openQA (Old) and /work/SRC/openSUSE:Factory/.openQA.new.1939 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "openQA" Tue Dec 9 12:52:52 2025 rev:780 rq:1321638 version:5.1765217707.d6e697fd Changes: -------- --- /work/SRC/openSUSE:Factory/openQA/openQA.changes 2025-12-08 11:54:39.975277218 +0100 +++ /work/SRC/openSUSE:Factory/.openQA.new.1939/openQA.changes 2025-12-09 12:58:04.723443604 +0100 @@ -1,0 +2,7 @@ +Mon Dec 08 20:38:05 UTC 2025 - [email protected] + +- Update to version 5.1765217707.d6e697fd: + * Test commenting on overview page together with TODO filter + * Fix job IDs that are considered for mass-commenting on overview page + +------------------------------------------------------------------- Old: ---- openQA-5.1765009312.be30f6e0.obscpio New: ---- openQA-5.1765217707.d6e697fd.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ openQA-client-test.spec ++++++ --- /var/tmp/diff_new_pack.XkyNkI/_old 2025-12-09 12:58:08.383597886 +0100 +++ /var/tmp/diff_new_pack.XkyNkI/_new 2025-12-09 12:58:08.395598392 +0100 @@ -18,7 +18,7 @@ %define short_name openQA-client Name: %{short_name}-test -Version: 5.1765009312.be30f6e0 +Version: 5.1765217707.d6e697fd Release: 0 Summary: Test package for %{short_name} License: GPL-2.0-or-later ++++++ openQA-devel-test.spec ++++++ --- /var/tmp/diff_new_pack.XkyNkI/_old 2025-12-09 12:58:08.751613399 +0100 +++ /var/tmp/diff_new_pack.XkyNkI/_new 2025-12-09 12:58:08.763613905 +0100 @@ -18,7 +18,7 @@ %define short_name openQA-devel Name: %{short_name}-test -Version: 5.1765009312.be30f6e0 +Version: 5.1765217707.d6e697fd Release: 0 Summary: Test package for %{short_name} License: GPL-2.0-or-later ++++++ openQA-test.spec ++++++ --- /var/tmp/diff_new_pack.XkyNkI/_old 2025-12-09 12:58:09.135629586 +0100 +++ /var/tmp/diff_new_pack.XkyNkI/_new 2025-12-09 12:58:09.159630598 +0100 @@ -18,7 +18,7 @@ %define short_name openQA Name: %{short_name}-test -Version: 5.1765009312.be30f6e0 +Version: 5.1765217707.d6e697fd Release: 0 Summary: Test package for openQA License: GPL-2.0-or-later ++++++ openQA-worker-test.spec ++++++ --- /var/tmp/diff_new_pack.XkyNkI/_old 2025-12-09 12:58:09.467643581 +0100 +++ /var/tmp/diff_new_pack.XkyNkI/_new 2025-12-09 12:58:09.479644087 +0100 @@ -18,7 +18,7 @@ %define short_name openQA-worker Name: %{short_name}-test -Version: 5.1765009312.be30f6e0 +Version: 5.1765217707.d6e697fd Release: 0 Summary: Test package for %{short_name} License: GPL-2.0-or-later ++++++ openQA.spec ++++++ --- /var/tmp/diff_new_pack.XkyNkI/_old 2025-12-09 12:58:09.843659431 +0100 +++ /var/tmp/diff_new_pack.XkyNkI/_new 2025-12-09 12:58:09.867660443 +0100 @@ -99,7 +99,7 @@ %define devel_requires %devel_no_selenium_requires chromedriver Name: openQA -Version: 5.1765009312.be30f6e0 +Version: 5.1765217707.d6e697fd Release: 0 Summary: The openQA web-frontend, scheduler and tools License: GPL-2.0-or-later ++++++ openQA-5.1765009312.be30f6e0.obscpio -> openQA-5.1765217707.d6e697fd.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openQA-5.1765009312.be30f6e0/lib/OpenQA/WebAPI/Controller/Test.pm new/openQA-5.1765217707.d6e697fd/lib/OpenQA/WebAPI/Controller/Test.pm --- old/openQA-5.1765009312.be30f6e0/lib/OpenQA/WebAPI/Controller/Test.pm 2025-12-06 09:21:52.000000000 +0100 +++ new/openQA-5.1765217707.d6e697fd/lib/OpenQA/WebAPI/Controller/Test.pm 2025-12-08 19:15:07.000000000 +0100 @@ -688,6 +688,7 @@ sub _prepare_job_results ($self, $jobs, $job_ids) { my %archs; my %results; + my @job_ids; my $aggregated = { none => 0, passed => 0, @@ -762,12 +763,13 @@ $results{$distri}{$version}{$flavor} //= {}; $results{$distri}{$version}{$flavor}{$test} //= {}; $results{$distri}{$version}{$flavor}{$test}{$arch} = $result; + push @job_ids, $id; # add description my $description = $settings_by_job_id{$id}->{JOB_DESCRIPTION} // $descriptions{$test_suite_names{$id}}; $results{$distri}{$version}{$flavor}{$test}{description} //= $description; } - return (\%archs, \%results, $aggregated); + return (\%archs, \%results, $aggregated, \@job_ids); } sub _prepare_groupids ($self) { @@ -845,7 +847,7 @@ my $limit = $search_args->{limit}; # one more than actual limit so we can check whether the limit was exceeded my $exceeded_limit = @$latest_job_ids >= $limit ? $limit - 1 : 0; my $jobs = $jobs_rs->latest_jobs_from_ids($latest_job_ids, $limit); - my ($archs, $results, $aggregated) = $self->_prepare_job_results($jobs, $latest_job_ids); + my ($archs, $results, $aggregated, $job_ids) = $self->_prepare_job_results($jobs, $latest_job_ids); # determine distri/version from job results if not explicitly specified via search args my @distris = keys %$results; @@ -886,7 +888,7 @@ archs => $archs, results => $results, aggregated => $aggregated, - job_ids => $latest_job_ids, + job_ids => $job_ids, until => $until, parallel_children_collapsable_results_sel => $config->{global}->{parallel_children_collapsable_results_sel}, summary_parts => \@summary_parts, diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/openQA-5.1765009312.be30f6e0/t/ui/10-tests_overview.t new/openQA-5.1765217707.d6e697fd/t/ui/10-tests_overview.t --- old/openQA-5.1765009312.be30f6e0/t/ui/10-tests_overview.t 2025-12-06 09:21:52.000000000 +0100 +++ new/openQA-5.1765217707.d6e697fd/t/ui/10-tests_overview.t 2025-12-08 19:15:07.000000000 +0100 @@ -637,6 +637,24 @@ my $running_job_ids = map { $_->id } $jobs->search({state => RUNNING})->all; is $running_job_ids, 2, 'all relevant jobs restarted'; }; + + subtest 'TODO filter' => sub { + $driver->get('/tests/overview?todo=1&build=0048&distri=opensuse&version=Factory&groupid=1001'); + my $submit_button = $driver->find_element('#add-comments-controls button[id="commentJobsBtn"]'); + $driver->find_element('button[title="Restart or comment jobs"]')->click; + $driver->find_element_by_id('text')->send_keys('comment text'); + is $submit_button->get_text, 'Comment on all 1 jobs', + 'submit button displayed with number of jobs (reduced by TODO filter)'; + $submit_button->click; + + # reload without TO-DO filter to check whether only the one job that matched the TO-DO filter was commented on + $driver->get('/tests/overview?build=0048&distri=opensuse&version=Factory&groupid=1001'); + ok wait_for_element(selector => '[name=jobid_td_99938] .label_comment'), + 'the one job that matched the TODO filter has a comment'; + is @{$driver->find_elements('.status.fa.fa-circle')}, 3, 'more than one job displayed without TODO filter'; + is @{$driver->find_elements('.label_comment')}, 1, + 'only the one job that matched with the TODO filter has a comment'; + }; }; subtest "job dependencies displayed on 'Test result overview' page" => sub { ++++++ openQA.obsinfo ++++++ --- /var/tmp/diff_new_pack.XkyNkI/_old 2025-12-09 12:58:36.096766051 +0100 +++ /var/tmp/diff_new_pack.XkyNkI/_new 2025-12-09 12:58:36.140767906 +0100 @@ -1,5 +1,5 @@ name: openQA -version: 5.1765009312.be30f6e0 -mtime: 1765009312 -commit: be30f6e0ced376807db17274f8d8cbde3883b78a +version: 5.1765217707.d6e697fd +mtime: 1765217707 +commit: d6e697fd706f3ba62423dfcbea8bae87dd8b7bc5
