decentraland / worlds-content-server / 29926143984

92%

main: 92%

Build Type

Pull #512

github

Committed by LautaroPetaccio

Commit Message

fix: harden deployment cancellation classification and observability

- Classify Node AbortError rejections (fs, streams) as cancellations in
  processing metrics, unwrapping the deadline reason from their cause
- Record validation failures as a distinct client-error outcome so
  outcome="error" only reflects server-side problems
- Log genuine failures that race with a disconnect or the processing
  deadline before returning the 499/408 cancellation response
- Document the 499 response for POST /entities
- Rename getConcurrency to getPositiveInteger since it also reads the
  processing timeout
- Document the single-signal memoization assumption in toDeploymentFile
- Align the worlds-manager mock deployer casing with the real adapter
- Give the paced-upload entity-deployer test hook a 30s budget to avoid
  flaking under parallel-worker load

Pull Request Pull Request #512: feat: reduce large scene deployment processing time

Coverage Stats

1725 of 2007 branches covered (85.95%)

Branch coverage included in aggregate %.

303 of 318 new or added lines in 10 files covered. (95.28%)

1 existing line in 1 file now uncovered.

3646 of 3876 relevant lines covered (94.07%)

113.38 hits per line