Remove unneeded legacy test around reconnect.

Now that we're testing this behavior in the client,
we don't need this mega-gross, complicated, flaky test.
Throw our own ConnectionError.

Instead of bubbling up the Redis error, we want to throw our own.

Also wrote documentation that is better for the reconnect method.