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: remove e2e tests and cleanup #618

Merged

Conversation

fabriziosestito
Copy link
Contributor

Description

Remove e2e tests, now superseded by: kubewarden/policy-evaluator#399

Also, splits unit tests and integration tests in the CI and Makefile targets.

Signed-off-by: Fabrizio Sestito <fabrizio.sestito@suse.com>
Signed-off-by: Fabrizio Sestito <fabrizio.sestito@suse.com>
@fabriziosestito fabriziosestito requested a review from a team as a code owner December 19, 2023 15:00
Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

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

It always feels good to remove code! :)

@flavio
Copy link
Member

flavio commented Dec 20, 2023

I'll update the required tests suite once the PR is merged.

@fabriziosestito fabriziosestito self-assigned this Dec 20, 2023
@fabriziosestito
Copy link
Contributor Author

@flavio I cannot merge because of the Expected Status, do you have the right to force-merge?

.github/workflows/ci.yml Outdated Show resolved Hide resolved
Signed-off-by: Fabrizio Sestito <fabrizio.sestito@suse.com>
@fabriziosestito fabriziosestito force-pushed the test/remove-e2e-tests-and-cleanup branch from 72ca4ff to b76733b Compare December 20, 2023 13:07
@flavio flavio merged commit fb2605f into kubewarden:main Dec 20, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants