@cccsaurora/howler-ui 3.1.0-dev.1679 → 3.1.0-dev.1690

package/components/elements/display/handlebars/helpers.test.js

@@ -71,13 +71,21 @@ describe('handlebars helpers', () => {
         expect(String(helperMap.entries.callback?.(null, options))).toBe('Invalid Object.');
         expect(helperMap.replace.callback?.('alpha beta', 'beta', 'gamma', options)).toBe('alpha gamma');
         expect(() => helperMap.replace.callback?.('alpha', undefined, 'x', options)).toThrow('Replace expects three arguments');
-        global.fetch = vi.fn(() => Promise.resolve({
-            json: () => Promise.resolve({ nested: { value: 'cached' } })
+        const mockFetch = vi.fn().mockResolvedValue(new Response(JSON.stringify({ nested: { value: 'cached' } }), {
+            headers: { 'Content-Type': 'application/json' }
         }));
-        await expect(helperMap.fetch.callback?.('https://example.com/data', 'nested.value', options)).resolves.toBe('cached');
-        await expect(helperMap.fetch.callback?.('https://example.com/data', 'nested.value', options)).resolves.toBe('cached');
-        await expect(helperMap.fetch.callback?.('https://example.com/data', 'missing.value', options)).resolves.toBeUndefined();
-        expect(global.fetch).toHaveBeenCalledTimes(1);
+        const originalFetch = globalThis.fetch;
+        try {
+            globalThis.fetch = mockFetch;
+            await expect(helperMap.fetch.callback?.('https://example.com/data', 'nested.value', options)).resolves.toBe('cached');
+            await expect(helperMap.fetch.callback?.('https://example.com/data', 'nested.value', options)).resolves.toBe('cached');
+            await expect(helperMap.fetch.callback?.('https://example.com/data', 'missing.value', options)).resolves.toBeUndefined();
+            expect(mockFetch).toHaveBeenCalledTimes(1);
+        }
+        finally {
+            globalThis.fetch = originalFetch;
+        }
+        expect(globalThis.fetch).toBe(originalFetch);
     });
     it('renders component-backed helpers', () => {
         mockExecuteFunction.mockReturnValue([]);

package/components/routes/analytics/AnalyticDetails.test.js

@@ -140,7 +140,7 @@ describe('AnalyticDetails', () => {
         expect(screen.getByText('owner@example.com')).toBeInTheDocument();
         expect(screen.getByText('avatar:user-2')).toBeInTheDocument();
         expect(screen.getByText('overview-tab')).toBeInTheDocument();
-        expect(mockSearchUsers).toHaveBeenCalledWith('uname:"owner1"');
+        await waitFor(() => expect(mockSearchUsers).toHaveBeenCalledWith('uname:"owner1"'));
         fireEvent.click(screen.getByText('switch-comments'));
         await waitFor(() => expect(screen.getByText('comments-tab')).toBeInTheDocument());
         fireEvent.click(screen.getByText('change-filter'));

package/package.json

@@ -97,7 +97,7 @@
     "url": "https://github.com/CybercentreCanada/howler"
   },
   "type": "module",
-  "version": "3.1.0-dev.1679",
+  "version": "3.1.0-dev.1690",
   "exports": {
     "./i18n": "./i18n.js",
     "./index.css": "./index.css",

package/tests/mocks.d.ts

@@ -1,3 +1,17 @@
+import type { useSearchParams } from 'react-router';
+import { type Mock } from 'vitest';
+type SearchParamsSetter = ReturnType<typeof useSearchParams>[1];
+type ReactRouterMock = {
+    mockSearchParams: URLSearchParams;
+    mockSetParams: Mock<SearchParamsSetter>;
+    mockLocation: {
+        pathname: string;
+        search: string;
+    };
+    mockParams: {
+        id: string | undefined;
+    };
+};
 /**
  * Sets up a mock for use-context-selector that uses React's native context
  * This allows tests to use context providers without the full use-context-selector implementation
@@ -7,17 +21,7 @@ export declare const setupContextSelectorMock: () => void;
  * Sets up a mock for react-router with common defaults
  * @param options - Override specific router behavior
  */
-export declare const setupReactRouterMock: () => {
-    mockSearchParams: URLSearchParams;
-    mockSetParams: import("vitest").Mock<(...args: any[]) => any>;
-    mockLocation: {
-        pathname: string;
-        search: string;
-    };
-    mockParams: {
-        id: undefined;
-    };
-};
+export declare const setupReactRouterMock: () => ReactRouterMock;
 /**
  * Sets up a mock localStorage instance
  */
@@ -26,3 +30,4 @@ export declare const setupLocalStorageMock: () => Storage;
  * Sets up a mock sessionStorage instance
  */
 export declare const setupSessionStorageMock: () => Storage;
+export {};