-
Notifications
You must be signed in to change notification settings - Fork 68
Open
Description
Description
Currently, the test failure summary in the class generator test only displays the first 5 failed resource kinds, with remaining failures summarized as 'and N more'. This makes it harder to identify all failing resources at a glance.
Current Behavior
In class_generator/tests/manifests/test_parse_explain.j2 template:
failure_summary = f"{len(failures)} resource(s) failed: {', '.join(failed_kinds[:5])}"
if len(failed_kinds) > 5:
failure_summary += f" and {len(failed_kinds) - 5} more"Expected Behavior
Display all failed resource kinds in the error message without truncation, so developers can see the complete list of failures immediately.
Suggested Fix
Remove the [:5] slice and the conditional append:
failure_summary = f"{len(failures)} resource(s) failed: {', '.join(failed_kinds)}"Context
- Related PR: feat: Add validatingAdmissionPolicy and validatingAdmissionPolicyBinding #2635
- Related comment: feat: Add validatingAdmissionPolicy and validatingAdmissionPolicyBinding #2635 (comment)
- Requested by: @myakove
Metadata
Metadata
Assignees
Labels
No labels