-
Notifications
You must be signed in to change notification settings - Fork 697
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
Include test scenario for multiple partitions #11950
Include test scenario for multiple partitions #11950
Conversation
An issue in the Ansible remediation was fixed by ComplianceAsCode#11174 but no test scenario was included to test this condition where there are privileged commands in different partitions.
🤖 A k8s content image for this PR is available at: Click here to see how to deploy itIf you alread have Compliance Operator deployed: Otherwise deploy the content and operator together by checking out ComplianceAsCode/compliance-operator and: |
Code Climate has analyzed commit e3370aa and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 59.4% (0.0% change). View more on Code Climate. |
This new scenario passes (see below) but the existing ones don't seem to. Is this what are you seeing as well?
|
Here they passed fine:
|
The tests pass on my RHEL 8 box.
But on my 9.4 box (similar issue with Ansible).
So I updated another RHEL 9.4 box and everything seems fine there:
So there seems to be something up that one box. I'm willing to merge this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Thanks!
Description:
An issue in the Ansible remediation was fixed by #11174 and later improved by #11263 but no test scenario was included to test this condition where there are privileged commands in different partitions.
Rationale:
Review Hints:
automatus tests should be enough.
It could also be checked the reports generated by automatus tests in the reports folder.