decentraland / comms-gatekeeper / 23614144453

82%

push

github

feat: kick banned users from all rooms on platform ban (#247)

* feat: kick banned users from all LiveKit rooms on platform ban

When a user is platform-banned via POST /users/:address/bans, they are
now immediately removed from all active LiveKit rooms managed by
comms-gatekeeper. Adds removeParticipantFromAllRooms to the LiveKit
adapter and wires it into the user moderation component as a
fire-and-forget call after ban creation, consistent with scene bans.

* fix: add removeParticipantFromAllRooms to livekit mock and fix test standards

- Add missing removeParticipantFromAllRooms to livekit-mock.ts, fixing
  TS2322 compile errors in scene-bans and voice-logic test suites
- Move test variables/mockRooms from describe-level const to beforeEach let
- Move mock resets to afterEach per dcl-testing standards
- Split multi-assertion it() blocks into single-assertion tests
- Add missing flushPromises() before kick assertion (fire-and-forget call)

* revert: restore yarn.lock to pre-branch state

* fix: remove redundant try/catch inside Promise.allSettled and log errors

Replace inner try/catch (which prevented allSettled from ever seeing
rejections) with inspecting settled results: log info on success and
warn on failure so network errors, auth issues and rate limits are
visible rather than silently discarded.

* fix: only log unexpected errors when removing participant from all rooms

Use Promise.all with inner try/catch per room. Silently ignore TwirpError
with code 'not_found' (participant not in room — expected when iterating
all rooms). Log a warning for any other error (network, auth, rate limit)
to preserve observability of genuine failures.

905 of 1215 branches covered (74.49%)

Branch coverage included in aggregate %.

13 of 13 new or added lines in 3 files covered. (100.0%)

2619 of 3047 relevant lines covered (85.95%)

41.26 hits per line