edge-cases: add performance test for many conversations (100+)
This commit is contained in:
parent
8816d5194f
commit
5fb39b0eb3
1 changed files with 11 additions and 4 deletions
|
|
@ -4757,12 +4757,19 @@ If any task fails:
|
|||
- Test follows existing performance test patterns
|
||||
- **Rollback**: Remove test from performance.spec.ts
|
||||
|
||||
- [ ] **Edge 4.3**: Test with many conversations (100+)
|
||||
- [x] **Edge 4.3**: Test with many conversations (100+)
|
||||
- **Scope**: Performance testing - Verify chat list rendering
|
||||
- **Dependencies**: None
|
||||
- **Dependencies**: None ✅
|
||||
- **Risk**: LOW
|
||||
- **Validation**: Many conversations render smoothly
|
||||
- **Rollback**: N/A (testing)
|
||||
- **Validation**: ✅ Performance test added to `apps/web/e2e/performance.spec.ts`:
|
||||
- Test mocks a conversation list with 120 conversations using Playwright route interception
|
||||
- Measures render time, Core Web Vitals (LCP, FCP, TTI)
|
||||
- Verifies render time < 5 seconds for 100+ conversations
|
||||
- Verifies LCP < 3 seconds for large lists
|
||||
- Counts rendered conversation items to verify rendering
|
||||
- Test follows existing performance test patterns
|
||||
- Frontend-only test (React component ChatSidebar), does not involve Rust chat server
|
||||
- **Rollback**: Remove test from performance.spec.ts
|
||||
|
||||
### Edge Case 5: Browser Compatibility
|
||||
- [x] **Edge 5.1**: Test BroadcastChannel support
|
||||
|
|
|
|||
Loading…
Reference in a new issue