test(notification): refactor assertions

Co-authored-by: Aykut Saraç <[email protected]> Signed-off-by: Ogun Babacan <[email protected]>
Trendyol · Nov 27, 2023 · cd22d6a · cd22d6a
1 parent 4ed0de6
commit cd22d6a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/src/components/alert/bl-alert.test.ts b/src/components/alert/bl-alert.test.ts
@@ -153,14 +153,11 @@ describe("Slot", () => {
     );
 
     const actionSlot = el.shadowRoot!.querySelector('slot[name="action"]') as HTMLSlotElement;
-    const actionElement = actionSlot?.assignedElements()[0] as HTMLElement;
+    const [actionElement, actionSpanElement] = actionSlot.assignedElements();
 
-    expect(actionElement).to.exist;
+    expect(actionElement as HTMLElement).to.exist;
     expect(actionElement.tagName).to.eq("BL-BUTTON");
-
-    const actionSpanElement = actionSlot?.assignedElements()[1] as HTMLElement;
-
-    expect(actionSpanElement).to.not.exist;
+    expect(actionSpanElement as HTMLElement).to.not.exist;
 
     const actionSecondarySlot = el.shadowRoot!.querySelector(
       'slot[name="action-secondary"]'

diff --git a/src/components/notification/bl-notification.ts b/src/components/notification/bl-notification.ts
@@ -148,7 +148,6 @@ export default class BlNotification extends LitElement {
     // Allow only moving up
     const movedY = Math.min(currentY - this.touchStartY, 0);
 
-    console.log(movedY);
     currentTarget.style.transform = `translateY(${movedY}px)`;
   }