Skip to content

Add PR comment renderer fallback tests - #54

Merged
awamonem merged 1 commit into
trajectly:mainfrom
Scottcjn:test-pr-comment-renderer-fallbacks
Mar 12, 2026
Merged

awamonem merged 1 commit into
trajectly:mainfrom
Scottcjn:test-pr-comment-renderer-fallbacks

Conversation

@Scottcjn

Copy link
Copy Markdown
Contributor

Summary

  • add coverage for the empty-report fallback row in render_pr_comment()
  • add coverage to ensure non-dict report rows are ignored without leaking into the output
  • add coverage for the missing repro_command fallback to `n/a`

Testing

  • pytest tests/unit/test_report_renderers.py -q
  • git diff --check

Closes #40.

@awamonem awamonem left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. This covers the fallback cases requested in #40 and the targeted test file passes. Approving.

@awamonem
awamonem merged commit aff1132 into trajectly:main Mar 12, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit tests for PR comment renderer fallback cases

2 participants