fix: LiveQueryServer crashes using cacheAdapter on disconnect from Redis 4 server

[Chilldev]

Pull Request

• Report security issues confidentially.
• Any contribution is under this license.
• Link this pull request to an issue.

Issue

Closes: #9432

Approach

Add empty handlers to redis client error, connect, reconnecting, ready events.

[parse-github-assistant]

Thanks for opening this pull request!

[mtrezza]

I guess you have seen it already - the CI fails for these tests:

1) RedisPubSub can create publisher
  - TypeError: Cannot read properties of undefined (reading 'on')

2) RedisPubSub can create subscriber
  - TypeError: Cannot read properties of undefined (reading 'on')

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.52%. Comparing base (ef68fb1) to head (5a507e7).
Report is 2 commits behind head on alpha.

Additional details and impacted files

@@           Coverage Diff           @@
##            alpha    #9616   +/-   ##
=======================================
  Coverage   93.52%   93.52%           
=======================================
  Files         186      186           
  Lines       14816    14831   +15     
=======================================
+ Hits        13856    13871   +15     
  Misses        960      960           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mtrezza]

Looks good; just to confirm, this is not a breaking change, right? For example, someone using Redis 3 can upgrade without any code changes.

[Chilldev]

I borrowed the code from RedisCacheAdapter. So we should be fine.

[parseplatformorg]

🎉 This change has been released in version 8.0.0-alpha.12