test: remove unnecessary waits from e2e tests

[Nashluffy]

Description

For all the tests I've changed, there are existing assertions that the cluster/sentinel/redis is ready, so the sleeps are unnecessary. From the looks of it, they predate the ready assertions.

Partial fix for #1104

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist

• Tests have been added/modified and all tests pass.
• Functionality/bugs have been confirmed to be unchanged or fixed.
• I have performed a self-review of my own code.
• Documentation has been updated or added where necessary.

Additional Context

It's very possible there's additional context I'm missing on why these sleeps are still in the codebase.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.03%. Comparing base (d121d86) to head (ec84176).
Report is 133 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1106      +/-   ##
==========================================
+ Coverage   35.20%   44.03%   +8.83%     
==========================================
  Files          19       21       +2     
  Lines        3213     3431     +218     
==========================================
+ Hits         1131     1511     +380     
+ Misses       2015     1832     -183     
- Partials       67       88      +21     

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

[drivebyer]

@Nashluffy Could you please rebase on the master branch