@salesforcedevs/dx-components 0.20.2 → 0.23.0-alpha.2

package/src/modules/dx/imageAndLabel/imageAndLabel.html

@@ -1,10 +1,22 @@
 <template>
     <div if:false={href} class="container">
-        <img class={imgClass} src={imgSrc} alt={imgAlt} part="image" />
+        <img
+            if:true={imgSrc}
+            class={imgClass}
+            src={imgSrc}
+            alt={imgAlt}
+            part="image"
+        />
         <span class="dx-text-label-1-dark" part="label">{label}</span>
     </div>
     <a if:true={href} href={href} class="container">
-        <img class={imgClass} src={imgSrc} alt={imgAlt} part="image" />
+        <img
+            if:true={imgSrc}
+            class={imgClass}
+            src={imgSrc}
+            alt={imgAlt}
+            part="image"
+        />
         <span class="dx-text-label-1-dark" part="label">{label}</span>
     </a>
 </template>

package/src/modules/dx/popover/__tests__/popover.test.ts

@@ -16,10 +16,6 @@ describe(TAG, () => {
             global.console = { ...global.console, error: jest.fn() };
         });
 
-        afterEach(() => {
-            jest.clearAllMocks();
-        });
-
         afterAll(() => {
             global.console = originalConsole;
         });
@@ -29,15 +25,13 @@ describe(TAG, () => {
 
             expect(console.error).not.toBeCalled();
 
-            const menuEl: HTMLElement = component.shadowRoot.querySelector(
-                ".popover-container"
-            );
+            const menuEl: HTMLElement =
+                component.shadowRoot.querySelector(".popover-container");
             expect(menuEl).not.toBeNull();
             expect(menuEl.classList).toHaveLength(1);
 
-            const popoverDiv: HTMLElement = menuEl.querySelector(
-                "div.popover"
-            )!;
+            const popoverDiv: HTMLElement =
+                menuEl.querySelector("div.popover")!;
             expect(popoverDiv).not.toBeNull();
             expect(popoverDiv.classList).toHaveLength(1);
 
@@ -47,15 +41,13 @@ describe(TAG, () => {
         it("renders with api properties", () => {
             const component = render(mockProps);
             expect(console.error).not.toBeCalled();
-            const menuEl: HTMLElement = component.shadowRoot.querySelector(
-                ".popover-container"
-            );
+            const menuEl: HTMLElement =
+                component.shadowRoot.querySelector(".popover-container");
             expect(menuEl).not.toBeNull();
             expect(menuEl.classList).toHaveLength(1);
 
-            const popoverDiv: HTMLElement = menuEl.querySelector(
-                "div.popover"
-            )!;
+            const popoverDiv: HTMLElement =
+                menuEl.querySelector("div.popover")!;
             expect(popoverDiv).not.toBeNull();
             expect(popoverDiv.classList).toHaveLength(4);
             expect(popoverDiv.classList).toContain(
@@ -74,9 +66,8 @@ describe(TAG, () => {
 
             expect(console.error).not.toBeCalled();
 
-            const popoverDiv: HTMLElement = component.shadowRoot.querySelector(
-                "div.popover"
-            );
+            const popoverDiv: HTMLElement =
+                component.shadowRoot.querySelector("div.popover");
             expect(popoverDiv.classList).toHaveLength(2);
             expect(popoverDiv.classList).toContain("popover-fullwidth");
         });
@@ -89,9 +80,8 @@ describe(TAG, () => {
                 'The "width" and "full-width" properties cannot be provided simultaneously.'
             );
 
-            const popoverDiv: HTMLElement = component.shadowRoot.querySelector(
-                "div.popover"
-            );
+            const popoverDiv: HTMLElement =
+                component.shadowRoot.querySelector("div.popover");
             expect(popoverDiv.style).not.toBeNull();
             expect(popoverDiv.classList).toHaveLength(5);
             expect(popoverDiv.classList).toContain(
@@ -120,9 +110,8 @@ describe(TAG, () => {
 
         it("doesn't render menu content before interaction", async () => {
             const component = render();
-            const content = component.shadowRoot.querySelector(
-                ".popover-content"
-            );
+            const content =
+                component.shadowRoot.querySelector(".popover-content");
 
             expect(content).toBeNull();
 
@@ -130,9 +119,8 @@ describe(TAG, () => {
 
             await Promise.resolve();
 
-            const contentAfterOpen = component.shadowRoot.querySelector(
-                ".popover-content"
-            );
+            const contentAfterOpen =
+                component.shadowRoot.querySelector(".popover-content");
 
             expect(contentAfterOpen).not.toBeNull();
 
@@ -145,9 +133,8 @@ describe(TAG, () => {
 
             const renderComponent = createRenderComponent(TAG, Popover);
             const component = renderComponent();
-            const content = component.shadowRoot.querySelector(
-                ".popover-content"
-            );
+            const content =
+                component.shadowRoot.querySelector(".popover-content");
 
             expect(content).not.toBeNull();
         });
@@ -183,9 +170,12 @@ describe(TAG, () => {
         it("doesn't fire open if passed control is disabled", () => {
             const element = render();
 
-            const { slot: controlSlot, slottedElement } = mockSlottedElement<
-                HTMLButtonElement
-            >(element, "button", "control");
+            const { slot: controlSlot, slottedElement } =
+                mockSlottedElement<HTMLButtonElement>(
+                    element,
+                    "button",
+                    "control"
+                );
             expect(controlSlot.assignedElements).toBeCalledTimes(1);
             expect(slottedElement.ariaHasPopup).toBe("true");
             expect(slottedElement.style.cursor).toBe("cursor");
@@ -237,9 +227,8 @@ describe(TAG, () => {
             const closeEvent = jest.fn();
             component.addEventListener("close", closeEvent);
 
-            const popoverDiv: HTMLElement = component.shadowRoot.querySelector(
-                ".popover"
-            );
+            const popoverDiv: HTMLElement =
+                component.shadowRoot.querySelector(".popover");
             expect(popoverDiv).not.toBeNull();
 
             expect(component.open).toBe(true);

package/src/modules/dx/sidebar/__tests__/sidebar.test.ts

@@ -25,10 +25,6 @@ const GTM_EVENT_NAME = "custEv_leftNavLinkClick";
 const TOGGLE_BUTTON_LABEL = "Toggle Sidebar";
 
 describe(TAG, () => {
-    afterEach(() => {
-        jest.clearAllMocks();
-    });
-
     describe("desktop", () => {
         beforeEach(() => {
             unmockQueryParameter();

package/src/modules/dx/sidebar/sidebar.ts

@@ -205,6 +205,7 @@ export default class Sidebar extends LightningElement {
             );
         }
         this.isSearchLoading = e.detail;
+        console.log(e)
     }
 
     private onSidebarSearchContentScroll() {

package/src/modules/dx/sidebarOld/__tests__/sidebarOld.test.ts

@@ -19,10 +19,6 @@ const GTM_EVENT_NAME = "custEv_leftNavLinkClick";
 const TOGGLE_BUTTON_LABEL = "Toggle Sidebar";
 
 describe(TAG, () => {
-    afterEach(() => {
-        jest.clearAllMocks();
-    });
-
     describe("desktop", () => {
         describe("component render", () => {
             it("should render as expanded with all the elements that sidebar includes", () => {