chore: resolved review comments

Author: aryamohanan

currencies.json

@@ -186,6 +186,17 @@
     "note": "",
     "core": false
   },
+  {
+    "name": "@redis/client",
+    "policy": "45-days",
+    "lastSupportedVersion": "",
+    "latestVersion": "",
+    "cloudNative": false,
+    "isBeta": false,
+    "ignoreUpdates": false,
+    "note": "",
+    "core": false
+  },
   {
     "name": "amqplib",
     "policy": "45-days",

package-lock.json

@@ -111,6 +111,7 @@
     "@opentelemetry/instrumentation": "0.50.0",
     "@opentelemetry/sdk-node": "^0.49.1",
     "@opentelemetry/sdk-trace-base": "^1.22.0",
+    "@redis/client": "^5.0.1",
     "@redis/client-v4": "npm:@redis/[email protected]",
     "@types/chai": "^4.3.0",
     "@types/detect-libc": "^1.0.0",

package.json

@@ -97,10 +97,7 @@ const legacyVersion = 'v3';
                 before(async () => {
                   controls = new ProcessControls({
                     useGlobalAgent: true,
-                    appPath:
-                      redisVersion === legacyVersion
-                        ? path.join(__dirname, 'legacyApp.js')
-                        : path.join(__dirname, 'app.js'),
+                    appPath: path.join(__dirname, 'app.js'),
                     env: {
                       REDIS_VERSION: redisVersion,
                       REDIS_PKG: redisPkg,
@@ -115,13 +112,6 @@ const legacyVersion = 'v3';
                   await agentControls.clearReceivedTraceData();
                 });
 
-                before(async () => {
-                  await controls.sendRequest({
-                    method: 'POST',
-                    path: '/clearkeys'
-                  });
-                });
-
                 after(async () => {
                   await controls.stop();
                 });

packages/collector/test/tracing/database/redis/test.js

@@ -179,7 +179,7 @@ function instrument(redis) {
               return creatPoolWrap(poolDescriptor.get.call(this));
             }
           });
-        } else if (typeof poolDescriptor.value === 'function') {
+        } else {
           shimmer.wrap(redis, 'createClientPool', creatPoolWrap);
         }
       }