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 tests to cover auto-gen rules #572

Closed
realshuting opened this issue Dec 27, 2019 · 5 comments
Closed

Add tests to cover auto-gen rules #572

realshuting opened this issue Dec 27, 2019 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@realshuting
Copy link
Member

After #518 , all the best practices which apply to pod will automatically generate a rule applying on pod controllers. Need to test all the best practices to make sure they are working as expected, also violation should be reported correctly.

@realshuting realshuting added enhancement New feature or request good first issue Good for newcomers labels Dec 27, 2019
@JimBugwadia
Copy link
Member

We need to add tests to cover auto-generated rules on Pod Controllers and also add more negative test cases.

@realshuting realshuting changed the title Test best practice policies and violation behavior Add tests to cover auto-gen rules Jul 16, 2020
@realshuting
Copy link
Member Author

Let's add tests to cover auto-gen feature, we can de-prioritize violation check.

@daviddyball
Copy link

daviddyball commented May 20, 2021

EDIT: Raised a separate issue to track my bug.

@chipzoller
Copy link
Member

Unclear if this now refers to tests in kyverno/policies or e2e tests which are outside those.

@realshuting
Copy link
Member Author

Let's close this issue as it lasts for a long time. Will try to create precise issues next time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

5 participants