fix: type for identifyHotjar userId arg

the-bugging · Jan 12, 2024 · f1da271 · f1da271
1 parent 82a3c83
commit f1da271
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/__tests__/index.test.ts b/src/__tests__/index.test.ts
@@ -68,6 +68,20 @@ describe('Tests useHotjar', () => {
     });
   });
 
+  it('should identifyHotjar for De-identified users', () => {
+    const { result } = renderHook(() => useHotjar());
+    const identifyHotjarSpy = jest.spyOn(result.current, 'identifyHotjar');
+    const { identifyHotjar } = result.current;
+
+    identifyHotjar(null, {
+      ab_test: 'variant-A',
+    });
+
+    expect(identifyHotjarSpy).toHaveBeenCalledWith(null, {
+      ab_test: 'variant-A',
+    });
+  });
+
   it('should stateChange with new relative path', () => {
     const { result } = renderHook(() => useHotjar());
     const stateChangeSpy = jest.spyOn(result.current, 'stateChange');

diff --git a/src/types.ts b/src/types.ts
@@ -12,7 +12,7 @@ export interface IUseHotjar {
     logCallback?: (...data: unknown[]) => void
   ) => boolean;
   identifyHotjar: (
-    userId: string,
+    userId: string | null,
     userInfo: TUserInfo,
     logCallback?: (...data: unknown[]) => void
   ) => boolean;