test(instrumentation-redis-4): fix tests with [email protected] and earlier (#2914)

In [email protected] and earlier versions of redis v4, client.disconnect()
will throw if the connect failed. That broke the "with empty string for
client URL, ..." test case, at least on macOS:

    npm run test:docker:run
    RUN_REDIS_TESTS=1 npm t

I cannot explain why this is not failing in CI. Perhaps something
platform specific? This is related to socket handling in the redis
client. I believe the relevant change in node-redis was:
redis/node-redis#2295
which was part of `@redis/[email protected]` which was included in
`[email protected]`.

Author: trentm

plugins/node/opentelemetry-instrumentation-redis-4/test/redis.test.ts

@@ -314,7 +314,12 @@ describe('redis@^4.0.0', () => {
         // Ignore. If the test Redis is not at the default port we expect this
         // to error.
       }
-      await newClient.disconnect();
+      try {
+        await newClient.disconnect();
+      } catch (_disconnectErr) {
+        // Ignore. In [email protected] and earlier this disconnect throws
+        // "The client is closed" if the connect failed.
+      }
 
       const [span] = getTestSpans();
       assert.strictEqual(span.name, 'redis-connect');