Skip to content

[quality] add regression coverage for _section_footer in render_pr_body - #412

Open
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
quality/test-render-pr-body-coverage
Open

[quality] add regression coverage for _section_footer in render_pr_body#412
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
quality/test-render-pr-body-coverage

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Test Improvement

Adds 20 new tests (10 parametrized for action + script copies) covering
_section_footer() in render_pr_body.py, which had zero test coverage.

Coverage added

  • separator (---) present in footer output
  • maintainers group reference (@projectbluefin/maintainers) present
  • gh pr merge command present
  • --admin flag present
  • --repo flag included when repo arg is non-empty
  • --repo flag absent when repo arg is empty or default
  • <pr-number> placeholder present
  • approval count (2) mentioned
  • full main() render includes footer with correct --repo flag

Both the action copy and the script copy are exercised via the existing
TEST_MODULES = [render_pr_body, SCRIPT_RENDER_PR_BODY] parametrize pattern.
All 70 tests pass locally.

Related Issue

Fixes #364


Filed by quality agent (hold-gated mode). Human review required.

— hive: agent=quality backend=copilot model=claude-opus-5 copilot=1.0.59

_section_footer() has zero test coverage in test_render_pr_body.py.
Both the action copy (.github/actions/render-pr-body/render_pr_body.py)
and the script copy (scripts/render_pr_body.py) are exercised via the
existing TEST_MODULES parametrize pattern.

New tests verify:
- separator (---) present in footer
- maintainers group reference present
- gh pr merge command present
- --admin flag present
- --repo flag included when repo arg is non-empty
- --repo flag absent when repo arg is empty or default
- <pr-number> placeholder present
- approval count (2) mentioned
- full main() render includes footer with correct --repo flag

Closes #364

Signed-off-by: quality-bot <quality-bot@projectbluefin.hive>
@kubestellar-hive
kubestellar-hive Bot requested review from castrojo and p5 as code owners August 7, 2026 20:49
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Work is intentionally paused. quality testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[quality] Add regression coverage for promotion PR body renderer

0 participants