refactor: retain original formatName functionility 4 other places

There were some screens that were not using the new customised formatName() so the refactor to formatName broke thier e2e tests, the temporary solution is to add a new function formatNameWithFirstNameFirst() which uses the old logic until the screen are updated opencrvs/opencrvs-core#6830
opencrvs · Nov 14, 2024 · 3be87d1 · 3be87d1
1 parent 747c1fb
commit 3be87d1
Show file tree

Hide file tree

Showing 11 changed files with 58 additions and 27 deletions.
diff --git a/e2e/helpers.ts b/e2e/helpers.ts
@@ -292,6 +292,10 @@ export const formatName = (name: PersonOrName) => {
   return joinValuesWith([name.familyName, name.firstNames])
 }
 
+export const formatNameWithFirstNameFirst = (name: PersonOrName) => {
+  return joinValuesWith([name.firstNames, name.familyName])
+}
+
 export const drawSignature = async (page: Page) => {
   const canvas = page.locator('#informantSignature_modal canvas')
   const rect = await canvas.boundingBox()

diff --git a/e2e/testcases/correction-birth/correct-birth-record-1.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-1.spec.ts
@@ -6,6 +6,7 @@ import {
   formatDateTo_ddMMMMyyyy,
   formatDateTo_yyyyMMdd,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getLocationNameFromFhirId,
   getToken,
@@ -712,8 +713,8 @@ test.describe('1. Correct record - 1', () => {
         await expect(
           page.getByText(
             'Full name (Child)' +
-              formatName(declaration.child.name[0]) +
-              formatName(updatedChildDetails)
+              formatNameWithFirstNameFirst(declaration.child.name[0]) +
+              formatNameWithFirstNameFirst(updatedChildDetails)
           )
         ).toBeVisible()
 
@@ -768,7 +769,9 @@ test.describe('1. Correct record - 1', () => {
         ).toBeVisible()
 
         await expect(
-          page.getByText(formatName(declaration.mother.name[0]))
+          page.getByText(
+            formatNameWithFirstNameFirst(declaration.mother.name[0])
+          )
         ).toBeVisible()
         await expect(page.getByText('Verified')).toBeVisible()
         await expect(
@@ -802,7 +805,9 @@ test.describe('1. Correct record - 1', () => {
         await page.getByRole('button', { name: 'Sent for approval' }).click()
 
         await expect(
-          page.getByText(formatName(declaration.child.name[0])).first()
+          page
+            .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
+            .first()
         ).toBeVisible()
       })
     })
@@ -847,7 +852,8 @@ test.describe('1. Correct record - 1', () => {
 
         await expect(
           page.getByText(
-            'Requested by' + formatName(declaration.mother.name[0])
+            'Requested by' +
+              formatNameWithFirstNameFirst(declaration.mother.name[0])
           )
         ).toBeVisible()
 

diff --git a/e2e/testcases/correction-birth/correct-birth-record-2.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-2.spec.ts
@@ -7,6 +7,7 @@ import {
   formatDateTo_ddMMMMyyyy,
   formatDateTo_yyyyMMdd,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -806,8 +807,8 @@ test.describe.serial('Correct record - 2', () => {
     await expect(
       page.getByText(
         'Full name (informant)' +
-          formatName(declaration.informant.name[0]) +
-          formatName(updatedInformantDetails)
+          formatNameWithFirstNameFirst(declaration.informant.name[0]) +
+          formatNameWithFirstNameFirst(updatedInformantDetails)
       )
     ).toBeVisible()
 
@@ -868,7 +869,7 @@ test.describe.serial('Correct record - 2', () => {
     ).toBeVisible()
 
     await expect(
-      page.getByText(formatName(declaration.father.name[0]))
+      page.getByText(formatNameWithFirstNameFirst(declaration.father.name[0]))
     ).toBeVisible()
     await expect(page.getByText('Identity does not match')).toBeVisible()
     await expect(

diff --git a/e2e/testcases/correction-birth/correct-birth-record-3.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-3.spec.ts
@@ -6,6 +6,7 @@ import {
   expectOutboxToBeEmpty,
   formatDateTo_ddMMMMyyyy,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -145,13 +146,15 @@ test.describe.serial(' Correct record - 3', () => {
       await expectOutboxToBeEmpty(page)
       await page.getByRole('button', { name: 'Ready to issue' }).click()
       await expect(
-        page.getByText(formatName(declaration.child.name[0])).first()
+        page
+          .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
+          .first()
       ).toBeVisible()
     })
 
     test('3.1.3 Record audit', async () => {
       await page
-        .getByText(formatName(declaration.child.name[0]))
+        .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
         .first()
         .click()
       await assignRecord(page)

diff --git a/e2e/testcases/correction-birth/correct-birth-record-4.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-4.spec.ts
@@ -7,6 +7,7 @@ import {
   formatDateTo_ddMMMMyyyy,
   formatDateTo_yyyyMMdd,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -674,8 +675,8 @@ test.describe.serial(' Correct record - 4', () => {
     await expect(
       page.getByText(
         'Full name (father)' +
-          formatName(declaration.father.name[0]) +
-          formatName(updatedFatherDetails)
+          formatNameWithFirstNameFirst(declaration.father.name[0]) +
+          formatNameWithFirstNameFirst(updatedFatherDetails)
       )
     ).toBeVisible()
 
@@ -765,7 +766,9 @@ test.describe.serial(' Correct record - 4', () => {
      */
 
     await expect(
-      page.getByText(formatName(declaration.child.name[0])).first()
+      page
+        .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
+        .first()
     ).toBeVisible()
   })
   test('4.8 Validate history in record audit', async () => {

diff --git a/e2e/testcases/correction-birth/correct-birth-record-5.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-5.spec.ts
@@ -6,6 +6,7 @@ import {
   expectOutboxToBeEmpty,
   formatDateTo_ddMMMMyyyy,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -565,8 +566,8 @@ test.describe.serial(' Correct record - 5', () => {
     await expect(
       page.getByText(
         'Full name (Child)' +
-          formatName(declaration.child.name[0]) +
-          formatName(updatedChildDetails)
+          formatNameWithFirstNameFirst(declaration.child.name[0]) +
+          formatNameWithFirstNameFirst(updatedChildDetails)
       )
     ).toBeVisible()
 
@@ -654,7 +655,9 @@ test.describe.serial(' Correct record - 5', () => {
      * - include the declaration in this tab
      */
 
-    await expect(page.getByText(formatName(updatedChildDetails))).toBeVisible()
+    await expect(
+      page.getByText(formatNameWithFirstNameFirst(updatedChildDetails))
+    ).toBeVisible()
   })
   test('5.8 Validate history in record audit', async () => {
     await page.getByText(formatName(updatedChildDetails)).click()

diff --git a/e2e/testcases/correction-birth/correct-birth-record-6.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-6.spec.ts
@@ -6,6 +6,7 @@ import {
   expectOutboxToBeEmpty,
   formatDateTo_ddMMMMyyyy,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -130,11 +131,13 @@ test.describe.serial(' Correct record - 6', () => {
       await expectOutboxToBeEmpty(page)
 
       await expect(
-        page.getByText(formatName(declaration.child.name[0])).first()
+        page
+          .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
+          .first()
       ).toBeVisible()
 
       await page
-        .getByText(formatName(declaration.child.name[0]))
+        .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
         .first()
         .click()
     })

diff --git a/e2e/testcases/correction-birth/correct-birth-record-7.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-7.spec.ts
@@ -5,6 +5,7 @@ import {
   expectAddress,
   expectOutboxToBeEmpty,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -455,8 +456,8 @@ test.describe.serial(' Correct record - 7', () => {
     await expect(
       page.getByText(
         'Full name (mother)' +
-          formatName(declaration.mother.name[0]) +
-          formatName(updatedMotherDetails)
+          formatNameWithFirstNameFirst(declaration.mother.name[0]) +
+          formatNameWithFirstNameFirst(updatedMotherDetails)
       )
     ).toBeVisible()
 

diff --git a/e2e/testcases/correction-birth/correct-birth-record-8.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-8.spec.ts
@@ -6,6 +6,7 @@ import {
   expectOutboxToBeEmpty,
   formatDateTo_ddMMMMyyyy,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -430,8 +431,8 @@ test.describe.serial(' Correct record - 8', () => {
     await expect(
       page.getByText(
         'Full name (father)' +
-          formatName(declaration.father.name[0]) +
-          formatName(updatedFatherDetails)
+          formatNameWithFirstNameFirst(declaration.father.name[0]) +
+          formatNameWithFirstNameFirst(updatedFatherDetails)
       )
     ).toBeVisible()
 
@@ -516,7 +517,9 @@ test.describe.serial(' Correct record - 8', () => {
      */
 
     await expect(
-      page.getByText(formatName(declaration.child.name[0])).first()
+      page
+        .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
+        .first()
     ).toBeVisible()
   })
 

diff --git a/e2e/testcases/correction-birth/correct-birth-record-9.spec.ts b/e2e/testcases/correction-birth/correct-birth-record-9.spec.ts
@@ -6,6 +6,7 @@ import {
   expectOutboxToBeEmpty,
   formatDateTo_ddMMMMyyyy,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -130,11 +131,13 @@ test.describe.serial(' Correct record - 9', () => {
       await expectOutboxToBeEmpty(page)
       await page.getByRole('button', { name: 'Ready to issue' }).click()
       await expect(
-        page.getByText(formatName(declaration.child.name[0])).first()
+        page
+          .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
+          .first()
       ).toBeVisible()
 
       await page
-        .getByText(formatName(declaration.child.name[0]))
+        .getByText(formatNameWithFirstNameFirst(declaration.child.name[0]))
         .first()
         .click()
     })

diff --git a/e2e/testcases/correction-death/correct-death-record-10.spec.ts b/e2e/testcases/correction-death/correct-death-record-10.spec.ts
@@ -6,6 +6,7 @@ import {
   expectOutboxToBeEmpty,
   formatDateTo_ddMMMMyyyy,
   formatName,
+  formatNameWithFirstNameFirst,
   getAction,
   getToken,
   goBackToReview,
@@ -800,8 +801,8 @@ test.describe('10. Correct record - 10', () => {
         await expect(
           page.getByText(
             'Full name (Deceased)' +
-              formatName(declaration.deceased.name[0]) +
-              formatName(updatedDeceasedDetails)
+              formatNameWithFirstNameFirst(declaration.deceased.name[0]) +
+              formatNameWithFirstNameFirst(updatedDeceasedDetails)
           )
         ).toBeVisible()