@blaze-cms/react-page-builder 0.146.0-node18.1 → 0.146.0-rc-eagle.0

package/tests/unit/src/components/List/helpers/__snapshots__/build-pagination-items.test.js.snap

@@ -10,7 +10,7 @@ Array [
     href="url?pb%5Blist-1%5D%5Bpage%5D=1"
     name=""
     onClick={[MockFunction]}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -29,7 +29,7 @@ Array [
     href="url?pb%5Blist-1%5D%5Bpage%5D=3"
     name=""
     onClick={[MockFunction]}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -43,7 +43,7 @@ Array [
     href="url?pb%5Blist-1%5D%5Bpage%5D=4"
     name=""
     onClick={[MockFunction]}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -62,7 +62,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=1"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -76,7 +76,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=2"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -90,7 +90,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=3"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -104,7 +104,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=4"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -123,7 +123,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=6"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -137,7 +137,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=7"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -151,7 +151,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=8"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -165,7 +165,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=9"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -179,7 +179,7 @@ Array [
     href="url?pb%5Blist-3%5D%5Bpage%5D=10"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -198,7 +198,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=4"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -212,7 +212,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=5"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -226,7 +226,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=6"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -240,7 +240,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=7"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -259,7 +259,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=9"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -273,7 +273,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=10"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -287,7 +287,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=11"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -301,7 +301,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=12"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >
@@ -315,7 +315,7 @@ Array [
     href="url?pb%5Blist-2%5D%5Bpage%5D=13"
     name=""
     onClick={null}
-    scroll={true}
+    scroll={false}
     shallow={true}
     title=""
   >

package/tests/unit/src/components/List/helpers/build-query-booster.test.js

@@ -13,7 +13,7 @@ describe('buildQueryBooster helper function', () => {
     );
     expect(noRecencyBoostProperty).toEqual({
       rawQueryStringified:
-        '{"function_score":{"query":{"test":"t"},"functions":[{"gauss":{"date":{"origin":"now","scale":"30d","offset":"7d","decay":0.5}}}],"boost_mode":"multiply"}}'
+        '{"function_score":{"query":{"test":"t"},"functions":[{"exp":{"date":{"scale":"14d","decay":0.99,"offset":"7d"}}},{"weight":0.01}],"score_mode":"sum","boost_mode":"multiply"}}'
     });
   });
 
@@ -27,7 +27,7 @@ describe('buildQueryBooster helper function', () => {
     );
     expect(noRecencyBoostProperty).toEqual({
       rawQueryStringified:
-        '{"function_score":{"query":{"test":"t"},"functions":[{"gauss":{"test":{"origin":"now","scale":"30d","offset":"7d","decay":0.5}}},{"gauss":{"date":{"origin":"now","scale":"30d","offset":"7d","decay":0.5}}}],"boost_mode":"multiply"}}'
+        '{"function_score":{"query":{"test":"t"},"functions":[{"gauss":{"test":{"origin":"now","scale":"30d","offset":"7d","decay":0.5}}},{"exp":{"date":{"scale":"14d","decay":0.99,"offset":"7d"}}},{"weight":0.01}],"score_mode":"sum","boost_mode":"multiply"}}'
     });
   });
 });

package/tests/unit/src/components/Loading/__snapshots__/Loading.test.js.snap

@@ -18,21 +18,9 @@ exports[`Loading component should render dots loader if dots string is passed 1`
 </DocumentFragment>
 `;
 
-exports[`Loading component should render spinner loader if no value or not supported value is passed to type 1`] = `
-<DocumentFragment>
-  <div
-    class="blaze-loading__spinner"
-  />
-</DocumentFragment>
-`;
+exports[`Loading component should render spinner loader if no value or not supported value is passed to type 1`] = `<DocumentFragment />`;
 
-exports[`Loading component should render spinner loader if no value or not supported value is passed to type 2`] = `
-<DocumentFragment>
-  <div
-    class="blaze-loading__spinner"
-  />
-</DocumentFragment>
-`;
+exports[`Loading component should render spinner loader if no value or not supported value is passed to type 2`] = `<DocumentFragment />`;
 
 exports[`Loading component should render spinner loader if spinner string is passed 1`] = `
 <DocumentFragment>

package/tests/unit/src/components/MenuItem/MenuItem.test.js

@@ -15,6 +15,10 @@ jest.mock('@blaze-cms/utils-handlebars', () => ({
   useStringTemplate: jest.fn((parent, [title]) => ({ loading: false, data: [title] }))
 }));
 
+jest.mock('next/router', () => ({
+  useRouter: () => ({ asPath: '/' })
+}));
+
 const componentProps = {
   id: 'id',
   name: 'mock name',
@@ -34,7 +38,7 @@ const componentProps = {
   },
   modifier: 'menu-item-mod',
   children: [
-    <div className="child_1" key="1" component={{ type: 'menuitem' }}>
+    <div className="child_1" key="1" data-component='{"type":"menuitem"}'>
       menu child
     </div>
   ],

package/tests/unit/src/components/MenuItem/MenuItemRender.test.js

@@ -7,8 +7,16 @@ import { render, fireEvent } from '@testing-library/react';
 import '@testing-library/jest-dom/extend-expect';
 import MenuRender from '../../../../../src/components/MenuItem/MenuItemRender';
 
+const MENU_ITEM_CHILDREN_CLASS = 'menu--item-children';
+
+let mockAsPathValue = '/';
+
+jest.mock('next/router', () => ({
+  useRouter: () => ({ asPath: mockAsPathValue })
+}));
+
 jest.mock('@blaze-cms/utils-handlebars', () => ({
-  useStringTemplate: jest.fn((parent, [title]) => ({ loadig: false, data: [title] }))
+  useStringTemplate: jest.fn((parent, [title]) => ({ loading: false, data: [title] }))
 }));
 
 describe('MenuRender component', () => {
@@ -52,4 +60,47 @@ describe('MenuRender component', () => {
     fireEvent.click(parentElement);
     expect(childElement).toBeInTheDocument();
   });
+
+  it('closes menu when asPath changes for hover event', () => {
+    const { getByText, container, rerender } = render(
+      <MenuRender eventType="hover" text="Parent">
+        <div data-testid="child">Child</div>
+      </MenuRender>
+    );
+
+    fireEvent.mouseEnter(getByText('Parent'));
+    let childContainer = container.querySelector(`.${MENU_ITEM_CHILDREN_CLASS}`);
+    expect(childContainer).not.toHaveClass('hidden');
+
+    mockAsPathValue = '/new-route';
+    rerender(
+      <MenuRender eventType="hover" text="Parent">
+        <div data-testid="child">Child</div>
+      </MenuRender>
+    );
+
+    childContainer = container.querySelector(`.${MENU_ITEM_CHILDREN_CLASS}`);
+    expect(childContainer).toHaveClass('hidden');
+  });
+
+  it('toggles display of children when chevron icon is clicked', () => {
+    const { container } = render(
+      <MenuRender eventType="click" text="Parent">
+        <div data-testid="child">Child</div>
+      </MenuRender>
+    );
+    let childContainer = container.querySelector(`.${MENU_ITEM_CHILDREN_CLASS}`);
+    expect(childContainer).toHaveClass('hidden');
+
+    const chevronIcon = container.querySelector('.menu--item--link--icon');
+    expect(chevronIcon).toBeInTheDocument();
+
+    fireEvent.click(chevronIcon);
+    childContainer = container.querySelector(`.${MENU_ITEM_CHILDREN_CLASS}`);
+    expect(childContainer).not.toHaveClass('hidden');
+
+    fireEvent.click(chevronIcon);
+    childContainer = container.querySelector(`.${MENU_ITEM_CHILDREN_CLASS}`);
+    expect(childContainer).toHaveClass('hidden');
+  });
 });

package/tests/unit/src/components/MenuItem/__snapshots__/MenuItem.test.js.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`MenuItem component should render a MenuItem based on passed parameters 1`] = `
 <DocumentFragment>
@@ -17,6 +17,7 @@ exports[`MenuItem component should render a MenuItem based on passed parameters
       <i
         class="menu--item--link--icon"
         role="button"
+        tabindex="0"
       >
         <svg
           fill="currentColor"
@@ -38,7 +39,7 @@ exports[`MenuItem component should render a MenuItem based on passed parameters
     >
       <div
         class="child_1"
-        component="[object Object]"
+        data-component="{\\"type\\":\\"menuitem\\"}"
       >
         menu child
       </div>

package/tests/unit/src/components/MenuItem/helpers/inject-helper-into-template.test.js

@@ -41,4 +41,48 @@ describe('injectHelperIntoTemplate', () => {
     const result = injectHelperIntoTemplate(null, 'url_encode');
     expect(result).toBeNull();
   });
+
+  it('should ignore {{else}} pattern and not inject helper', () => {
+    const template = '{{#if condition}}true{{else}}false{{/if}}';
+    const expected = '{{#if condition}}true{{else}}false{{/if}}';
+    const result = injectHelperIntoTemplate(template, 'url_encode');
+    expect(result).toBe(expected);
+  });
+
+  it('should ignore block helpers starting with # and not inject helper', () => {
+    const template = '{{#if currentParent.active}}Active{{/if}}';
+    const expected = '{{#if currentParent.active}}Active{{/if}}';
+    const result = injectHelperIntoTemplate(template, 'url_encode');
+    expect(result).toBe(expected);
+  });
+
+  it('should ignore closing helpers starting with / and not inject helper', () => {
+    const template = '{{/each}}{{/if}}{{/unless}}';
+    const expected = '{{/each}}{{/if}}{{/unless}}';
+    const result = injectHelperIntoTemplate(template, 'url_encode');
+    expect(result).toBe(expected);
+  });
+
+  it('should inject helper into regular placeholders but ignore control flow statements', () => {
+    const template = '{{#if active}}{{currentParent.name}}{{else}}{{defaultName}}{{/if}}';
+    const expected =
+      '{{#if active}}{{url_encode currentParent.name}}{{else}}{{url_encode defaultName}}{{/if}}';
+    const result = injectHelperIntoTemplate(template, 'url_encode');
+    expect(result).toBe(expected);
+  });
+
+  it('should handle complex template with mixed block helpers and regular placeholders', () => {
+    const template = '{{#each items}}{{name}} - {{price}}{{else}}No items{{/each}}';
+    const expected =
+      '{{#each items}}{{url_encode name}} - {{url_encode price}}{{else}}No items{{/each}}';
+    const result = injectHelperIntoTemplate(template, 'url_encode');
+    expect(result).toBe(expected);
+  });
+
+  it('should ignore {{{else}}} in triple-stash expressions', () => {
+    const template = '{{{#if condition}}}{{{value}}}{{{else}}}default{{{/if}}}';
+    const expected = '{{{#if condition}}}{{{url_encode value}}}{{{else}}}default{{{/if}}}';
+    const result = injectHelperIntoTemplate(template, 'url_encode');
+    expect(result).toBe(expected);
+  });
 });

package/tests/unit/src/components/MenuItem/mocks.js

@@ -9,7 +9,7 @@ const MOCKED_MENUITEMS_PROPS = {
   entity: 'page',
   modifier: 'menu-item-mod',
   children: [
-    <div className="child_1" key="1" component={{ type: 'menuitem' }}>
+    <div className="child_1" key="1" data-component='{"type":"menuitem"}'>
       menu child
     </div>
   ],

package/tests/unit/src/components/Modal/Modal.test.js

@@ -13,7 +13,7 @@ const props = {
   },
   children: [
     <div className="child_1">some stuff</div>,
-    <div className="child_2" modifier="child-modi">
+    <div className="child_2" data-modifier="child-modifier">
       another child
     </div>
   ]

package/tests/unit/src/components/Modal/__snapshots__/Modal.test.js.snap

@@ -9,7 +9,7 @@ exports[`Modal component should render without throwing an error and match snaps
   </div>
   <div
     class="child_2"
-    modifier="child-modi"
+    data-modifier="child-modifier"
   >
     another child
   </div>

package/tests/unit/src/components/SearchContent/SearchContent.test.js

@@ -12,8 +12,16 @@ jest.mock('@apollo/client', () => ({
   useQuery: jest.fn(() => ({ data: {}, loading: false, error: null }))
 }));
 
+const mockPush = jest.fn();
+const mockOn = jest.fn();
+const mockOff = jest.fn();
+
 jest.mock('next/router', () => ({
-  useRouter: jest.fn(() => ({ asPath: 'test-url', events: { on: jest.fn(), off: jest.fn() } }))
+  useRouter: jest.fn(() => ({
+    asPath: 'test-url',
+    push: mockPush,
+    events: { on: mockOn, off: mockOff }
+  }))
 }));
 
 const SEARCH_INPUT_TEST_ID = 'search-content-input';
@@ -54,4 +62,27 @@ describe('SearchContent', () => {
     fireEvent.click(closeIcon);
     await waitFor(() => expect(queryByTestId(SEARCH_INPUT_TEST_ID)).toBeNull());
   });
+
+  it('should trigger autosuggest again after navigation and typing', async () => {
+    const { getByTestId, queryByTestId } = render(
+      <MockedProvider mocks={[]} addTypename={false}>
+        <SearchContent entities={entities} />
+      </MockedProvider>
+    );
+
+    const input = getByTestId(SEARCH_INPUT_TEST_ID);
+
+    fireEvent.change(input, { target: { value: 'novartis' } });
+
+    await waitFor(() => expect(queryByTestId('search-content-results')).toBeTruthy());
+
+    fireEvent.keyDown(input, { key: 'Enter', code: 'Enter' });
+
+    const routeChangeHandler = mockOn.mock.calls[0][1];
+    routeChangeHandler();
+
+    fireEvent.change(input, { target: { value: 'blaze' } });
+
+    await waitFor(() => expect(queryByTestId('search-content-results')).toBeTruthy());
+  });
 });

package/tests/unit/src/components/SearchFilter/SearchFilter/FiltersList.test.js

@@ -1,3 +1,6 @@
+/**
+ * @jest-environment jsdom
+ */
 import '@testing-library/jest-dom/extend-expect';
 import FiltersList from '../../../../../../src/components/SearchFilter/SearchFilter/FiltersList';
 

package/tests/unit/src/components/SearchFilter/SearchFilter/__snapshots__/CloseMobileForm.test.js.snap

@@ -6,7 +6,7 @@ exports[`CloseMobileForm component should render without throwing error and matc
     class="filter__header filter__header--mobile"
   >
     <p>
-      Search refinement
+      MOBILE_REFINEMENT_TITLE
     </p>
     <i
       aria-label="Close"

package/tests/unit/src/components/SearchFilter/SearchFilter/__snapshots__/MobileFormToolbar.test.js.snap

@@ -6,19 +6,19 @@ exports[`MobileFormToolbar component should render without throwing error and ma
     class="filter__refine filter__refine--mobile-open"
   >
     <div
-      aria-label="Reset search filters"
+      aria-label="RESET_ALL_FILTERS"
       class="reset"
       data-testid="reset-button"
       role="button"
     >
-      Reset all filters
+      RESET_ALL_FILTERS
     </div>
     <button
       class="search"
       form="xxx"
       type="submit"
     >
-      Search
+      SEARCH_MOBILE_BUTTON_TEXT
     </button>
   </div>
 </DocumentFragment>

package/tests/unit/src/components/SearchFilter/SearchFilter/__snapshots__/ResetDesktopForm.test.js.snap

@@ -6,7 +6,7 @@ exports[`ResetDesktopForm component should render without throwing error and mat
     class="filter__reset"
   >
     <button
-      aria-label="Reset search filters"
+      aria-label="RESET_ALL_FILTERS"
       class="icon-button icon-button--reset"
       data-testid="reset-icon"
       title="reset--button"
@@ -32,7 +32,7 @@ exports[`ResetDesktopForm component should render without throwing error and mat
       data-testid="reset-button"
       role="button"
     >
-      Reset all filters
+      RESET_ALL_FILTERS
     </div>
   </div>
 </DocumentFragment>