decentraland / worlds-content-server / 28983405533

89%

main: 91%

Build Type

Pull #504

github

Committed by LautaroPetaccio

Commit Message

fix: address second-review findings on the partial-upload path

- skip the newer-deployment check on resume batches: intermediate batches
  never deploy by themselves, so the pre-finalize re-check (kept right before
  deployScene) is the load-bearing guard; this removes a world_scenes query
  from every batch of a multi-request upload
- project only the content hashes out of the pending scenes' entity jsonb in
  getActivePendingKeys, instead of shipping every staged manifest to the GC
  once per delete batch
- restructure the partial-deploy integration spec per the dcl-testing
  standard: all setup and actions in beforeEach, it() blocks assert-only,
  flows expressed as nested describes (14 scenarios -> 32 assert-only cases)

Pull Request Pull Request #504: feat: support partial (multi-request) scene uploads

Coverage Stats

1465 of 1755 branches covered (83.48%)

Branch coverage included in aggregate %.

145 of 162 new or added lines in 9 files covered. (89.51%)

4 existing lines in 3 files now uncovered.

3117 of 3398 relevant lines covered (91.73%)

86.76 hits per line