refactor: remove code duplication on unwrapping

open-telemetry · Oct 10, 2022 · b34355d · b34355d
1 parent b9658e1
commit b34355d
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/plugins/node/instrumentation-fs/src/instrumentation.ts b/plugins/node/instrumentation-fs/src/instrumentation.ts
@@ -64,22 +64,19 @@ export default class FsInstrumentation extends InstrumentationBase<FS> {
             );
           }
           for (const fName of CALLBACK_FUNCTIONS) {
+            if (isWrapped(fs[fName])) {
+              this._unwrap(fs, fName);
+            }
             if (fName === 'exists') {
               // handling separately because of the inconsistent cb style:
               // `exists` doesn't have error as the first argument, but the result
-              if (isWrapped(fs[fName])) {
-                this._unwrap(fs, fName);
-              }
               this._wrap(
                 fs,
                 fName,
                 <any>this._patchExistsCallbackFunction.bind(this, fName)
               );
               continue;
             }
-            if (isWrapped(fs[fName])) {
-              this._unwrap(fs, fName);
-            }
             this._wrap(
               fs,
               fName,