nimmo/forgejo-auto-pr-review
Watch
1
0
Fork
You've already forked forgejo-auto-pr-review
 0
Code Issues 5 Pull requests Projects Releases Packages 1 Activity Actions

Expand tests with Forgejo and model fixtures #8

New issue
Closed
opened 2026-07-20 15:53:58 +01:00 by nimmo · 0 comments
nimmo commented 2026-07-20 15:53:58 +01:00
Owner
Copy link

Build a meaningful automated test suite around mocked Forgejo and model interactions.

Scope:

  • Fixtures for events, PR metadata/diffs, comments, and API errors.
  • Tests for config validation, guidance precedence, size handling, provider parsing, comment updates, deduplication, injection resistance, and secret redaction.
  • Mock model calls so normal tests never require a live model.

Acceptance:

  • Core behavior is covered by deterministic unit/integration tests.
  • Regression tests cover the failure modes discovered during initial rollout.
Build a meaningful automated test suite around mocked Forgejo and model interactions. Scope: - Fixtures for events, PR metadata/diffs, comments, and API errors. - Tests for config validation, guidance precedence, size handling, provider parsing, comment updates, deduplication, injection resistance, and secret redaction. - Mock model calls so normal tests never require a live model. Acceptance: - Core behavior is covered by deterministic unit/integration tests. - Regression tests cover the failure modes discovered during initial rollout.
nimmo closed this issue 2026-07-20 22:00:59 +01:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/action-yaml-parser
issue-4-normalize-analysis
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
nimmo/forgejo-auto-pr-review#8
No description provided.