fix tsc errors with test files (#239)

Author: zachkrall

.github/workflows/pr-check.yml

@@ -47,16 +47,16 @@ jobs:
           NODE_ENV: development
         run: pnpm install
 
+      # Check that packages can build without errors
+      - name: Builds without errors
+        env:
+          NODE_ENV: production
+        run: pnpm build
+
       # Check for linting errors
       - name: Check for linting errors
         run: pnpm lint
 
       # Check that project unit tests are successful
       - name: Run unit tests
         run: pnpm test
-
-      # Check that packages can build without errors
-      - name: Builds without errors
-        env:
-          NODE_ENV: production
-        run: pnpm build

justfile

@@ -0,0 +1,3 @@
+# run the CI workflow locally
+local-ci:
+    act -W '.github/workflows/pr-check.yml' --container-architecture linux/amd64 -s GITHUB_TOKEN="$(gh auth token)"

packages/react/package.json

@@ -19,7 +19,7 @@
     "test-watch": "vitest --watch",
     "build": "tsup",
     "dev": "tsup --watch",
-    "lint": "eslint ./src",
+    "lint": "eslint ./src && tsc --noEmit -p .",
     "pack": "npm pack"
   },
   "keywords": [],

packages/react/src/lib/VoiceProvider.test.tsx

@@ -9,7 +9,11 @@ describe('useVoice', () => {
   it('should in', async () => {
     const hook = renderHook(() => useVoice(), {
       wrapper: ({ children }) => {
-        return <VoiceProvider apiKey={'abc'}>{children}</VoiceProvider>;
+        return (
+          <VoiceProvider auth={{ type: 'apiKey', value: 'abc' }}>
+            {children}
+          </VoiceProvider>
+        );
       },
     });
 

packages/react/src/lib/useMessages.test.ts

@@ -69,7 +69,9 @@ describe('useMessages hook', () => {
 
   beforeEach(() => {
     vi.clearAllMocks();
-    hook = renderHook(() => useMessages({ sendMessageToParent }));
+    hook = renderHook(() =>
+      useMessages({ messageHistoryLimit: 100, sendMessageToParent }),
+    );
     userMessage = {
       type: 'user_message',
       message: {
@@ -121,53 +123,82 @@ describe('useMessages hook', () => {
 
   it('should add user messages to `messages` immediately', () => {
     act(() => {
-      hook.result.current.onMessage(userMessage);
+      hook.result.current.onMessage({
+        ...userMessage,
+        receivedAt: new Date(),
+      });
     });
 
-    expect(hook.result.current.lastUserMessage).toEqual(userMessage);
-    expect(hook.result.current.messages).toContainEqual(userMessage);
+    expect(hook.result.current.lastUserMessage).toMatchObject(
+      expect.objectContaining(userMessage),
+    );
+    expect(hook.result.current.messages).toMatchObject(
+      expect.arrayContaining([expect.objectContaining(userMessage)]),
+    );
   });
 
   it('should add voice messages to the voice message map', () => {
     act(() => {
-      hook.result.current.onMessage(agentMessage);
+      hook.result.current.onMessage({
+        ...agentMessage,
+        receivedAt: new Date(),
+      });
     });
 
     expect(hook.result.current.lastVoiceMessage).toBeNull();
-    expect(hook.result.current.messages).not.toContainEqual(agentMessage);
+    expect(hook.result.current.messages).toMatchObject([]);
   });
 
   it('should expose the voice message after the associated audio clip is played', () => {
     // add the message
     act(() => {
-      hook.result.current.onMessage(agentMessage);
+      hook.result.current.onMessage({
+        ...agentMessage,
+        receivedAt: new Date(),
+      });
     });
 
     // simulate playing audio
     act(() => {
       hook.result.current.onPlayAudio(agentMessage.id ?? '');
     });
 
-    expect(hook.result.current.lastVoiceMessage).toEqual(agentMessage);
-    expect(hook.result.current.messages).toContainEqual(agentMessage);
+    expect(hook.result.current.lastVoiceMessage).toMatchObject(
+      expect.objectContaining(agentMessage),
+    );
+    expect(hook.result.current.messages).toMatchObject(
+      expect.arrayContaining([expect.objectContaining(agentMessage)]),
+    );
   });
 
   it('should expose the voice message after the associated audio clip is played', () => {
     act(() => {
-      hook.result.current.onMessage(agentMessage); // First, add the message
+      // First, add the message
+      hook.result.current.onMessage({
+        ...agentMessage,
+        receivedAt: new Date(),
+      });
     });
 
     act(() => {
       hook.result.current.onPlayAudio(agentMessage.id ?? '');
     });
 
-    expect(hook.result.current.lastVoiceMessage).toEqual(agentMessage);
-    expect(hook.result.current.messages).toContainEqual(agentMessage);
+    expect(hook.result.current.lastVoiceMessage).toMatchObject(
+      expect.objectContaining(agentMessage),
+    );
+    expect(hook.result.current.messages).toMatchObject(
+      expect.arrayContaining([expect.objectContaining(agentMessage)]),
+    );
   });
 
   it('should only add voice messages once', () => {
     act(() => {
-      hook.result.current.onMessage(agentMessage); // First, add the message
+      // First, add the message
+      hook.result.current.onMessage({
+        ...agentMessage,
+        receivedAt: new Date(),
+      });
     });
 
     act(() => {
@@ -186,14 +217,19 @@ describe('useMessages hook', () => {
 
   it('should call the sendMessageToParent callback when the associated audio clip plays', () => {
     act(() => {
-      hook.result.current.onMessage(agentMessage);
+      hook.result.current.onMessage({
+        ...agentMessage,
+        receivedAt: new Date(),
+      });
     });
 
     act(() => {
       hook.result.current.onPlayAudio(agentMessage.id ?? '');
     });
 
-    expect(sendMessageToParent).toHaveBeenCalledWith(agentMessage);
+    expect(sendMessageToParent).toHaveBeenCalledWith(
+      expect.objectContaining(agentMessage),
+    );
   });
 
   it('should clear all messages and states on disconnect', () => {

packages/react/src/lib/useToolStatus.test.ts

@@ -1,5 +1,5 @@
 import { act, renderHook } from '@testing-library/react';
-import type { Hume } from 'Hume';
+import type { Hume } from 'hume';
 import { describe, expect, it } from 'vitest';
 
 import { useToolStatus } from './useToolStatus';