Skip to content

Commit

Permalink
remove time delays
Browse files Browse the repository at this point in the history
  • Loading branch information
John Pratt committed Jan 16, 2025
1 parent a88f83a commit 69c474b
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions nala/features/personalization/target-on.test.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -18,21 +18,18 @@ test(`${features[0].name},${features[0].tags}`, async ({ page, baseURL }) => {
await test.step('step-1: verify default test page', async () => {
console.info(`[Test Page]: ${defaultURL}`);
await page.goto(defaultURL);
await page.waitForTimeout(1000);
await expect(marquee.headingXL).toHaveText('Heading XL Marquee standard medium left');

Check failure on line 21 in nala/features/personalization/target-on.test.js

View workflow job for this annotation

GitHub Actions / Running Nala E2E UI Tests (20.x)

[milo-live-chromium] › features/personalization/target-on.test.js:12:5 › @check for running Target tests 

  1) [milo-live-chromium] › features/personalization/target-on.test.js:12:5 › @check for running Target tests,@targeton0 @smoke @regression @milo  

    Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
    Error: Timed out 5000ms waiting for expect(locator).toHaveText(expected)

    Locator: locator('.marquee').first().locator('.heading-xl').first()
    Expected string: "Heading XL Marquee standard medium left"
    Received: <element(s) not found>
    Call log:
      - expect.toHaveText with timeout 5000ms
      - waiting for locator('.marquee').first().locator('.heading-xl').first()


      19 |     console.info(`[Test Page]: ${defaultURL}`);
      20 |     await page.goto(defaultURL);
    > 21 |     await expect(marquee.headingXL).toHaveText('Heading XL Marquee standard medium left');
         |                                     ^
      22 |   });
      23 |
      24 |   await test.step('step-2: verify metadata test page', async () => {

        at /home/runner/work/milo/milo/nala/features/personalization/target-on.test.js:21:37
        at /home/runner/work/milo/milo/nala/features/personalization/target-on.test.js:18:3
});

await test.step('step-2: verify metadata test page', async () => {
console.info(`[Test Page]: ${metadataURL}`);
await page.goto(metadataURL);
await page.waitForTimeout(1000);
await expect(marquee.headingXL).toHaveText('Target is running');

Check failure on line 27 in nala/features/personalization/target-on.test.js

View workflow job for this annotation

GitHub Actions / Running Nala E2E UI Tests (20.x)

[milo-live-chromium] › features/personalization/target-on.test.js:12:5 › @check for running Target tests 

  1) [milo-live-chromium] › features/personalization/target-on.test.js:12:5 › @check for running Target tests,@targeton0 @smoke @regression @milo  
    Error: Timed out 5000ms waiting for expect(locator).toHaveText(expected)

    Locator: locator('.marquee').first().locator('.heading-xl').first()
    Expected string: "Target is running"
    Received string: "Heading XL Marquee standard medium left"
    Call log:
      - expect.toHaveText with timeout 5000ms
      - waiting for locator('.marquee').first().locator('.heading-xl').first()
      -   locator resolved to <h2 class="heading-xl" id="heading-xl-marquee-standard-medium-left">…</h2>
      -   unexpected value "Heading XL Marquee standard medium left"


      25 |     console.info(`[Test Page]: ${metadataURL}`);
      26 |     await page.goto(metadataURL);
    > 27 |     await expect(marquee.headingXL).toHaveText('Target is running');
         |                                     ^
      28 |   });
      29 |
      30 |   await test.step('step-3: verify parameter test page', async () => {

        at /home/runner/work/milo/milo/nala/features/personalization/target-on.test.js:27:37
        at /home/runner/work/milo/milo/nala/features/personalization/target-on.test.js:24:3
});

await test.step('step-3: verify parameter test page', async () => {
console.info(`[Test Page]: ${parameterURL}`);
await page.goto(parameterURL);
await page.waitForTimeout(1000);
await expect(marquee.headingXL).toHaveText('Target is running');
});
});

0 comments on commit 69c474b

Please sign in to comment.