fix: correct typo in telemetryEvent

Author: felixfbecker

src/HoverOverlay.tsx

@@ -65,7 +65,7 @@ export interface HoverOverlayProps {
     /** Called when the close button is clicked */
     onCloseButtonClick?: (event: React.MouseEvent<HTMLElement>) => void
 
-    logTelemetryEvent: (event: string) => void
+    logTelemetryEvent: (event: string, data?: any) => void
 }
 
 /** Returns true if the input is successful jump URL result */

src/hoverifier.test.ts

@@ -4,6 +4,7 @@ import { distinctUntilChanged, filter, map } from 'rxjs/operators'
 import { TestScheduler } from 'rxjs/testing'
 import { Position } from 'vscode-languageserver-types'
 
+import { noop } from 'lodash'
 import { propertyIsDefined } from './helpers'
 import { createHoverifier, LOADER_DELAY, TOOLTIP_DISPLAY_DELAY } from './hoverifier'
 import { HoverOverlayProps } from './HoverOverlay'
@@ -12,8 +13,6 @@ import { createHoverMerged, createStubHoverFetcher, createStubJumpURLFetcher } f
 import { clickPosition } from './testutils/mouse'
 import { LOADING } from './types'
 
-const noop = () => undefined
-
 describe('Hoverifier', () => {
     const dom = new DOM()
     // after(dom.cleanup)
@@ -43,7 +42,7 @@ describe('Hoverifier', () => {
                     fetchHover: createStubHoverFetcher(hover, delayTime),
                     fetchJumpURL: createStubJumpURLFetcher(defURL, delayTime),
                     pushHistory: noop,
-                    logTelementryEvent: () => noop,
+                    logTelemetryEvent: noop,
                 })
 
                 const positionJumps = new Subject<{

src/hoverifier.ts

@@ -61,7 +61,7 @@ interface HoverifierOptions {
     /**
      * Called to log telemetry events
      */
-    logTelementryEvent: (eventLabel: string, eventProperties?: any) => void
+    logTelemetryEvent: (event: string, data?: any) => void
 
     fetchHover: HoverFetcher
     fetchJumpURL: JumpURLFetcher
@@ -207,7 +207,7 @@ export const createHoverifier = ({
     pushHistory,
     fetchHover,
     fetchJumpURL,
-    logTelementryEvent,
+    logTelemetryEvent,
 }: HoverifierOptions): Hoverifier => {
     // Internal state that is not exposed to the caller
     // Shared between all hoverified code views
@@ -452,7 +452,7 @@ export const createHoverifier = ({
                 filter(HoverMerged.is)
             )
             .subscribe(() => {
-                logTelementryEvent('SymbolHovered')
+                logTelemetryEvent('SymbolHovered')
             })
     )
 
@@ -536,7 +536,7 @@ export const createHoverifier = ({
     subscription.add(
         goToDefinitionClicks.subscribe(event => {
             // Telemetry
-            logTelementryEvent('GoToDefClicked')
+            logTelemetryEvent('GoToDefClicked')
 
             // If we don't have a result yet that would be jumped to by the native <a> tag...
             if (!isJumpURL(container.values.definitionURLOrError)) {