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

Add unit test for kubernetes object remediations #5636

Merged
merged 1 commit into from
Apr 17, 2020

Conversation

JAORMX
Copy link
Contributor

@JAORMX JAORMX commented Apr 17, 2020

This adds a test that will iterate through all the relevant remediations
that are kubernetes objects and will verify if they're correctly formed
or not.

This adds a test that will iterate through all the relevant remediations
that are kubernetes objects and will verify if they're correctly formed
or not.
Copy link
Collaborator

@jhrozek jhrozek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@JAORMX JAORMX merged commit 8f7196d into ComplianceAsCode:master Apr 17, 2020
@JAORMX JAORMX deleted the kube-unit-tests branch April 17, 2020 12:36
JAORMX added a commit to JAORMX/content that referenced this pull request Apr 17, 2020
Some audit related Machineconfig objects were introduced as ignition
remediations. They were malformed. This fixes that.

This makes the unit tests pass. Which can be verified as follows:

    make -f tests/unit/kubernetes/Makefile unit

These tests were recently introduced in the following PR: ComplianceAsCode#5636
JAORMX added a commit that referenced this pull request Apr 17, 2020
Some audit related Machineconfig objects were introduced as ignition
remediations. They were malformed. This fixes that.

This makes the unit tests pass. Which can be verified as follows:

    make -f tests/unit/kubernetes/Makefile unit

These tests were recently introduced in the following PR: #5636
@yuumasato yuumasato added this to the 0.1.50 milestone May 11, 2020
This pull request was closed.
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