fix(instrumentation-user-insteraction)!: make getZoneWithPrototype pr…

…ivate
open-telemetry · Dec 5, 2024 · b3dda26 · b3dda26
1 parent 99e83f9
commit b3dda26
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/plugins/web/opentelemetry-instrumentation-user-interaction/src/instrumentation.ts b/plugins/web/opentelemetry-instrumentation-user-interaction/src/instrumentation.ts
@@ -575,7 +575,7 @@ export class UserInteractionInstrumentation extends InstrumentationBase<UserInte
    * implements enable function
    */
   override enable() {
-    const ZoneWithPrototype = this.getZoneWithPrototype();
+    const ZoneWithPrototype = this._getZoneWithPrototype();
     this._diag.debug(
       'applying patch to',
       this.moduleName,
@@ -645,7 +645,7 @@ export class UserInteractionInstrumentation extends InstrumentationBase<UserInte
    * implements unpatch function
    */
   override disable() {
-    const ZoneWithPrototype = this.getZoneWithPrototype();
+    const ZoneWithPrototype = this._getZoneWithPrototype();
     this._diag.debug(
       'removing patch from',
       this.moduleName,
@@ -680,7 +680,7 @@ export class UserInteractionInstrumentation extends InstrumentationBase<UserInte
   /**
    * returns Zone
    */
-  getZoneWithPrototype(): ZoneTypeWithPrototype | undefined {
+  private _getZoneWithPrototype(): ZoneTypeWithPrototype | undefined {
     const _window: WindowWithZone = window as unknown as WindowWithZone;
     return _window.Zone;
   }

diff --git a/...ns/web/opentelemetry-instrumentation-user-interaction/test/userInteraction.nozone.test.ts b/...ns/web/opentelemetry-instrumentation-user-interaction/test/userInteraction.nozone.test.ts
@@ -58,7 +58,9 @@ describe('UserInteractionInstrumentation', () => {
       });
 
       sandbox
-        .stub(userInteractionInstrumentation, 'getZoneWithPrototype')
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore this is private, but it exists
+        .stub(userInteractionInstrumentation, '_getZoneWithPrototype')
         .callsFake(() => {
           return false as any;
         });