fix: linting errors

majidalfuttaim · Nov 19, 2024 · c15fa09 · c15fa09
1 parent 563afee
commit c15fa09
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 12 deletions.
diff --git a/src/components/carousel/carousel.test.ts b/src/components/carousel/carousel.test.ts
@@ -322,7 +322,7 @@ describe('<sl-carousel>', () => {
           <sl-carousel-item>Node 6</sl-carousel-item>
         </sl-carousel>
       `);
-      const expectedSlides = el.querySelectorAll('.expected')!;
+      const expectedSlides = el.querySelectorAll('.expected');
       const nextButton: HTMLElement = el.shadowRoot!.querySelector('.carousel__navigation-button--next')!;
 
       // Act
@@ -351,7 +351,7 @@ describe('<sl-carousel>', () => {
           <sl-carousel-item class="expected">Node 6</sl-carousel-item>
         </sl-carousel>
       `);
-      const expectedSlides = el.querySelectorAll('.expected')!;
+      const expectedSlides = el.querySelectorAll('.expected');
       const nextButton: HTMLElement = el.shadowRoot!.querySelector('.carousel__navigation-button--next')!;
 
       // Act

diff --git a/src/components/mf-nav-item/subnav-controller.ts b/src/components/mf-nav-item/subnav-controller.ts
@@ -125,13 +125,13 @@ export class SubnavController implements ReactiveController {
       event.stopPropagation();
       if (this.popupRef.value.active) {
         if (navItems[0] instanceof HTMLElement) {
-          navItems[0].focus();
+          navItems[0]?.focus();
         }
       } else {
         this.enableSubnav(false);
         this.host.updateComplete.then(() => {
-          if (navItems[0] instanceof HTMLElement) {
-            navItems[0].focus();
+          if (navItems![0] instanceof HTMLElement) {
+            navItems![0].focus();
           }
         });
         this.host.requestUpdate();

diff --git a/src/components/radio-group/radio-group.test.ts b/src/components/radio-group/radio-group.test.ts
@@ -217,7 +217,7 @@ describe('when submitting a form', () => {
       </form>
     `);
     const button = form.querySelector('sl-button')!;
-    const radio = form.querySelectorAll('sl-radio')[1]!;
+    const radio = form.querySelectorAll('sl-radio')[1];
     const submitHandler = sinon.spy((event: SubmitEvent) => {
       formData = new FormData(form);
 
@@ -261,7 +261,7 @@ describe('when a size is applied', () => {
         <sl-radio id="radio-2" value="2"></sl-radio>
       </sl-radio-group>
     `);
-    const [radio1, radio2] = radioGroup.querySelectorAll('sl-radio')!;
+    const [radio1, radio2] = radioGroup.querySelectorAll('sl-radio');
 
     expect(radio1.size).to.equal('large');
     expect(radio2.size).to.equal('large');
@@ -274,7 +274,7 @@ describe('when a size is applied', () => {
         <sl-radio-button id="radio-2" value="2"></sl-radio-button>
       </sl-radio-group>
     `);
-    const [radio1, radio2] = radioGroup.querySelectorAll('sl-radio-button')!;
+    const [radio1, radio2] = radioGroup.querySelectorAll('sl-radio-button');
 
     expect(radio1.size).to.equal('large');
     expect(radio2.size).to.equal('large');
@@ -287,7 +287,7 @@ describe('when a size is applied', () => {
         <sl-radio-button id="radio-2" value="2"></sl-radio-button>
       </sl-radio-group>
     `);
-    const [radio1, radio2] = radioGroup.querySelectorAll('sl-radio-button')!;
+    const [radio1, radio2] = radioGroup.querySelectorAll('sl-radio-button');
 
     expect(radio1.size).to.equal('small');
     expect(radio2.size).to.equal('small');

diff --git a/src/components/select/select.test.ts b/src/components/select/select.test.ts
@@ -318,7 +318,7 @@ describe('<sl-select>', () => {
           <sl-option value="option-3">Option 3</sl-option>
         </sl-select>
       `);
-      const secondOption = el.querySelectorAll('sl-option')[1]!;
+      const secondOption = el.querySelectorAll('sl-option')[1];
 
       expect(el.checkValidity()).to.be.true;
       expect(el.hasAttribute('data-required')).to.be.true;
@@ -347,7 +347,7 @@ describe('<sl-select>', () => {
           <sl-option value="option-3">Option 3</sl-option>
         </sl-select>
       `);
-      const secondOption = el.querySelectorAll('sl-option')[1]!;
+      const secondOption = el.querySelectorAll('sl-option')[1];
 
       expect(el.hasAttribute('data-required')).to.be.true;
       expect(el.hasAttribute('data-optional')).to.be.false;

diff --git a/src/components/tab/tab.test.ts b/src/components/tab/tab.test.ts
@@ -102,7 +102,7 @@ describe('<sl-tab>', () => {
         </sl-tab-group>
       `);
       const closeButton = tabGroup
-        .querySelectorAll('sl-tab')[0]!
+        .querySelectorAll('sl-tab')[0]
         .shadowRoot!.querySelector<SlIconButton>('[part~="close-button"]')!;
 
       const handleClose = sinon.spy();