-
Notifications
You must be signed in to change notification settings - Fork 47
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
tests: Show clearer error message when cleanup tests fail #1038
base: main
Are you sure you want to change the base?
Conversation
Previously the error message contained the whole object, but the text was too long and it was truncated, so often we could not read the name and namespace of the objects. Now we show only group, version, kind, namespace and name. Signed-off-by: Andrej Krejcir <[email protected]>
/cc @0xFelix @codingben |
Quality Gate passedIssues Measures |
There's any chance to also print the status or something that will help to understand why it wasn't deleted? |
|
||
// Collect resources to more readable names | ||
objectTypesAndNames := make([]string, 0, len(list.Items)) | ||
for _, item := range list.Items { |
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.
Can you move it to a new function that will return readable names?
objectTypesAndNames := getObjectTypesAndNames(list.Items)
Expect(objectTypesAndNames).To(BeEmpty(), "Some resources were not deleted.")
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.
/approve
IMO it is okay to inline or to extract it as function, I'm fine with both.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: 0xFelix The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
Previously the error message contained the whole object, but the text was too long and it was truncated, so often we could not read the name and namespace of the objects.
Now we show only group, version, kind, namespace and name.
Release note: