Apply suggestions from code review

Co-authored-by: Maria Hutt <[email protected]>

Author: OS-martacarlos

core/src/components/input/test/basic/input.e2e.ts

@@ -63,7 +63,7 @@ configs().forEach(({ title, screenshot, config }) => {
       });
     });
     test.describe('input click behaviors', () => {
-      test('should not have visual regressions', async ({ page }) => {
+      test('should not have visual regressions when user double clicks', async ({ page }) => {
         await page.setContent(
           `
           <ion-input
@@ -75,15 +75,12 @@ configs().forEach(({ title, screenshot, config }) => {
           config
         );
         const input = page.locator('ion-input');
-        input.dblclick();
+        const nativeInput = input.locator('input');
+        await nativeInput.dblclick();
         // Validates the display of an input with a clear button.
         await expect(input).toHaveScreenshot(screenshot(`input-dbclick`));
       });
-      test('should not have visual regressions with stacked label', async ({ page, browserName }) => {
-        if (browserName === 'firefox') {
-          //TODO: Figure out why it's not working for firefox
-          test.skip();
-        }
+      test('should not have visual regressions with stacked label when user double clicks', async ({ page, browserName }) => {
         await page.setContent(
           `
           <ion-input
@@ -96,7 +93,8 @@ configs().forEach(({ title, screenshot, config }) => {
           config
         );
         const input = page.locator('ion-input');
-        input.dblclick();
+        const nativeInput = input.locator('input');
+        await nativeInput.dblclick();
         // Validates the display of an input with a clear button.
         await expect(input).toHaveScreenshot(screenshot(`input-dbclick-stacked`));
       });
@@ -224,11 +222,7 @@ configs({ modes: ['ionic-md'] }).forEach(({ title, screenshot, config }) => {
       await expect(container).toHaveScreenshot(screenshot(`input-clear-button-focused`));
     });
 
-    test('should not have visual regressions when user dbclicks', async ({ page, browserName }) => {
-      if (browserName === 'firefox' || browserName === 'webkit') {
-        //TODO: Figure out why it's not working for webkit & firefox
-        test.skip();
-      }
+    test('should not have visual regressions when user double clicks', async ({ page, browserName }) => {
       await page.setContent(
         `
           <ion-input
@@ -241,8 +235,8 @@ configs({ modes: ['ionic-md'] }).forEach(({ title, screenshot, config }) => {
         config
       );
       const input = page.locator('ion-input');
-      await input.dblclick();
-      await page.waitForChanges();
+      const nativeInput = input.locator('input');
+      await nativeInput.dblclick();
       await expect(input).toHaveScreenshot(screenshot(`input-dbclick`));
     });
   });