Pending-plans route integration test creates states the plan service now refuses
Opened by Zhu Yue ·
Reproduction
On current origin/main (including 7ce6d7b38, MOTIR-4106), run:
pnpm exec vitest run --config vitest.config.ts tests/integration/ai/pendingPlansRoute.test.ts
It deterministically fails 6 of 16 tests.
Evidence
The helper planIn() in tests/integration/ai/pendingPlansRoute.test.ts creates a plan, then calls markPlanned, approvePlan, or declinePlan without establishing the proposal state those transitions now require. The failures include PlanNotInExpectedStatusError: Plan … is declined; this action requires it to be planned and pending reads that omit expected generated/planned rows. The same failures occurred in CI run 33624829060, Vitest shard 5/12, and locally after merging origin/main into subtask/MOTIR-4138-deploy-freshness-report.
Where seen
Found while completing MOTIR-4138; the failing test and its pending-plan surface are unrelated to MOTIR-4138’s deploy-freshness workflow change.
Discussion
No comments yet.
Adding to this discussion signs you in on app.motir.co and brings you back to this request.