@designcrowd/library.brand-page 5.14.8-flacky-nav-21 → 5.14.8

package/src/brand-page/utils/migration.util.test.ts

@@ -879,7 +879,7 @@ describe('migration.util', () => {
     } as any;
   };
 
-  test('migrateBrandPageModel should return brand page model with migrated textItems if missing for brand site', async () => {
+  test('migrateBrandPageModel should return brand page model with migrated textItems if missing for brand site', () => {
     // Arrange
     const oldTemplate: BrandPageData = {
       pages: [
@@ -916,22 +916,20 @@ describe('migration.util', () => {
     } as any;
 
     // Act
-    const model = await migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+    const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
     // Assert
     expect(model).toMatchSnapshot();
 
-    const textItems = (model.brandPageModel.pages[0].contents[0] as TextContent)
-      .textItems;
+    const textItems = (model.pages[0].contents[0] as TextContent).textItems;
 
     expect(textItems).not.toBeNull();
     expect(textItems![0].textStyle).not.toBeNull();
   });
 
-  test('migrateBrandPageModel should return brand page model with default textstyle object if textItem has missing textStyle object for brand site', async () => {
+  test('migrateBrandPageModel should return brand page model with default textstyle object if textItem has missing textStyle object for brand site', () => {
     // Arrange
     const oldTemplate: BrandPageModel = {
       pages: [
@@ -973,16 +971,14 @@ describe('migration.util', () => {
     } as any;
 
     // Act
-    const model = await migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+    const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
     // Assert
     expect(model).toMatchSnapshot();
 
-    const textItems = (model.brandPageModel.pages[0].contents[0] as TextContent)
-      .textItems;
+    const textItems = (model.pages[0].contents[0] as TextContent).textItems;
 
     expect(textItems).not.toBeNull();
     expect(textItems![0].textStyle).not.toBeNull();
@@ -990,30 +986,26 @@ describe('migration.util', () => {
 
   describe('Brandpage Models', () => {
     describe('Button Contents', () => {
-      test('migrateBrandPageModel should return missing text styles', async () => {
+      test('migrateBrandPageModel should return missing text styles', () => {
         // Act
-        const model = await migrateBrandPageModel(getBrandPageTypeModel(), {
+        const model = migrateBrandPageModel(getBrandPageTypeModel(), {
           isUserDraft: false,
-          isServerRender: false,
         });
 
-        const content = model.brandPageModel.pages[0]
-          .contents[1] as ButtonWithModalContent;
+        const content = model.pages[0].contents[1] as ButtonWithModalContent;
 
         expect(content.titleStyle.fontFamily).toBe('Lato');
         expect(content.titleStyle.lineHeight).toBe('1.5');
       });
     });
     describe('Profile Contents', () => {
-      test('migrateBrandPageModel should return missing text styles', async () => {
+      test('migrateBrandPageModel should return missing text styles', () => {
         // Act
-        const model = await migrateBrandPageModel(getBrandPageTypeModel(), {
+        const model = migrateBrandPageModel(getBrandPageTypeModel(), {
           isUserDraft: false,
-          isServerRender: false,
         });
 
-        const content = model.brandPageModel.pages[0]
-          .contents[1] as ProfileContent;
+        const content = model.pages[0].contents[1] as ProfileContent;
 
         expect(content.titleStyle.fontFamily).toBe('Lato');
         expect(content.titleStyle.lineHeight).toBe('1.5');
@@ -1022,15 +1014,13 @@ describe('migration.util', () => {
   });
 
   describe('Banner Contents - text Align and Font Family should not be empty', () => {
-    test('migrateBrandPageModel should return banner contents with text style to have textAlign and FontFamily if its missing', async () => {
+    test('migrateBrandPageModel should return banner contents with text style to have textAlign and FontFamily if its missing', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const bannerContent = model.brandPageModel.pages[0]
-        .contents[0] as BannerSectionContent;
+      const bannerContent = model.pages[0].contents[0] as BannerSectionContent;
 
       expect(bannerContent.titleStyle.textAlign).toBe('center');
       expect(bannerContent.descriptionStyle.textAlign).toBe('center');
@@ -1043,7 +1033,7 @@ describe('migration.util', () => {
       expect(bannerContent.buttonTextStyle.lineHeight).toBe(LINE_HEIGHT_BODY);
     });
 
-    test('migrateBannerContentToRawImageUrl should return banner images url without q and w parameter when given a model with unsplash image url', async () => {
+    test('migrateBannerContentToRawImageUrl should return banner images url without q and w parameter when given a model with unsplash image url', () => {
       // Arrange
       const oldTemplate: BrandPageModel = {
         pages: [
@@ -1098,19 +1088,14 @@ describe('migration.util', () => {
       } as any;
 
       // Act
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
       // Assert
       expect(model).toMatchSnapshot();
 
-      const bannerContent = model.brandPageModel.pages[0]
-        .contents[0] as BannerSectionContent;
+      const bannerContent = model.pages[0].contents[0] as BannerSectionContent;
 
       const url = new URL(bannerContent.imageUrl);
 
@@ -1118,29 +1103,25 @@ describe('migration.util', () => {
       expect(url.searchParams.has('w')).toBe(false);
     });
 
-    test('migrateBrandPageModel should return banner contents with text style to have textAlign based on display style', async () => {
-      const model = await migrateBrandPageModel(getBsModel(), {
+    test('migrateBrandPageModel should return banner contents with text style to have textAlign based on display style', () => {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const bannerContent = model.brandPageModel.pages[0]
-        .contents[3] as BannerSectionContent;
+      const bannerContent = model.pages[0].contents[3] as BannerSectionContent;
 
       expect(bannerContent.titleStyle.textAlign).toBe('left');
       expect(bannerContent.descriptionStyle.textAlign).toBe('left');
       expect(bannerContent.buttonTextStyle.textAlign).toBe('left');
     });
 
-    test('migrateBrandPageModel should return text style with textAlign and FontFamily for newly added Banner Content', async () => {
+    test('migrateBrandPageModel should return text style with textAlign and FontFamily for newly added Banner Content', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const bannerContent = model.brandPageModel.pages[1]
-        .contents[0] as BannerSectionContent;
+      const bannerContent = model.pages[1].contents[0] as BannerSectionContent;
 
       expect(bannerContent.titleStyle.textAlign).toBe('center');
       expect(bannerContent.descriptionStyle.textAlign).toBe('center');
@@ -1150,15 +1131,13 @@ describe('migration.util', () => {
       expect(bannerContent.buttonTextStyle.fontFamily).toBe('Lato');
     });
 
-    test('migrateBrandPageModel should not change existing alignments for existing contents for banner contents', async () => {
+    test('migrateBrandPageModel should not change existing alignments for existing contents for banner contents', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const bannerContent = model.brandPageModel.pages[1]
-        .contents[1] as BannerSectionContent;
+      const bannerContent = model.pages[1].contents[1] as BannerSectionContent;
 
       expect(bannerContent.titleStyle.textAlign).toBe('right');
       expect(bannerContent.descriptionStyle.textAlign).toBe('left');
@@ -1168,29 +1147,21 @@ describe('migration.util', () => {
       expect(bannerContent.buttonTextStyle.fontFamily).toBe('Lato');
     });
 
-    test('migrateBrandPageModel should convert lineheight rem to unitless', async () => {
+    test('migrateBrandPageModel should convert lineheight rem to unitless', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
-        isUserDraft: false,
-        isServerRender: false,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
-      const bannerContent = model.brandPageModel.pages[0]
-        .contents[0] as BannerSectionContent;
+      const bannerContent = model.pages[0].contents[0] as BannerSectionContent;
 
       expect(bannerContent.titleStyle.lineHeight).toBe(LINE_HEIGHT_TITLE);
       expect(bannerContent.descriptionStyle.lineHeight).toBe(LINE_HEIGHT_BODY);
       expect(bannerContent.buttonTextStyle.lineHeight).toBe(LINE_HEIGHT_BODY);
     });
-    test('migrateBrandPageModel should not replace lineheight if theres existing', async () => {
+    test('migrateBrandPageModel should not replace lineheight if theres existing', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
-        isUserDraft: false,
-        isServerRender: false,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
-      const bannerContent = model.brandPageModel.pages[1]
-        .contents[1] as BannerSectionContent;
+      const bannerContent = model.pages[1].contents[1] as BannerSectionContent;
 
       expect(bannerContent.titleStyle.lineHeight).toBe('1.6');
       expect(bannerContent.descriptionStyle.lineHeight).toBe('1.2');
@@ -1199,14 +1170,13 @@ describe('migration.util', () => {
   });
 
   describe('Testimonial Contents - text Align and Font Family should not be empty', () => {
-    test('migrateBrandPageModel should return testimonial contents with text style that are not assigned to have textAlign and FontFamily', async () => {
+    test('migrateBrandPageModel should return testimonial contents with text style that are not assigned to have textAlign and FontFamily', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const testimonialContent = model.brandPageModel.pages[0]
+      const testimonialContent = model.pages[0]
         .contents[1] as TestimonialSectionContent;
 
       expect(testimonialContent.titleStyle.textAlign).toBe('center');
@@ -1218,15 +1188,14 @@ describe('migration.util', () => {
       expect(testimonialContent.nameStyle.fontFamily).toBe('Lato');
       expect(testimonialContent.positionStyle.fontFamily).toBe('Lato');
     });
-    test('migrateBrandPageModel should return testimonial contents with text style to have textAlign and FontFamily - column displayStyle', async () => {
+    test('migrateBrandPageModel should return testimonial contents with text style to have textAlign and FontFamily - column displayStyle', () => {
       // Act
 
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const testimonialContent = model.brandPageModel.pages[1]
+      const testimonialContent = model.pages[1]
         .contents[3] as TestimonialSectionContent;
 
       expect(testimonialContent.titleStyle.textAlign).toBe('center');
@@ -1238,14 +1207,13 @@ describe('migration.util', () => {
       expect(testimonialContent.nameStyle.fontFamily).toBe('Lato');
       expect(testimonialContent.positionStyle.fontFamily).toBe('Lato');
     });
-    test('migrateBrandPageModel should not change existing alignments for existing contents for testimonial contents', async () => {
+    test('migrateBrandPageModel should not change existing alignments for existing contents for testimonial contents', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const testimonialContent = model.brandPageModel.pages[1]
+      const testimonialContent = model.pages[1]
         .contents[4] as TestimonialSectionContent;
 
       expect(testimonialContent.titleStyle.textAlign).toBe('right');
@@ -1259,60 +1227,52 @@ describe('migration.util', () => {
     });
   });
   describe('About Contents - text Align and Font Family should not be empty', () => {
-    test('migrateBrandPageModel should return About contents with text style to have textAlign and FontFamily', async () => {
+    test('migrateBrandPageModel should return About contents with text style to have textAlign and FontFamily', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const aboutContents = model.brandPageModel.pages[0]
-        .contents[2] as AboutSectionContent;
+      const aboutContents = model.pages[0].contents[2] as AboutSectionContent;
 
       expect(aboutContents.titleStyle.textAlign).toBe('center');
       expect(aboutContents.paragraphStyle.textAlign).toBe('center');
       expect(aboutContents.titleStyle.fontFamily).toBe('Lato');
       expect(aboutContents.paragraphStyle.fontFamily).toBe('Lato');
     });
-    test('migrateBrandPageModel should return Newly Added About contents with text style to have textAlign and FontFamily', async () => {
+    test('migrateBrandPageModel should return Newly Added About contents with text style to have textAlign and FontFamily', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const aboutContents = model.brandPageModel.pages[2]
-        .contents[0] as AboutSectionContent;
+      const aboutContents = model.pages[2].contents[0] as AboutSectionContent;
 
       expect(aboutContents.titleStyle.textAlign).toBe('center');
       expect(aboutContents.paragraphStyle.textAlign).toBe('center');
       expect(aboutContents.titleStyle.fontFamily).toBe('Lato');
       expect(aboutContents.paragraphStyle.fontFamily).toBe('Lato');
     });
-    test('migrateBrandPageModel should return text Align and Font family with the correct alignment based on display style', async () => {
+    test('migrateBrandPageModel should return text Align and Font family with the correct alignment based on display style', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const aboutContents = model.brandPageModel.pages[2]
-        .contents[1] as AboutSectionContent;
+      const aboutContents = model.pages[2].contents[1] as AboutSectionContent;
 
       expect(aboutContents.titleStyle.textAlign).toBe('left');
       expect(aboutContents.paragraphStyle.textAlign).toBe('left');
       expect(aboutContents.titleStyle.fontFamily).toBe('Lato');
       expect(aboutContents.paragraphStyle.fontFamily).toBe('Lato');
     });
-    test('migrateBrandPageModel should  not change text alignment of existing contents', async () => {
+    test('migrateBrandPageModel should  not change text alignment of existing contents', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const aboutContents = model.brandPageModel.pages[2]
-        .contents[2] as AboutSectionContent;
+      const aboutContents = model.pages[2].contents[2] as AboutSectionContent;
 
       expect(aboutContents.titleStyle.textAlign).toBe('left');
       expect(aboutContents.paragraphStyle.textAlign).toBe('right');
@@ -1321,14 +1281,13 @@ describe('migration.util', () => {
     });
   });
 
-  test('migrateBrandPageModel should return profile contents with text style to have textAlign and FontFamily', async () => {
+  test('migrateBrandPageModel should return profile contents with text style to have textAlign and FontFamily', () => {
     // Act
-    const model = await migrateBrandPageModel(getBsModel(), {
+    const model = migrateBrandPageModel(getBsModel(), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
-    const content = model.brandPageModel.pages[0].contents[4] as ProfileContent;
+    const content = model.pages[0].contents[4] as ProfileContent;
 
     expect(content.titleStyle.textAlign).toBe('center');
     expect(content.nameStyle.textAlign).toBe('center');
@@ -1338,15 +1297,13 @@ describe('migration.util', () => {
     expect(content.descriptionStyle.fontFamily).toBe('Lato');
   });
 
-  test('migrateBrandPageModel should return mailing list contents with text style to have textAlign and FontFamily', async () => {
+  test('migrateBrandPageModel should return mailing list contents with text style to have textAlign and FontFamily', () => {
     // Act
-    const model = await migrateBrandPageModel(getBsModel(), {
+    const model = migrateBrandPageModel(getBsModel(), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
-    const content = model.brandPageModel.pages[0]
-      .contents[5] as MailingListContent;
+    const content = model.pages[0].contents[5] as MailingListContent;
 
     expect(content.titleStyle.textAlign).toBe('center');
     expect(content.buttonTextStyle.textAlign).toBe('center');
@@ -1354,15 +1311,13 @@ describe('migration.util', () => {
     expect(content.buttonTextStyle.fontFamily).toBe('Lato');
   });
 
-  test('migrateBrandPageModel should return support me contents with text style to have textAlign and FontFamily', async () => {
+  test('migrateBrandPageModel should return support me contents with text style to have textAlign and FontFamily', () => {
     // Act
-    const model = await migrateBrandPageModel(getBsModel(), {
+    const model = migrateBrandPageModel(getBsModel(), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
-    const content = model.brandPageModel.pages[0]
-      .contents[6] as SupportMeContent;
+    const content = model.pages[0].contents[6] as SupportMeContent;
 
     expect(content.titleStyle.textAlign).toBe('center');
     expect(content.headingStyle.textAlign).toBe('center');
@@ -1376,15 +1331,13 @@ describe('migration.util', () => {
     expect(content.itemTextStyle.fontFamily).toBe('Lato');
   });
 
-  test('migrateBrandPageModel should return images contents with text style to have textAlign and FontFamily', async () => {
+  test('migrateBrandPageModel should return images contents with text style to have textAlign and FontFamily', () => {
     // Act
-    const model = await migrateBrandPageModel(getBsModel(), {
+    const model = migrateBrandPageModel(getBsModel(), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
-    const content = model.brandPageModel.pages[0]
-      .contents[7] as ImagesSectionContent;
+    const content = model.pages[0].contents[7] as ImagesSectionContent;
 
     expect(content.titleStyle.textAlign).toBe('center');
     expect(content.headingStyle.textAlign).toBe('center');
@@ -1394,72 +1347,62 @@ describe('migration.util', () => {
     expect(content.paragraphStyle.fontFamily).toBe('Lato');
   });
 
-  test('migrateBrandPageModel should return payment contents with text style to have textAlign and FontFamily', async () => {
+  test('migrateBrandPageModel should return payment contents with text style to have textAlign and FontFamily', () => {
     // Act
-    const model = await migrateBrandPageModel(getBsModel(), {
+    const model = migrateBrandPageModel(getBsModel(), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
-    const content = model.brandPageModel.pages[0]
-      .contents[8] as RequestPaymentContent;
+    const content = model.pages[0].contents[8] as RequestPaymentContent;
 
     expect(content.itemTextStyle.textAlign).toBe('left');
     expect(content.itemTextStyle.fontFamily).toBe('Lato');
   });
 
-  test('migrateBrandPageModel should return social links contents with text style to have textAlign and FontFamily', async () => {
+  test('migrateBrandPageModel should return social links contents with text style to have textAlign and FontFamily', () => {
     // Act
-    const model = await migrateBrandPageModel(getBsModel(), {
+    const model = migrateBrandPageModel(getBsModel(), {
       isUserDraft: false,
-      isServerRender: false,
     });
 
-    const content = model.brandPageModel.pages[0]
-      .contents[9] as SocialLinksContent;
+    const content = model.pages[0].contents[9] as SocialLinksContent;
 
     expect(content.titleStyle.textAlign).toBe('center');
     expect(content.titleStyle.fontFamily).toBe('Lato');
   });
 
   describe('Contact Form Content - Layout style testing', () => {
-    test('migrateBrandPageModel should return empty layoutStyle for existing contact forms that dont have layoutStyle', async () => {
+    test('migrateBrandPageModel should return empty layoutStyle for existing contact forms that dont have layoutStyle', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const content = model.brandPageModel.pages[3]
-        .contents[0] as ContactFormContent;
+      const content = model.pages[3].contents[0] as ContactFormContent;
 
       expect(content.layoutStyle).toBe('');
     });
-    test('migrateBrandPageModel should return empty layoutStyle for Brandpage only', async () => {
+    test('migrateBrandPageModel should return empty layoutStyle for Brandpage only', () => {
       // Arrange
       const brandPageModel: BrandPageModel = {
         ...getBsModel(),
         brandPageType: BRAND_PAGE_TYPES.default,
       } as any;
-      const model = await migrateBrandPageModel(brandPageModel, {
+      const model = migrateBrandPageModel(brandPageModel, {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const content = model.brandPageModel.pages[3]
-        .contents[1] as ContactFormContent;
+      const content = model.pages[3].contents[1] as ContactFormContent;
 
       expect(content.layoutStyle).toBe('');
     });
 
-    test('migrateBrandPageModel should not affect existing content blocks with layout style', async () => {
-      const model = await migrateBrandPageModel(getBsModel(), {
+    test('migrateBrandPageModel should not affect existing content blocks with layout style', () => {
+      const model = migrateBrandPageModel(getBsModel(), {
         isUserDraft: false,
-        isServerRender: false,
       });
 
-      const content = model.brandPageModel.pages[3]
-        .contents[1] as ContactFormContent;
+      const content = model.pages[3].contents[1] as ContactFormContent;
 
       expect(content.layoutStyle).toBe(TEXT_ONLY);
     });
@@ -1533,14 +1476,11 @@ describe('migration.util', () => {
     });
   });
   describe('Global Styles - check text Styles', () => {
-    test('migrateBrandPageModel should set text styles to be defined', async () => {
+    test('migrateBrandPageModel should set text styles to be defined', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
-        isUserDraft: false,
-        isServerRender: false,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
-      const content = model.brandPageModel.styles.textStyles;
+      const content = model.styles.textStyles;
 
       expect(content.textStyleOptions['titles'].style.lineHeight).toBe(
         LINE_HEIGHT_TITLE,
@@ -1603,14 +1543,11 @@ describe('migration.util', () => {
         undefined,
       );
     });
-    test('migrateBrandPageModel should convert rem lineheight to unitless', async () => {
+    test('migrateBrandPageModel should convert rem lineheight to unitless', () => {
       // Act
-      const model = await migrateBrandPageModel(getBsModel(), {
-        isUserDraft: false,
-        isServerRender: false,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
-      const content = model.brandPageModel.styles.textStyles;
+      const content = model.styles.textStyles;
 
       expect(content.textStyleOptions['buttons'].style.lineHeight).toBe(
         LINE_HEIGHT_BODY,
@@ -1627,7 +1564,7 @@ describe('migration.util', () => {
     const nonUserUrl =
       'https://images.unsplash.com/photo-1657664072470-99b02c2143f2.jpg';
 
-    test('should remove extension from faviconUrl and ogImageUrl', async () => {
+    test('should remove extension from faviconUrl and ogImageUrl', () => {
       const oldTemplate: BrandPageModel = {
         faviconUrl: userUploadedUrl,
         ogImageUrl: userUploadedUrl,
@@ -1635,19 +1572,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
-      expect(model.brandPageModel.faviconUrl).toBe(expectedUrl);
-      expect(model.brandPageModel.ogImageUrl).toBe(expectedUrl);
+      expect(model.faviconUrl).toBe(expectedUrl);
+      expect(model.ogImageUrl).toBe(expectedUrl);
     });
 
-    test('should remove extension from content imageUrl for type "image"', async () => {
+    test('should remove extension from content imageUrl for type "image"', () => {
       const oldTemplate: BrandPageModel = {
         pages: [
           {
@@ -1665,19 +1598,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const pageContent = model.brandPageModel.pages[0].contents[0] as any;
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const pageContent = model.pages[0].contents[0] as any;
 
       expect(pageContent.imageUrl).toBe(expectedUrl);
     });
 
-    test('should remove extension from content imageUrl inside "images-section"', async () => {
+    test('should remove extension from content imageUrl inside "images-section"', () => {
       const oldTemplate: BrandPageModel = {
         pages: [
           {
@@ -1704,21 +1633,17 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const section = model.brandPageModel.pages[0].contents[0] as any;
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const section = model.pages[0].contents[0] as any;
 
       section.items.forEach((item: any) => {
         expect(item.imageUrl).toBe(expectedUrl);
       });
     });
 
-    test('should remove extension from content imageUrl inside "image-gallery"', async () => {
+    test('should remove extension from content imageUrl inside "image-gallery"', () => {
       const oldTemplate: BrandPageModel = {
         pages: [
           {
@@ -1745,21 +1670,17 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const gallery = model.brandPageModel.pages[0].contents[0] as any;
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const gallery = model.pages[0].contents[0] as any;
 
       gallery.images.forEach((img: any) => {
         expect(img.imageUrl).toBe(expectedUrl);
       });
     });
 
-    test('should remove extension from content modalImageUrl inside "general-link"', async () => {
+    test('should remove extension from content modalImageUrl inside "general-link"', () => {
       const oldTemplate: BrandPageModel = {
         pages: [
           {
@@ -1777,19 +1698,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const generalLink = model.brandPageModel.pages[0].contents[0] as any;
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const generalLink = model.pages[0].contents[0] as any;
 
       expect(generalLink.iconUrl).toBe(expectedUrl);
     });
 
-    test('should remove extension from content imageUrl inside "about-section"', async () => {
+    test('should remove extension from content imageUrl inside "about-section"', () => {
       const oldTemplate: BrandPageModel = {
         pages: [
           {
@@ -1807,19 +1724,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const aboutSection = model.brandPageModel.pages[0].contents[0] as any;
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const aboutSection = model.pages[0].contents[0] as any;
 
       expect(aboutSection.imageUrl).toBe(expectedUrl);
     });
 
-    test('should remove extension from content imageUrl inside "social-links"', async () => {
+    test('should remove extension from content imageUrl inside "social-links"', () => {
       const oldTemplate: BrandPageModel = {
         pages: [
           {
@@ -1846,21 +1759,17 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const socialLinks = model.brandPageModel.pages[0].contents[0] as any;
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const socialLinks = model.pages[0].contents[0] as any;
 
       socialLinks.links.forEach((link: any) => {
         expect(link.imageUrl).toBe(expectedUrl);
       });
     });
 
-    test('should remove extension from social links in pageFooterDesktop and pageFooterMobile', async () => {
+    test('should remove extension from social links in pageFooterDesktop and pageFooterMobile', () => {
       const oldTemplate: BrandPageModel = {
         pageFooterDesktop: {
           socialLinks: [
@@ -1890,17 +1799,11 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const desktopLinks =
-        model.brandPageModel.pageFooterDesktop?.socialLinks || [];
-      const mobileLinks =
-        model.brandPageModel.pageFooterMobile?.socialLinks || [];
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const desktopLinks = model.pageFooterDesktop?.socialLinks || [];
+      const mobileLinks = model.pageFooterMobile?.socialLinks || [];
 
       desktopLinks.forEach((link: any) => {
         expect(link.imageUrl).toBe(expectedUrl);
@@ -1910,7 +1813,7 @@ describe('migration.util', () => {
       });
     });
 
-    test('should leave social link URL unchanged if file extension is not present', async () => {
+    test('should leave social link URL unchanged if file extension is not present', () => {
       const noExtensionUrl = expectedUrl;
       const oldTemplate: BrandPageModel = {
         pageFooterDesktop: {
@@ -1933,17 +1836,11 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const desktopLinks =
-        model.brandPageModel.pageFooterDesktop?.socialLinks || [];
-      const mobileLinks =
-        model.brandPageModel.pageFooterMobile?.socialLinks || [];
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const desktopLinks = model.pageFooterDesktop?.socialLinks || [];
+      const mobileLinks = model.pageFooterMobile?.socialLinks || [];
 
       desktopLinks.forEach((link: any) => {
         expect(link.imageUrl).toBe(noExtensionUrl);
@@ -1953,7 +1850,7 @@ describe('migration.util', () => {
       });
     });
 
-    test('should leave social link URL unchanged if the domain is different', async () => {
+    test('should leave social link URL unchanged if the domain is different', () => {
       const oldTemplate: BrandPageModel = {
         pageFooterDesktop: {
           socialLinks: [
@@ -1975,17 +1872,11 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
-      const desktopLinks =
-        model.brandPageModel.pageFooterDesktop?.socialLinks || [];
-      const mobileLinks =
-        model.brandPageModel.pageFooterMobile?.socialLinks || [];
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
+      const desktopLinks = model.pageFooterDesktop?.socialLinks || [];
+      const mobileLinks = model.pageFooterMobile?.socialLinks || [];
 
       desktopLinks.forEach((link: any) => {
         expect(link.imageUrl).toBe(nonUserUrl);
@@ -1995,7 +1886,7 @@ describe('migration.util', () => {
       });
     });
 
-    test('should leave URL unchanged if file extension is not present', async () => {
+    test('should leave URL unchanged if file extension is not present', () => {
       const urlWithoutExtension = expectedUrl;
       const oldTemplate: BrandPageModel = {
         faviconUrl: urlWithoutExtension,
@@ -2016,23 +1907,19 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
-      expect(model.brandPageModel.faviconUrl).toBe(urlWithoutExtension);
-      expect(model.brandPageModel.ogImageUrl).toBe(urlWithoutExtension);
+      expect(model.faviconUrl).toBe(urlWithoutExtension);
+      expect(model.ogImageUrl).toBe(urlWithoutExtension);
 
-      const pageContent = model.brandPageModel.pages[0].contents[0] as any;
+      const pageContent = model.pages[0].contents[0] as any;
 
       expect(pageContent.imageUrl).toBe(urlWithoutExtension);
     });
 
-    test('should leave URL unchanged if the domain is different', async () => {
+    test('should leave URL unchanged if the domain is different', () => {
       const oldTemplate: BrandPageModel = {
         faviconUrl: nonUserUrl,
         ogImageUrl: nonUserUrl,
@@ -2052,17 +1939,13 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-          isServerRender: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
-      expect(model.brandPageModel.faviconUrl).toBe(nonUserUrl);
-      expect(model.brandPageModel.ogImageUrl).toBe(nonUserUrl);
-      const pageContent = model.brandPageModel.pages[0].contents[0] as any;
+      expect(model.faviconUrl).toBe(nonUserUrl);
+      expect(model.ogImageUrl).toBe(nonUserUrl);
+      const pageContent = model.pages[0].contents[0] as any;
 
       expect(pageContent.imageUrl).toBe(nonUserUrl);
     });