decentraland / social-service-ea / 13201066297

90%

push

github

feat: Enhance ws errors and reconnections (#48)

* feat: Enhance ws errors and reconnections

* test: Enhance and adapt transport tests

* test: Ws Handler initial tests

* feat: Avoid heartbeat mechanism and try with proper cleanup

* fix: Some fixes and error handlings

* feat: Implement a ws pool to enhance multiple connections

* test: Adapt tests + new component tests

* fix: Improve times to init and close sockets and connections

* feat: Make the pool more robust

* feat: Use redis for the connections pool

* test: Fix tests

* chore: Uninstall async mutex

* Update src/adapters/ws-pool.ts

Co-authored-by: Alejo Thomas Ortega <alejothomasortega@gmail.com>
Signed-off-by: Kevin Szuchet <31735779+kevinszuchet@users.noreply.github.com>

* Update src/controllers/handlers/ws-handler.ts

Co-authored-by: Alejo Thomas Ortega <alejothomasortega@gmail.com>
Signed-off-by: Kevin Szuchet <31735779+kevinszuchet@users.noreply.github.com>

* feat: Remove concurrent connections limit

* chore: Remove noise from the logs

* chore: Remove unnecessary metric

* refactor: Rename env var and const

* feat: Send notifications with retry (#54)

* feat: Send notifications with retryu

* test: Adapt tests to new behavior

* refactor: Avoid propagating error after retries

* fix: Use the proper profile image url

* fix: Connectivity status streaming (#52)

* fix: Updates were not being emitted

* fix: Avoid spamming connectivity status updates (#53)

* fix: Avoid spamming connectivity status updates

* test: Add tests

---------

Signed-off-by: Kevin Szuchet <31735779+kevinszuchet@users.noreply.github.com>

* refactor: Change order of if/else for simplicity

* refactor: Rename clientId -> wsConnectionId

* chore: Remove comments

* test: Add missing test case

---------

Signed-off-by: Kevin Szuchet <31735779+kevinszuchet@users.noreply.github.com>
Co-authored-by: Alejo Thomas Ortega <alejothomasortega@gmail.com>

266 of 322 branches covered (82.61%)

Branch coverage included in aggregate %.

217 of 243 new or added lines in 10 files covered. (89.3%)

852 of 995 relevant lines covered (85.63%)

9.83 hits per line