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

test: Move and cleanup OpenSCAP tests (HMS-4658) #2391

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

regexowl
Copy link
Collaborator

This moves OpenSCAP tests to Oscap.test.tsx and cleans them up.

@regexowl regexowl force-pushed the move-and-cleanup-compliance-tests branch from 026375b to d7b59eb Compare August 30, 2024 08:28
@mgold1234 mgold1234 force-pushed the move-and-cleanup-compliance-tests branch from d7b59eb to bfae3f2 Compare September 1, 2024 13:04
@regexowl regexowl changed the title test: Move and cleanup OpenSCAP tests test: Move and cleanup OpenSCAP tests (HMS-4658) Sep 2, 2024
@regexowl regexowl force-pushed the move-and-cleanup-compliance-tests branch from bfae3f2 to e0c90c5 Compare September 2, 2024 15:00
@regexowl regexowl force-pushed the move-and-cleanup-compliance-tests branch from e0c90c5 to 979198d Compare September 3, 2024 07:59
Copy link

codecov bot commented Sep 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.64%. Comparing base (1a7350e) to head (dd4bda8).
Report is 2151 commits behind head on main.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2391      +/-   ##
==========================================
+ Coverage   75.71%   83.64%   +7.93%     
==========================================
  Files          33      153     +120     
  Lines         597    17235   +16638     
  Branches      144     1673    +1529     
==========================================
+ Hits          452    14416   +13964     
- Misses        139     2800    +2661     
- Partials        6       19      +13     

see 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

This moves OpenSCAP tests from `CreateImageWizard.compliance.tsx` test file directly to `Oscap.test.tsx`.
This cleans up OpenSCAP test file, utilising helper functions we already have and updating names of test and describe blocks to be consistent throughout the test files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants