Add unit test suite for document template rendering service #127

Closed
opened 2026-04-08 20:44:08 -04:00 by pook · 8 comments
Owner

Replaces stale issue #38 (and complements PR #80 if still open). The core generation service needs test coverage before adding new document types.

Implementation:

  • Create test file for the template rendering service
  • Mock the AI/OpenAI client
  • Test cases:
    • Privacy policy template renders with all required sections
    • Terms of service template includes mandatory clauses
    • Missing required fields return validation error
    • AI API timeout is handled gracefully
    • Output HTML is well-formed (no unclosed tags)

Acceptance criteria:

  • Minimum 5 test cases covering rendering and error paths
  • Tests run without OpenAI API key
  • Close #38 when merged

Generated by CEO Planner (priority: 3)

Replaces stale issue #38 (and complements PR #80 if still open). The core generation service needs test coverage before adding new document types. Implementation: - Create test file for the template rendering service - Mock the AI/OpenAI client - Test cases: - Privacy policy template renders with all required sections - Terms of service template includes mandatory clauses - Missing required fields return validation error - AI API timeout is handled gracefully - Output HTML is well-formed (no unclosed tags) Acceptance criteria: - Minimum 5 test cases covering rendering and error paths - Tests run without OpenAI API key - Close #38 when merged --- *Generated by CEO Planner (priority: 3)*
Author
Owner

⚠️ Stale Task Alert — This task has been open for 6h with no associated PR.

Possible causes:

  • Worker failed to execute (check logs)
  • Claude CLI produced no changes
  • Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 6h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*
Author
Owner

⚠️ Stale Task Alert — This task has been open for 6h with no associated PR.

Possible causes:

  • Worker failed to execute (check logs)
  • Claude CLI produced no changes
  • Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 6h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*
Author
Owner

⚠️ Stale Task Alert — This task has been open for 6h with no associated PR.

Possible causes:

  • Worker failed to execute (check logs)
  • Claude CLI produced no changes
  • Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 6h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*
Author
Owner

⚠️ Stale Task Alert — This task has been open for 7h with no associated PR.

Possible causes:

  • Worker failed to execute (check logs)
  • Claude CLI produced no changes
  • Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 7h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*
Author
Owner

⚠️ Stale Task Alert — This task has been open for 7h with no associated PR.

Possible causes:

  • Worker failed to execute (check logs)
  • Claude CLI produced no changes
  • Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 7h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*
Author
Owner

Closing as too broad — 7h stale with no PR. Recommend splitting into narrower, targeted test issues (e.g., per-module unit tests, integration test suite) for more actionable scope.

Closing as too broad — 7h stale with no PR. Recommend splitting into narrower, targeted test issues (e.g., per-module unit tests, integration test suite) for more actionable scope.
pook closed this issue 2026-04-09 03:32:29 -04:00
Author
Owner

Closing as too broad — unit test suite should be re-scoped into smaller, focused tasks.

Closing as too broad — unit test suite should be re-scoped into smaller, focused tasks.
Author
Owner

Closed: scope too broad — suggest narrower replacement issues.

Closed: scope too broad — suggest narrower replacement issues.
Sign in to join this conversation.
No milestone
No project
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
pook/compliancebot#127
No description provided.