From be6d940c90e53eec857bb65c0a07904314a59cbe Mon Sep 17 00:00:00 2001 From: Ali Sabzevari Date: Mon, 12 Jul 2021 16:07:42 +0200 Subject: [PATCH] chore(context-zone-peer-dep): fix lint warnings --- .../src/ZoneContextManager.ts | 23 ++++++++++--------- .../src/types.ts | 8 +++---- 2 files changed, 16 insertions(+), 15 deletions(-) diff --git a/packages/opentelemetry-context-zone-peer-dep/src/ZoneContextManager.ts b/packages/opentelemetry-context-zone-peer-dep/src/ZoneContextManager.ts index 9f56f46657..b55e3ca087 100644 --- a/packages/opentelemetry-context-zone-peer-dep/src/ZoneContextManager.ts +++ b/packages/opentelemetry-context-zone-peer-dep/src/ZoneContextManager.ts @@ -15,7 +15,7 @@ */ import { Context, ContextManager, ROOT_CONTEXT } from '@opentelemetry/api'; -import { Func, TargetWithEvents } from './types'; +import { TargetWithEvents } from './types'; import { isListenerObject } from './util'; /* Key name to be used to save a context reference in Zone */ @@ -53,9 +53,10 @@ export class ZoneContextManager implements ContextManager { * @param context A context (span) to be executed within target function * @param target Function to be executed within the context */ + // eslint-disable-next-line @typescript-eslint/ban-types private _bindFunction(context: Context, target: T): T { const manager = this; - const contextWrapper = function (this: any, ...args: unknown[]) { + const contextWrapper = function (this: unknown, ...args: unknown[]) { return manager.with(context, () => target.apply(this, args)); }; Object.defineProperty(contextWrapper, 'length', { @@ -134,16 +135,16 @@ export class ZoneContextManager implements ContextManager { */ private _patchAddEventListener( target: TargetWithEvents, - original: Function, + original: NonNullable, context: Context ) { const contextManager = this; return function ( - this: {}, - event: string, - listener: Func, - opts?: any + this: TargetWithEvents, + event, + listener, + opts ) { if (target.__ot_listeners === undefined) { target.__ot_listeners = {}; @@ -157,7 +158,7 @@ export class ZoneContextManager implements ContextManager { // store a weak reference of the user listener to ours listeners.set(listener, patchedListener); return original.call(this, event, patchedListener, opts); - }; + } as TargetWithEvents['addEventListener']; } /** @@ -167,9 +168,9 @@ export class ZoneContextManager implements ContextManager { */ private _patchRemoveEventListener( target: TargetWithEvents, - original: Function + original: NonNullable ) { - return function (this: {}, event: string, listener: Func) { + return function (this: TargetWithEvents, event, listener) { if ( target.__ot_listeners === undefined || target.__ot_listeners[event] === undefined @@ -180,7 +181,7 @@ export class ZoneContextManager implements ContextManager { const patchedListener = events.get(listener); events.delete(listener); return original.call(this, event, patchedListener || listener); - }; + } as TargetWithEvents['removeEventListener']; } /** diff --git a/packages/opentelemetry-context-zone-peer-dep/src/types.ts b/packages/opentelemetry-context-zone-peer-dep/src/types.ts index 4c073178c1..fbae9e3045 100644 --- a/packages/opentelemetry-context-zone-peer-dep/src/types.ts +++ b/packages/opentelemetry-context-zone-peer-dep/src/types.ts @@ -23,13 +23,13 @@ export type Func = (...args: unknown[]) => T; export interface TargetWithEvents { addEventListener?( event: string, - listener: (...args: any[]) => void, + listener: (...args: unknown[]) => void, opts?: { once: boolean } - ): any; + ): unknown; removeEventListener?( event: string, - listener: (...args: any[]) => void, + listener: (...args: unknown[]) => void, opts?: { once: boolean } - ): any; + ): unknown; __ot_listeners?: { [name: string]: WeakMap, Func> }; }