test(angular): minor test fixes

Author: brandyscarney

packages/angular/test/base/e2e/src/lazy/inputs.spec.ts

@@ -89,17 +89,17 @@ test.describe('Inputs', () => {
   });
 
   test('should update values when erasing input', async ({ page }) => {
-    // Focus the input and use global keyboard to press backspace
-    await page.locator('ion-input').nth(0).locator('input').focus();
-    await page.keyboard.press('Backspace');
+    // Focus the input and press backspace to remove last character
+    await page.locator('ion-input').nth(0).locator('input').click();
+    await page.locator('ion-input').nth(0).locator('input').press('Backspace');
     // Check mirror element reflects the change
     await expect(page.locator('ion-input').nth(1)).toHaveJSProperty('value', 'some tex');
     // Check note text (Angular binding)
     await expect(page.locator('#input-note')).toHaveText('some tex');
 
-    // Focus the last OTP input and use global keyboard to press backspace
-    await page.locator('ion-input-otp input').last().focus();
-    await page.keyboard.press('Backspace');
+    // Focus the last OTP input and press backspace
+    await page.locator('ion-input-otp input').last().click();
+    await page.locator('ion-input-otp input').last().press('Backspace');
     // Check mirror element reflects the change
     await expect(page.locator('ion-input-otp').nth(1)).toHaveJSProperty('value', '123');
     // Check note text (Angular binding)

packages/angular/test/base/e2e/src/lazy/providers.spec.ts

@@ -15,7 +15,7 @@ test.describe('Providers', () => {
     await expect(page.locator('#is-desktop')).toHaveText('true');
     await expect(page.locator('#is-mobile')).toHaveText('false');
     await expect(page.locator('#keyboard-height')).toHaveText('12345');
-    await expect(page.locator('#query-params')).toHaveText('firstParam: null, firstParam: null');
+    await expect(page.locator('#query-params')).toHaveText('firstParam: null, secondParam: null');
   });
 
   test('should detect testing mode', async ({ page }) => {
@@ -27,7 +27,7 @@ test.describe('Providers', () => {
   test('should get query params', async ({ page }) => {
     await page.goto('/lazy/providers?firstParam=abc&secondParam=true');
 
-    await expect(page.locator('#query-params')).toHaveText('firstParam: abc, firstParam: true');
+    await expect(page.locator('#query-params')).toHaveText('firstParam: abc, secondParam: true');
   });
 
   // https://github.com/ionic-team/ionic-framework/issues/28337

packages/angular/test/base/e2e/src/lazy/range.spec.ts

@@ -6,7 +6,7 @@ test.describe('Range', () => {
   });
 
   test('should have form control initial value', async ({ page }) => {
-    await expect(page.locator('ion-range input')).toHaveValue('5');
+    await expect(page.locator('ion-range')).toHaveJSProperty('value', 5);
   });
 
   test('should reflect Ionic form control status classes', async ({ page }) => {

packages/angular/test/base/e2e/src/standalone/router-link.spec.ts

@@ -30,7 +30,7 @@ test.describe('RouterLink', () => {
   // checking here that it is only removed from Ionic components.
   // https://github.com/ionic-team/ionic-framework/issues/20632
   test('should have tabindex="0" with a native span', async ({ page }) => {
-    await expect(page.locator('span[routerlink="/standalone/popover"]')).toHaveAttribute('tabindex', '0');
+    await expect(page.locator('span[routerlink]')).toHaveAttribute('tabindex', '0');
   });
 
   test('should not have tabindex set with an ionic button', async ({ page }) => {

packages/angular/test/base/src/app/lazy/providers/providers.component.ts

@@ -87,7 +87,7 @@ export class ProvidersComponent {
     });
     const firstQuery = platform.getQueryParam('firstParam');
     const secondQuery = platform.getQueryParam('secondParam');
-    this.queryParams = `firstParam: ${firstQuery}, firstParam: ${secondQuery}`;
+    this.queryParams = `firstParam: ${firstQuery}, secondParam: ${secondQuery}`;
 
     this.isDesktop = platform.is('desktop');
     this.isMobile = platform.is('mobile');