fix tests and missing refactor

src/PortingEmbed/PortingFormContainer.tsx

@@ -50,9 +50,12 @@ export function PortingFormContainer({
   useEffect(() => {
     if (!error) return
 
-    onError(new Error(error), { code: 'initializationError' })
+    onError(new Error(error), {
+      code: 'initializationError',
+      porting: subscription?.porting || undefined,
+    })
     console.error(error)
-  }, [error, onError])
+  }, [error, onError, subscription?.porting])
 
   // TODO: move onLoaded into the initial data fetching instead of an effect, so
   // it can't trigger multiple times, and will only be triggered for the initial
@@ -134,6 +137,7 @@ export function PortingFormContainer({
       )
       onError(e instanceof ApiError ? e : new Error('Something went wrong.'), {
         code: 'unexpectedError',
+        porting: subscription?.porting || undefined,
       })
     } finally {
       setSubmitting(false)

src/PortingEmbed/__tests__/PortingFormContainer.test.tsx

@@ -380,15 +380,14 @@ describe('PortingFormContainer', () => {
       )
 
       await waitFor(() => {
-        expect(onError).toBeCalledWith(
-          undefined,
-          {
+        expect(onError).toBeCalledWith(new Error('Porting is declined'), {
+          code: 'portingDeclined',
+          porting: {
             ...porting,
             status: 'declined',
             declinedCode: 'portingUserInformationMismatch',
           },
-          'portingDeclined'
-        )
+        })
       })
     })
   })

src/PortingEmbed/features/DonorProvider/DonorProviderForm.tsx

@@ -39,7 +39,10 @@ export function DonorProviderForm({
   const options = useOptionsContext()
 
   if (!serviceProviders || !serviceProviders.items.length || fetchError) {
-    options?.onError(fetchError || new Error('Service Providers not found.'))
+    options?.onError(fetchError || new Error('Service Providers not found.'), {
+      code: 'providersNotFound',
+      porting,
+    })
     return null
   }