Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix show results for multiple groups in /test/overview #3973

Merged
merged 1 commit into from Jun 23, 2021

Conversation

okurz
Copy link
Member

@okurz okurz commented Jun 23, 2021

When the query has different groupid and every group
different builds the current operation only returns the jobs
for the last build of the first group.

This PR ensures to return all the jobs for all the last builds for
all the selected groups

https://progress.opensuse.org/issues/91650

Supersedes #3971

When the query has different groupid and every group
different builds the current operation only returns the jobs
for the last build of the first group.

This PR ensures to return all the jobs for all the last builds for
all the selected groups

https://progress.opensuse.org/issues/91650
@codecov
Copy link

codecov bot commented Jun 23, 2021

Codecov Report

Merging #3973 (ffc1376) into master (02699d6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3973   +/-   ##
=======================================
  Coverage   96.86%   96.87%           
=======================================
  Files         370      370           
  Lines       32966    32980   +14     
=======================================
+ Hits        31934    31949   +15     
+ Misses       1032     1031    -1     
Impacted Files Coverage Δ
lib/OpenQA/WebAPI/Controller/Test.pm 93.76% <100.00%> (+0.01%) ⬆️
lib/OpenQA/WebAPI/Plugin/Helpers.pm 94.09% <100.00%> (+0.12%) ⬆️
t/10-tests_overview.t 100.00% <100.00%> (ø)
t/lib/OpenQA/Test/Utils.pm 78.83% <0.00%> (+0.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02699d6...ffc1376. Read the comment docs.

@okurz okurz merged commit d2f81c8 into os-autoinst:master Jun 23, 2021
@okurz okurz deleted the feature/test_overview branch June 23, 2021 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants