Test failure - "DefaultReconnectionHandler.test - invokes update on each attempt"

Failing Test(s)

• DefaultReconnectionHandler.test - invokes update on each attempt

Error Message

FAIL tests/DefaultReconnectionHandler.test.ts (23.354 s)
    ● DefaultReconnectionHandler › invokes update on each attempt
  
      expect(jest.fn()).toHaveBeenCalledTimes(expected)
  
      Expected number of calls: 6
      Received number of calls: 5
  
        90 |
        91 |     await delay(500);
      > 92 |     expect(testDisplay.update).toHaveBeenCalledTimes(maxRetries);
           |                                ^
        93 |
        94 |   })
        95 | });
  
        at Object.<anonymous> (tests/DefaultReconnectionHandler.test.ts:92:32)

Build

[pranavkm]

I’d go for it. I was the one that added the tests, and it got 0 traction from the remainder of the team. We’re really not losing any meaningful coverage removing it.

[BrennanConroy]

it.skip('invokes update on each attempt', ...)