fix: correct test bugs in ShopperExperienceV2

- Fix registry test to check for correct method names (registerComponent, registerImporter)
- Simplify usePageDesignerParams test by mocking useConfig instead of full provider setup

Author: mjuraschik

packages/commerce-sdk-react/src/components/ShopperExperienceV2/registry.test.ts

@@ -41,7 +41,8 @@ describe('registry', () => {
         test('registry has expected methods', () => {
             // Cast to access internal methods for testing
             const registryInstance = registry as unknown as Record<string, unknown>
-            expect(typeof registryInstance.register).toBe('function')
+            expect(typeof registryInstance.registerComponent).toBe('function')
+            expect(typeof registryInstance.registerImporter).toBe('function')
             expect(typeof registryInstance.getComponent).toBe('function')
             expect(typeof registryInstance.getFallback).toBe('function')
             expect(typeof registryInstance.preload).toBe('function')

packages/commerce-sdk-react/src/hooks/ShopperExperience/usePageDesignerParams.test.tsx

@@ -5,70 +5,38 @@
  * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/BSD-3-Clause
  */
 import {renderHook} from '@testing-library/react'
-import React from 'react'
 import {usePageDesignerParams} from './usePageDesignerParams'
-import CommerceApiProvider from '../../provider'
-import {QueryClient, QueryClientProvider} from '@tanstack/react-query'
+import useConfig from '../useConfig'
 
-const DEFAULT_CONFIG = {
-    proxy: 'http://localhost:8888/mobify/proxy/api',
-    redirectURI: 'http://localhost:8888/callback',
-    clientId: '12345678-1234-1234-1234-123412341234',
-    organizationId: 'f_ecom_zzrmy_orgf_001',
-    shortCode: '12345678',
-    siteId: 'RefArchGlobal',
-    locale: 'en-US',
-    currency: 'USD'
-}
+jest.mock('../useConfig')
+const mockUseConfig = useConfig as jest.MockedFunction<typeof useConfig>
 
-const createWrapper = (pageDesignerParams?: {
-    mode?: 'edit' | 'preview'
-    pdToken?: string
-    pageId?: string
-}) => {
-    const queryClient = new QueryClient({
-        defaultOptions: {queries: {retry: false}, mutations: {retry: false}}
+describe('usePageDesignerParams', () => {
+    beforeEach(() => {
+        jest.clearAllMocks()
     })
-    const Wrapper = ({children}: {children: React.ReactNode}) => (
-        <QueryClientProvider client={queryClient}>
-            <CommerceApiProvider {...DEFAULT_CONFIG} pageDesignerParams={pageDesignerParams}>
-                {children}
-            </CommerceApiProvider>
-        </QueryClientProvider>
-    )
-    return Wrapper
-}
 
-describe('usePageDesignerParams', () => {
     test('returns empty object when no pageDesignerParams provided', () => {
-        const {result} = renderHook(() => usePageDesignerParams(), {
-            wrapper: createWrapper()
-        })
-
+        mockUseConfig.mockReturnValue({} as any)
+        const {result} = renderHook(() => usePageDesignerParams())
         expect(result.current).toEqual({})
     })
 
     test('returns mode when provided', () => {
-        const {result} = renderHook(() => usePageDesignerParams(), {
-            wrapper: createWrapper({mode: 'edit'})
-        })
-
+        mockUseConfig.mockReturnValue({pageDesignerParams: {mode: 'edit'}} as any)
+        const {result} = renderHook(() => usePageDesignerParams())
         expect(result.current.mode).toBe('edit')
     })
 
     test('returns pdToken when provided', () => {
-        const {result} = renderHook(() => usePageDesignerParams(), {
-            wrapper: createWrapper({pdToken: 'test-token'})
-        })
-
+        mockUseConfig.mockReturnValue({pageDesignerParams: {pdToken: 'test-token'}} as any)
+        const {result} = renderHook(() => usePageDesignerParams())
         expect(result.current.pdToken).toBe('test-token')
     })
 
     test('returns pageId when provided', () => {
-        const {result} = renderHook(() => usePageDesignerParams(), {
-            wrapper: createWrapper({pageId: 'test-page-id'})
-        })
-
+        mockUseConfig.mockReturnValue({pageDesignerParams: {pageId: 'test-page-id'}} as any)
+        const {result} = renderHook(() => usePageDesignerParams())
         expect(result.current.pageId).toBe('test-page-id')
     })
 
@@ -78,19 +46,16 @@ describe('usePageDesignerParams', () => {
             pdToken: 'my-token',
             pageId: 'my-page'
         }
-
-        const {result} = renderHook(() => usePageDesignerParams(), {
-            wrapper: createWrapper(pageDesignerParams)
-        })
-
+        mockUseConfig.mockReturnValue({pageDesignerParams} as any)
+        const {result} = renderHook(() => usePageDesignerParams())
         expect(result.current).toEqual(pageDesignerParams)
     })
 
     test('returns partial params when partially provided', () => {
-        const {result} = renderHook(() => usePageDesignerParams(), {
-            wrapper: createWrapper({mode: 'edit', pdToken: 'token'})
-        })
-
+        mockUseConfig.mockReturnValue({
+            pageDesignerParams: {mode: 'edit', pdToken: 'token'}
+        } as any)
+        const {result} = renderHook(() => usePageDesignerParams())
         expect(result.current.mode).toBe('edit')
         expect(result.current.pdToken).toBe('token')
         expect(result.current.pageId).toBeUndefined()