feat(contractor-onboarding): implement employment update capability for contractors (#703)

Author: gabrielseco

src/flows/ContractorOnboarding/hooks.tsx

@@ -32,6 +32,7 @@ import {
   useCreateEmployment,
   useEmployment,
   useJSONSchemaForm,
+  useUpdateEmployment,
 } from '@/src/flows/Onboarding/api';
 import {
   disabledInviteButtonEmploymentStatus,
@@ -144,8 +145,15 @@ export const useContractorOnboarding = ({
   }, [employmentStatus]);
 
   const createEmploymentMutation = useCreateEmployment();
+  const updateEmploymentMutation = useUpdateEmployment(
+    internalCountryCode as string,
+    options,
+  );
   const createContractorContractDocumentMutation =
     useCreateContractorContractDocument();
+  const { mutateAsync: updateEmploymentMutationAsync } = mutationToPromise(
+    updateEmploymentMutation,
+  );
   const signContractDocumentMutation = useSignContractDocument();
   const manageContractorSubscriptionMutation =
     usePostManageContractorSubscriptions();
@@ -655,10 +663,9 @@ export const useContractorOnboarding = ({
             throw error;
           }
         } else if (internalEmploymentId) {
-          // TODO: Provisional it seems you cannot update a contractor employment
-          // TODO: we'll need to check later if the provisional start date gets updated for the statement of work
-          return Promise.resolve({
-            data: { employmentId: internalEmploymentId },
+          return updateEmploymentMutationAsync({
+            employmentId: internalEmploymentId,
+            basic_information: parsedValues,
           });
         }
 

src/flows/ContractorOnboarding/tests/ContractorOnboarding.test.tsx

@@ -34,6 +34,7 @@ import {
   fillSignature,
   generateUniqueEmploymentId,
 } from '@/src/flows/ContractorOnboarding/tests/helpers';
+import { employmentUpdatedResponse } from '@/src/flows/Onboarding/tests/fixtures';
 
 const mockOnSubmit = vi.fn();
 const mockOnSuccess = vi.fn();
@@ -341,6 +342,9 @@ describe('ContractorOnboardingFlow', () => {
       http.post('*/v1/employments/*/invite', () => {
         return HttpResponse.json(inviteResponse);
       }),
+      http.patch('*/v1/employments/*', async () => {
+        return HttpResponse.json(employmentUpdatedResponse);
+      }),
     );
   });
 
@@ -608,7 +612,7 @@ describe('ContractorOnboardingFlow', () => {
     await screen.findByText(/Step: Pricing Plan/i);
 
     // Verify PATCH was NOT called (contractors can't be updated)
-    expect(patchSpy).not.toHaveBeenCalled();
+    expect(patchSpy).toHaveBeenCalled();
   });
 
   it('should create contract document when submitting contract details', async () => {