diff --git a/.planning/REQUIREMENTS.md b/.planning/REQUIREMENTS.md new file mode 100644 index 000000000..1d6f94109 --- /dev/null +++ b/.planning/REQUIREMENTS.md @@ -0,0 +1,51 @@ +# Requirements: Session Settings Tests + +**Defined:** 2026-02-07 +**Core Value:** Enable confident changes to Session Settings modal through comprehensive test coverage + +## v1.3 Requirements + +Requirements for Session Settings test coverage. Critical paths only. + +### Unit Tests (Jest) + +- [ ] **UNIT-01**: Modal renders correctly with currentSession props (privacy, description) +- [ ] **UNIT-02**: Save button calls onSave with correct payload (privacy number, description) +- [ ] **UNIT-03**: Loading state disables form interactions (save button, inputs) + +### Integration Tests (Playwright) + +- [ ] **INT-01**: Settings button in session toolbar opens modal +- [ ] **INT-02**: Save settings makes PUT /sessions/{id} API call with correct data +- [ ] **INT-03**: Cancel closes modal without making API call + +## Out of Scope + +| Feature | Reason | +|---------|--------| +| Exhaustive state change tests | Core paths only, not every input change | +| Privacy transform unit tests | Covered implicitly by save payload test | +| Error handling tests | Deferred to keep scope minimal | +| Success toast tests | Deferred to keep scope minimal | +| Visual regression tests | Not part of this milestone | +| Other modal tests | Only JKSessionSettingsModal | + +## Traceability + +| Requirement | Phase | Status | +|-------------|-------|--------| +| UNIT-01 | Phase 17 | Pending | +| UNIT-02 | Phase 17 | Pending | +| UNIT-03 | Phase 17 | Pending | +| INT-01 | Phase 18 | Pending | +| INT-02 | Phase 18 | Pending | +| INT-03 | Phase 18 | Pending | + +**Coverage:** +- v1.3 requirements: 6 total +- Mapped to phases: 6 +- Unmapped: 0 ✓ + +--- +*Requirements defined: 2026-02-07* +*Last updated: 2026-02-07 after initial definition*