decentraland / social-service-ea / 28064991444

90%

main: 90%

Pull #420

github

test: await places requests sequentially to avoid cross-test pollution

The 'adding a place that already exists' test issued its requests inside an
async callback passed to Array.prototype.forEach, which does not await them.
The assertions ran as floating promises after the test (and its afterEach
community cleanup) finished, so a late-resolving request could hit an
already-removed community and fail with a 404 attributed to an unrelated test.
Use a sequential for-loop so the requests are awaited inside the test body.

2232 of 2628 branches covered (84.93%)

Branch coverage included in aggregate %.

71 of 75 new or added lines in 55 files covered. (94.67%)

5488 of 5979 relevant lines covered (91.79%)

72.59 hits per line