@blaze-cms/react-page-builder 0.128.0-admin-updates.3 → 0.128.0-admin-updates.7

package/tests/unit/src/components/DataSummary/DataSummaryTypes/__snapshots__/ItemDetailsSummary.test.js.snap

@@ -13,12 +13,22 @@ exports[`ItemDetailsSummary component should render without throwing an error an
     <span
       class=""
     >
-      something
+      <span
+        class="item-details-summary-label"
+      >
+        first
+      </span>
+       something
     </span>
     <span
       class=""
     >
-      other thing
+      <span
+        class="item-details-summary-label"
+      >
+        second
+      </span>
+       other thing
     </span>
   </div>
 </DocumentFragment>
@@ -37,12 +47,22 @@ exports[`ItemDetailsSummary component should render without throwing an error if
     <span
       class=""
     >
-      something
+      <span
+        class="item-details-summary-label"
+      >
+        first
+      </span>
+       something
     </span>
     <span
       class=""
     >
-      other thing
+      <span
+        class="item-details-summary-label"
+      >
+        second
+      </span>
+       other thing
     </span>
   </div>
 </DocumentFragment>

package/tests/unit/src/components/List/components/Cards/__snapshots__/CardsRender.test.js.snap

@@ -25,11 +25,11 @@ exports[`CardsRender component should render based on listData received 1`] = `
             <div
               class="card__content card__content--portrait"
             >
-              <h2
+              <div
                 class="card__title card__title--portrait "
               >
                 item 1
-              </h2>
+              </div>
             </div>
           </div>
         </div>
@@ -46,11 +46,11 @@ exports[`CardsRender component should render based on listData received 1`] = `
             <div
               class="card__content card__content--portrait"
             >
-              <h2
+              <div
                 class="card__title card__title--portrait "
               >
                 item 1
-              </h2>
+              </div>
             </div>
           </div>
         </div>
@@ -67,11 +67,11 @@ exports[`CardsRender component should render based on listData received 1`] = `
             <div
               class="card__content card__content--portrait"
             >
-              <h2
+              <div
                 class="card__title card__title--portrait "
               >
                 item 1
-              </h2>
+              </div>
             </div>
           </div>
         </div>
@@ -100,11 +100,11 @@ exports[`CardsRender component should render without wrapper, when requested 1`]
         <div
           class="card__content card__content--portrait"
         >
-          <h2
+          <div
             class="card__title card__title--portrait "
           >
             item 1
-          </h2>
+          </div>
         </div>
       </div>
     </div>
@@ -121,11 +121,11 @@ exports[`CardsRender component should render without wrapper, when requested 1`]
         <div
           class="card__content card__content--portrait"
         >
-          <h2
+          <div
             class="card__title card__title--portrait "
           >
             item 1
-          </h2>
+          </div>
         </div>
       </div>
     </div>
@@ -142,11 +142,11 @@ exports[`CardsRender component should render without wrapper, when requested 1`]
         <div
           class="card__content card__content--portrait"
         >
-          <h2
+          <div
             class="card__title card__title--portrait "
           >
             item 1
-          </h2>
+          </div>
         </div>
       </div>
     </div>

package/tests/unit/src/components/List/components/helpers/check-if-should-render-banner.test.js

@@ -1,44 +1,75 @@
-import checkIfShouldRenderBanner from '../../../../../../../src/components/List/components/helpers/check-if-should-render-banner';
+import { checkIfShouldRenderBanner } from '../../../../../../../src/components/List/components/helpers/check-if-should-render-banner';
 
-describe('checkIfShouldRenderBanner helper function', () => {
-  const noBanner = checkIfShouldRenderBanner(1, {
-    interval: 2,
-    repeat: false,
-    hasBanner: false
-  });
-  const noInterval = checkIfShouldRenderBanner(1, {
-    interval: 2,
-    repeat: false,
-    hasBanner: false
-  });
-  const positionMatch = checkIfShouldRenderBanner(1, {
-    interval: 2,
-    repeat: false,
-    hasBanner: true
-  });
-
-  const repeatOffMatch = checkIfShouldRenderBanner(3, {
-    interval: 2,
-    repeat: false,
-    hasBanner: true
-  });
-  const repeatOnMatch = checkIfShouldRenderBanner(3, {
-    interval: 2,
-    repeat: true,
-    hasBanner: true
-  });
+const BANNER_MOCK = {
+  repeatEvery: 3,
+  startingPoint: 5,
+  maxRepetitions: 2
+};
 
+describe('checkIfShouldRenderBanner helper function', () => {
   it('should return false if no interval or hasBanner is false', () => {
+    const noBanner = checkIfShouldRenderBanner(1, {
+      interval: 2,
+      repeat: false,
+      hasBanner: false
+    });
+    const noInterval = checkIfShouldRenderBanner(1, {
+      interval: 2,
+      repeat: false,
+      hasBanner: false
+    });
     expect(noBanner).toBe(false);
     expect(noInterval).toBe(false);
   });
   it('should return true if interval matches position', () => {
+    const positionMatch = checkIfShouldRenderBanner(1, {
+      interval: 2,
+      repeat: false,
+      hasBanner: true
+    });
     expect(positionMatch).toBe(true);
   });
   it('should return false if repeat is false and position repeat mtaches', () => {
+    const repeatOffMatch = checkIfShouldRenderBanner(3, {
+      interval: 2,
+      repeat: false,
+      hasBanner: true
+    });
     expect(repeatOffMatch).toBe(false);
   });
   it('should return true if repeat is true and position repeat mtaches', () => {
+    const repeatOnMatch = checkIfShouldRenderBanner(3, {
+      interval: 2,
+      repeat: true,
+      hasBanner: true
+    });
     expect(repeatOnMatch).toBe(true);
   });
+
+  it('should return false if no banner is present', () => {
+    const noBanner = checkIfShouldRenderBanner(1);
+    expect(noBanner).toBe(false);
+  });
+
+  it('should return false if index is lower than startingPoint', () => {
+    const lowIndex = checkIfShouldRenderBanner(3, BANNER_MOCK);
+    expect(lowIndex).toBe(false);
+  });
+
+  it('should return true if index matches startingPoint', () => {
+    const startingPointMatch = checkIfShouldRenderBanner(4, BANNER_MOCK);
+    expect(startingPointMatch).toBe(true);
+  });
+
+  it('should return true if repeat point matches and max repetitions hasnt been reached', () => {
+    const repetitionMatch = checkIfShouldRenderBanner(7, BANNER_MOCK);
+    const finalRepetitionMatch = checkIfShouldRenderBanner(10, BANNER_MOCK);
+    expect(repetitionMatch).toBe(true);
+    expect(finalRepetitionMatch).toBe(false);
+  });
+
+  it('should return false if repeat point matches and max repetitions has been reached', () => {
+    const maxRepetitionReached = checkIfShouldRenderBanner(13, BANNER_MOCK);
+    expect(maxRepetitionReached).toBe(false);
+  });
 });