@designcrowd/library.brand-page 5.12.13-flacky-nav-2 → 5.12.13

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,7 +916,7 @@ describe('migration.util', () => {
     } as any;
 
     // Act
-    const model = await migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+    const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
       isUserDraft: false,
     });
 
@@ -929,7 +929,7 @@ describe('migration.util', () => {
     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: [
@@ -971,7 +971,7 @@ describe('migration.util', () => {
     } as any;
 
     // Act
-    const model = await migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+    const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
       isUserDraft: false,
     });
 
@@ -986,9 +986,9 @@ 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,
         });
 
@@ -999,9 +999,9 @@ describe('migration.util', () => {
       });
     });
     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,
         });
 
@@ -1014,9 +1014,9 @@ 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,
       });
 
@@ -1033,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: [
@@ -1088,12 +1088,9 @@ describe('migration.util', () => {
       } as any;
 
       // Act
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
       // Assert
       expect(model).toMatchSnapshot();
@@ -1106,8 +1103,8 @@ 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,
       });
 
@@ -1118,9 +1115,9 @@ describe('migration.util', () => {
       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,
       });
 
@@ -1134,9 +1131,9 @@ 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,
       });
 
@@ -1150,11 +1147,9 @@ 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,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
       const bannerContent = model.pages[0].contents[0] as BannerSectionContent;
 
@@ -1162,11 +1157,9 @@ describe('migration.util', () => {
       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,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
       const bannerContent = model.pages[1].contents[1] as BannerSectionContent;
 
@@ -1177,9 +1170,9 @@ 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,
       });
 
@@ -1195,10 +1188,10 @@ 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,
       });
 
@@ -1214,9 +1207,9 @@ 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,
       });
 
@@ -1234,9 +1227,9 @@ 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,
       });
 
@@ -1247,9 +1240,9 @@ describe('migration.util', () => {
       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,
       });
 
@@ -1260,9 +1253,9 @@ describe('migration.util', () => {
       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,
       });
 
@@ -1273,9 +1266,9 @@ describe('migration.util', () => {
       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,
       });
 
@@ -1288,9 +1281,9 @@ 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,
     });
 
@@ -1304,9 +1297,9 @@ 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,
     });
 
@@ -1318,9 +1311,9 @@ 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,
     });
 
@@ -1338,9 +1331,9 @@ 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,
     });
 
@@ -1354,9 +1347,9 @@ 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,
     });
 
@@ -1366,9 +1359,9 @@ describe('migration.util', () => {
     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,
     });
 
@@ -1379,9 +1372,9 @@ describe('migration.util', () => {
   });
 
   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,
       });
 
@@ -1389,13 +1382,13 @@ describe('migration.util', () => {
 
       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,
       });
 
@@ -1404,8 +1397,8 @@ describe('migration.util', () => {
       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,
       });
 
@@ -1483,11 +1476,9 @@ 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,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
       const content = model.styles.textStyles;
 
@@ -1552,11 +1543,9 @@ 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,
-      });
+      const model = migrateBrandPageModel(getBsModel(), { isUserDraft: false });
 
       const content = model.styles.textStyles;
 
@@ -1575,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,
@@ -1583,18 +1572,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
       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: [
           {
@@ -1612,18 +1598,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      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: [
           {
@@ -1650,12 +1633,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
       const section = model.pages[0].contents[0] as any;
 
       section.items.forEach((item: any) => {
@@ -1663,7 +1643,7 @@ describe('migration.util', () => {
       });
     });
 
-    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: [
           {
@@ -1690,12 +1670,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
       const gallery = model.pages[0].contents[0] as any;
 
       gallery.images.forEach((img: any) => {
@@ -1703,7 +1680,7 @@ describe('migration.util', () => {
       });
     });
 
-    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: [
           {
@@ -1721,18 +1698,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      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: [
           {
@@ -1750,18 +1724,15 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      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: [
           {
@@ -1788,12 +1759,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
       const socialLinks = model.pages[0].contents[0] as any;
 
       socialLinks.links.forEach((link: any) => {
@@ -1801,7 +1769,7 @@ describe('migration.util', () => {
       });
     });
 
-    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: [
@@ -1831,12 +1799,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
       const desktopLinks = model.pageFooterDesktop?.socialLinks || [];
       const mobileLinks = model.pageFooterMobile?.socialLinks || [];
 
@@ -1848,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: {
@@ -1871,12 +1836,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
       const desktopLinks = model.pageFooterDesktop?.socialLinks || [];
       const mobileLinks = model.pageFooterMobile?.socialLinks || [];
 
@@ -1888,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: [
@@ -1910,12 +1872,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
       const desktopLinks = model.pageFooterDesktop?.socialLinks || [];
       const mobileLinks = model.pageFooterMobile?.socialLinks || [];
 
@@ -1927,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,
@@ -1948,12 +1907,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
       expect(model.faviconUrl).toBe(urlWithoutExtension);
       expect(model.ogImageUrl).toBe(urlWithoutExtension);
@@ -1963,7 +1919,7 @@ describe('migration.util', () => {
       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,
@@ -1983,12 +1939,9 @@ describe('migration.util', () => {
         brandPageType: 'brandSite',
       } as any;
 
-      const model = await migrateBrandPageModel(
-        new BrandPageModel(oldTemplate),
-        {
-          isUserDraft: false,
-        },
-      );
+      const model = migrateBrandPageModel(new BrandPageModel(oldTemplate), {
+        isUserDraft: false,
+      });
 
       expect(model.faviconUrl).toBe(nonUserUrl);
       expect(model.ogImageUrl).toBe(nonUserUrl);