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

Change: Handle delta results with changed severity, qod and hostname #3902

Merged
merged 3 commits into from
Oct 19, 2023

Conversation

a-h-abdelsalam
Copy link
Contributor

What

Handle delta results with changed severity, qod and hostname. In addition to description differences, differences in severity, qod or hostname are considered as a change in delta results comparison.

Why

Within the new delta reports handling, results that have changes only in severity, qod or hostname are now considered changes instead of gone/new.

References

GEA-320

@a-h-abdelsalam a-h-abdelsalam requested a review from a team as a code owner October 17, 2023 13:50
In addition to description differences, differences in severity,
qod or hostname are considered as a change in delta results comparison.
@github-actions
Copy link

Conventional Commits Report

😢 No conventional commits found.

👉 Learn more about the conventional commits usage at Greenbone.

@a-h-abdelsalam a-h-abdelsalam changed the title Changed: Handle delta results with changed severity, qod and hostname Change: Handle delta results with changed severity, qod and hostname Oct 17, 2023
@a-h-abdelsalam a-h-abdelsalam enabled auto-merge (squash) October 17, 2023 13:59
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Merging #3902 (5398dc7) into main (9275bbd) will increase coverage by 0.01%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##             main    #3902      +/-   ##
==========================================
+ Coverage   59.90%   59.92%   +0.01%     
==========================================
  Files        1021     1022       +1     
  Lines       25098    25113      +15     
  Branches     7426     7437      +11     
==========================================
+ Hits        15035    15049      +14     
- Misses       9115     9116       +1     
  Partials      948      948              
Files Coverage Δ
src/web/components/icon/deltadifferenceicon.js 100.00% <100.00%> (ø)
src/web/pages/results/row.js 87.80% <100.00%> (+5.04%) ⬆️
src/web/pages/results/details.js 47.77% <75.00%> (+0.05%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bjoernricks
Copy link
Contributor

Could you extend your PR for some tests?

a-h-abdelsalam and others added 2 commits October 17, 2023 17:18
Co-authored-by: Björn Ricks <bjoern.ricks@greenbone.net>
@a-h-abdelsalam a-h-abdelsalam merged commit 564ecf2 into main Oct 19, 2023
13 checks passed
@a-h-abdelsalam a-h-abdelsalam deleted the delta-reports-v2 branch October 19, 2023 09:09
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.

3 participants