fix tests 2

Author: quisido

.github/instructions/react.instructions.md

@@ -35,3 +35,11 @@ applyTo: "**/*.tsx"
 - Always provide stable keys in lists. Avoid using array indices as keys.
 - Avoid `setState`/`useState` updates based on stale closures. Prefer functional
   updates when derived from previous state.
+
+  ```ts
+  // Prefer:
+  setCount((oldCount: number): number => oldCount + 1)
+
+  // Over:
+  setCount(oldCount + 1)
+  ```

packages/eslint-config/.vscode/launch.json

@@ -1,10 +1,12 @@
 {
   "configurations": [
     {
+      "console": "integratedTerminal",
       "name": "Build",
       "request": "launch",
       "runtimeArgs": ["run", "build"],
       "runtimeExecutable": "npm",
+      "skipFiles": ["<node_internals>/**"],
       "type": "node",
     }
   ],

packages/fmrs/src/map-object-to-entries.test.ts

@@ -0,0 +1,12 @@
+import { describe, expect, it } from 'vitest';
+import { mapObjectToEntries } from './index.js';
+
+describe('mapObjectToEntries', (): void => {
+  it('should map an object to entries', (): void => {
+    expect(mapObjectToEntries({ bool: true, num: 123, str: 'test' })).toEqual([
+      ['bool', true],
+      ['num', 123],
+      ['str', 'test'],
+    ]);
+  });
+});

packages/fmrs/src/map-object-to-keys.test.ts

@@ -0,0 +1,12 @@
+import { describe, expect, it } from 'vitest';
+import { mapObjectToKeys } from './index.js';
+
+describe('mapObjectToKeys', (): void => {
+  it('should map an object to its keys', (): void => {
+    expect(mapObjectToKeys({ bool: true, num: 123, str: 'test' })).toEqual([
+      'bool',
+      'num',
+      'str',
+    ]);
+  });
+});

packages/fullstory-react/src/hooks/use-mock-fullstory-browser.test.ts

@@ -0,0 +1,33 @@
+import type { ApiV2 } from '@fullstory/snippet';
+import { renderHook } from '@testing-library/react';
+import { describe, expect, it, vi } from 'vitest';
+import useMockFullstoryBrowser from './use-mock-fullstory-browser.js';
+
+const TEST_FULL_STORY = vi.fn();
+
+describe('useMockFullstoryBrowser', (): void => {
+  describe('FullStory', (): void => {
+    it('should default to no-op', (): void => {
+      const { result } = renderHook(useMockFullstoryBrowser);
+
+      const { FullStory: fullStory } = result.current;
+
+      expect((): void => {
+        fullStory('init');
+      }).not.toThrow();
+    });
+
+    it('should call a provided Fullstory API', (): void => {
+      const { result } = renderHook(useMockFullstoryBrowser, {
+        initialProps: {
+          FullStory: TEST_FULL_STORY as unknown as ApiV2,
+        },
+      });
+
+      const { FullStory: fullStory } = result.current;
+      fullStory('init');
+
+      expect(TEST_FULL_STORY).toHaveBeenCalledWith('init');
+    });
+  });
+});

packages/fullstory-react/src/hooks/use-mock-fullstory-browser.ts

@@ -6,7 +6,7 @@ import useShallowMemo from 'use-shallow-memo';
 import merge from '../utils/merge.js';
 
 export default function useMockFullstoryBrowser(
-  partial: Partial<Omit<typeof fullstoryBrowser, 'default'>>,
+  partial: Partial<Omit<typeof fullstoryBrowser, 'default'>> = {},
 ): Omit<typeof fullstoryBrowser, 'default'> {
   // States
   const [initialized, setInitialized] = useState(false);