fix: resolve replay mode errors in Redis and Firestore no-op request handlers (#138)

Author: sohankshirsagar

src/instrumentation/libraries/firestore/Instrumentation.ts

@@ -699,11 +699,11 @@ export class FirestoreInstrumentation extends TdInstrumentationBase {
         if (self.mode === TuskDriftMode.REPLAY) {
           return handleReplayMode({
             noOpRequestHandler: () => {
-              if (!this.originalCollectionDocFn) {
+              if (!self.originalCollectionDocFn) {
                 logger.error(`[FirestoreInstrumentation] Original doc function not available`);
                 return Promise.reject(new Error("Original doc function not available"));
               }
-              return this.originalCollectionDocFn.call(this, "");
+              return self.originalCollectionDocFn.call(this);
             },
             isServerRequest: false,
             replayModeHandler: () => {

src/instrumentation/libraries/redis/Instrumentation.ts

@@ -215,7 +215,11 @@ export class RedisInstrumentation extends TdInstrumentationBase {
 
           return handleReplayMode({
             noOpRequestHandler: () => {
-              return undefined;
+              const upperCmd = commandName.toUpperCase();
+              if (upperCmd === "EVAL" || upperCmd === "EVALSHA") {
+                return [];
+              }
+              return null;
             },
             isServerRequest: false,
             replayModeHandler: () => {
@@ -418,7 +422,13 @@ export class RedisInstrumentation extends TdInstrumentationBase {
           const stackTrace = captureStackTrace(["RedisInstrumentation"]);
 
           return handleReplayMode({
-            noOpRequestHandler: () => undefined,
+            noOpRequestHandler: () => {
+              const upperCmd = commandName.toUpperCase();
+              if (upperCmd === "EVAL" || upperCmd === "EVALSHA") {
+                return [];
+              }
+              return null;
+            },
             isServerRequest: false,
             replayModeHandler: () => {
               return SpanUtils.createAndExecuteSpan(