bckelley/cgm-remote-monitor
1
0
Fork 0
mirror of https://github.com/bckelley/cgm-remote-monitor.git synced 2026-08-24 03:14:12 -05:00
Code Issues Packages Projects Releases Wiki Activity

Improve test performance by optimizing server boot handling

Browse Source 
Refactor tests/api.shape-handling.test.js to use `before()` for server boot instead of `beforeEach()`, reducing boot overhead and improving test stability. Update docs/test-specs/flaky-tests.md and replit.md to reflect the fix and progress tracker.

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: dd54303b-52a7-4c5b-9110-6cd8643ee3bb
Replit-Commit-Checkpoint-Type: intermediate_checkpoint
Replit-Commit-Event-Id: fa6ecd04-4886-43f8-b6ed-5ca9e21eb746
Replit-Helium-Checkpoint-Created: true
This commit is contained in:
bewest
2026-01-19 13:14:46 -08:00
committed by Ben West
parent a2c4e529ea
commit c3c893334d
3 changed files with 103 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
replit.md
+3 -3
View File
@@ -140,18 +140,18 @@ npm run test:slow # Tests with slow threshold logging
### Known Test Issues (Last Analysis: January 19, 2026)
**Summary:** ✅ **TESTS STABLE** - Stress testing (3-5 iterations per test file) shows 100% pass rate across completed runs. One test file (api.shape-handling) timed out during stress testing due to slow server boot overhead.
**Summary:** ✅ **TESTS STABLE** - Stress testing (3-5 iterations per test file) shows 100% pass rate across all test files including api.shape-handling (now optimized).
**Recent Fixes:**
- `api.shape-handling.test.js` - Fixed by optimizing server boot (beforeEach→before); now runs in ~6s with 172ms/test avg
- `api.deduplication.test.js` - Fixed timeout issues by increasing timeout to 30s and optimizing cleanup
- `boluswizardpreview.test.js` - All 10 tests now pass
- `api3.renderer.test.js` - XML/CSV content type tests now pass
**Flaky Test Status:** See `docs/test-specs/flaky-tests.md` for current status summary.
**Flaky Test Status:** See `docs/test-specs/flaky-tests.md` for current status summary and improvement roadmap.
#### Slow Tests
Some tests are slow due to server boot overhead (2-3s per test):
- `api.shape-handling.test.js` - Slow; stress test timed out before completion (needs longer timeout)
- `concurrent-writes.test.js` - AAPS sync simulation tests are slow by design but pass consistently
- `v1 API Batch Operations` - Large batch operations take longer but pass consistently