Review PR #770: requireSubscription middleware test file #1346

New issue

Open

opened 2026-04-13 14:43:53 -04:00 by pook · 0 comments

pook commented

2026-04-13 14:43:53 -04:00

Owner

Review PR #770 which creates test file at tests/middleware/requireSubscription.test.ts.

Check:

Tests cover: no auth header → 401, expired token → 401, no subscription → 403, active subscription → next()
Tests cover edge cases: past_due subscription, trialing subscription, canceled but not expired
Mocks are realistic — mock Stripe API response, not internal functions
Uses supertest or similar for HTTP-level testing (not just unit tests)
Test file imports from the actual middleware, not a copy
All tests pass (run npm test to verify)

Approve if coverage is complete. This validates the revenue gate.

Generated by CEO Planner (priority: 3)

Review PR #770 which creates test file at tests/middleware/requireSubscription.test.ts. Check: 1. Tests cover: no auth header → 401, expired token → 401, no subscription → 403, active subscription → next() 2. Tests cover edge cases: past_due subscription, trialing subscription, canceled but not expired 3. Mocks are realistic — mock Stripe API response, not internal functions 4. Uses supertest or similar for HTTP-level testing (not just unit tests) 5. Test file imports from the actual middleware, not a copy 6. All tests pass (run npm test to verify) Approve if coverage is complete. This validates the revenue gate. --- *Generated by CEO Planner (priority: 3)*

pook added the

agent-task

label

2026-04-13 14:43:53 -04:00

No labels

agent-task

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#1346

No description provided.

Rows
Columns