From 2c78e39ab8e00b7e0540cb64282544cff8cd1056 Mon Sep 17 00:00:00 2001 From: Nuwan Date: Sun, 8 Feb 2026 11:59:55 +0530 Subject: [PATCH] docs: define milestone v1.3 requirements 6 requirements across 2 categories: - UNIT-01 through UNIT-03: Jest unit tests - INT-01 through INT-03: Playwright integration tests --- .planning/REQUIREMENTS.md | 51 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 51 insertions(+) create mode 100644 .planning/REQUIREMENTS.md 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*