Skip to content

Remove limit on displaying failed resource kinds in test error output #2636

@coderabbitai

Description

@coderabbitai

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

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions