GrottoCenter / grottocenter-api / 24807171278

86%

develop: 87%

Build Type

Pull #1537

github

Committed by ClemRz

Commit Message

test(perf): parallelize test suite and fix stability (5x speedup)

Replace HTTP-hitting property tests with conventional integration tests
to eliminate socket hang ups and connection corruption. Add a parallel
test runner that shards by domain folder, each shard against its own
database cloned from a pre-seeded PostgreSQL template. The template
rebuilds automatically when fixtures, models, or SQL migrations change.

Stability:
- Replace 33 HTTP property tests with ~70 conventional it() tests
- Set Connection: close in parallel mode to prevent TCP reuse corruption
- Fix Cave update after hook (broken Waterline association restore)
- Fix rate limiter tests to reuse connections via supertest.agent()
- Fix blacklist property test clock skew tolerance
- Set global Mocha timeout to 10s (was 2s default)

Performance:
- Parallel test runner with folder-based sharding (auto-detected CPUs)
- Template database snapshot with mtime-based staleness detection
- Move auth brute-force delay and argon2 cost to Sails config
- config/env/test.js overrides: delay=0, minimal argon2 cost
- Batch 4 sequential bootstrap SQL queries into one round-trip
- Extract shared fixture order to test/fixtureOrder.js

New commands:
- npm run test              — parallel with auto-snapshot (default)
- npm run test:sequential   — single-process fallback
- npm run test:snapshot     — manually rebuild the template

Pull Request Pull Request #1537: test(perf): parallelize test suite and fix stability (5x speedup)

Coverage Stats

2850 of 3448 branches covered (82.66%)

Branch coverage included in aggregate %.

4 of 5 new or added lines in 2 files covered. (80.0%)

1 existing line in 1 file now uncovered.

5916 of 6712 relevant lines covered (88.14%)

51.13 hits per line