forked from openshift/compliance-operator
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Refactor e2e test setup #264
Merged
openshift-merge-robot
merged 40 commits into
ComplianceAsCode:master
from
rhmdnd:refactor-e2e-test-setup
Apr 10, 2023
Merged
Refactor e2e test setup #264
openshift-merge-robot
merged 40 commits into
ComplianceAsCode:master
from
rhmdnd:refactor-e2e-test-setup
Apr 10, 2023
Commits on Apr 8, 2023
-
We recently made a change to ensure we clean up resources properly after we perform e2e tests. Specifically to avoid race conditions that leaves the compliance operator resources in a bricked state, even when the tests pass. This commit uses a similar pattern for test setup by leveraging the `setUp` method before calling `m.Run()` to make sure the cluster is ready before we run the actual tests. One advantage is that we're simplifying the logic to a single layer before the tests run. Further changes will make it easier to decouple the framework, context, and e2eutils into a single set of utilities we can share across repositories and test suites.
Configuration menu - View commit details
-
Copy full SHA for adc9669 - Browse repository at this point
Copy the full SHA adc9669View commit details -
Now that the framework is mostly decoupled from the tests, we can reuse it for separate test modules for parallel and serial tests. The benefit of this is that we can flatten the tests. Which aid readability, reduces the ammount of variables we need to pass into tests regardless if they're used, allows us to use native go tools for test discovery, and gives use the opportunity to remove the custom test runner implementation we have to handle parallel and serial tests.
Configuration menu - View commit details
-
Copy full SHA for 31538f0 - Browse repository at this point
Copy the full SHA 31538f0View commit details -
Configuration menu - View commit details
-
Copy full SHA for dea1379 - Browse repository at this point
Copy the full SHA dea1379View commit details -
Port TestInvalidBundleWithUnexistentRef to parallel tests
Move TestInvalidBundleWithUnexistentRef to the new parallel test module, which provides a flat structure and doesn't rely on the custom test runner from helpers.go.
Configuration menu - View commit details
-
Copy full SHA for e193df0 - Browse repository at this point
Copy the full SHA e193df0View commit details -
Configuration menu - View commit details
-
Copy full SHA for a2fa7b8 - Browse repository at this point
Copy the full SHA a2fa7b8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6fdc3f3 - Browse repository at this point
Copy the full SHA 6fdc3f3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5849c68 - Browse repository at this point
Copy the full SHA 5849c68View commit details -
Configuration menu - View commit details
-
Copy full SHA for 00f91c4 - Browse repository at this point
Copy the full SHA 00f91c4View commit details -
Configuration menu - View commit details
-
Copy full SHA for a404552 - Browse repository at this point
Copy the full SHA a404552View commit details -
Configuration menu - View commit details
-
Copy full SHA for 78b3a73 - Browse repository at this point
Copy the full SHA 78b3a73View commit details -
Configuration menu - View commit details
-
Copy full SHA for 419997e - Browse repository at this point
Copy the full SHA 419997eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 31aa738 - Browse repository at this point
Copy the full SHA 31aa738View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5db30ae - Browse repository at this point
Copy the full SHA 5db30aeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9147a45 - Browse repository at this point
Copy the full SHA 9147a45View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9c45a2b - Browse repository at this point
Copy the full SHA 9c45a2bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 752b2b2 - Browse repository at this point
Copy the full SHA 752b2b2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6393fee - Browse repository at this point
Copy the full SHA 6393feeView commit details -
Configuration menu - View commit details
-
Copy full SHA for f024ba2 - Browse repository at this point
Copy the full SHA f024ba2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5af0571 - Browse repository at this point
Copy the full SHA 5af0571View commit details -
Configuration menu - View commit details
-
Copy full SHA for eed6a45 - Browse repository at this point
Copy the full SHA eed6a45View commit details -
Configuration menu - View commit details
-
Copy full SHA for b282e8a - Browse repository at this point
Copy the full SHA b282e8aView commit details -
Configuration menu - View commit details
-
Copy full SHA for ebb1cf1 - Browse repository at this point
Copy the full SHA ebb1cf1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ea874a - Browse repository at this point
Copy the full SHA 8ea874aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5f44887 - Browse repository at this point
Copy the full SHA 5f44887View commit details -
Configuration menu - View commit details
-
Copy full SHA for e682288 - Browse repository at this point
Copy the full SHA e682288View commit details -
Configuration menu - View commit details
-
Copy full SHA for e728b64 - Browse repository at this point
Copy the full SHA e728b64View commit details -
Configuration menu - View commit details
-
Copy full SHA for 93d28fa - Browse repository at this point
Copy the full SHA 93d28faView commit details -
Configuration menu - View commit details
-
Copy full SHA for db5aa70 - Browse repository at this point
Copy the full SHA db5aa70View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5c688b7 - Browse repository at this point
Copy the full SHA 5c688b7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5d78ea5 - Browse repository at this point
Copy the full SHA 5d78ea5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 354b3a9 - Browse repository at this point
Copy the full SHA 354b3a9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 35f75c5 - Browse repository at this point
Copy the full SHA 35f75c5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0c9e3d0 - Browse repository at this point
Copy the full SHA 0c9e3d0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 33d2888 - Browse repository at this point
Copy the full SHA 33d2888View commit details -
Configuration menu - View commit details
-
Copy full SHA for fc8ca45 - Browse repository at this point
Copy the full SHA fc8ca45View commit details -
Configuration menu - View commit details
-
Copy full SHA for 126b8d3 - Browse repository at this point
Copy the full SHA 126b8d3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 75e8a15 - Browse repository at this point
Copy the full SHA 75e8a15View commit details -
Configuration menu - View commit details
-
Copy full SHA for ac3968c - Browse repository at this point
Copy the full SHA ac3968cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2374f73 - Browse repository at this point
Copy the full SHA 2374f73View commit details -
Configuration menu - View commit details
-
Copy full SHA for ffd0ecd - Browse repository at this point
Copy the full SHA ffd0ecdView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.