Skip to content

Commit

Permalink
test: increase testing timeout
Browse files Browse the repository at this point in the history
  • Loading branch information
@dazedbear
dazedbear committed Jun 10, 2024
1 parent 1d34c14 commit 71d2c44
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion tests/e2e/pages/notion-article-list.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ notionPages.forEach((pageName) => {
page,
}) => {
await page.goto(pagePath)
await page.waitForTimeout(1500) // wait for layout render script execution
await page.waitForTimeout(3000) // wait for layout render script execution
await expect(page).toHaveScreenshot(

Check failure on line 72 in tests/e2e/pages/notion-article-list.spec.ts

View workflow job for this annotation

GitHub Actions / Playwright E2E Tests (Stage Deployment)

[page-test-desktop] › pages/notion-article-list.spec.ts:67:9 › Page test for Notion article list page: article › regular page should render visually the same as expected 

  3) [page-test-desktop] › pages/notion-article-list.spec.ts:67:9 › Page test for Notion article list page: article › regular page should render visually the same as expected 
    Error: Screenshot comparison failed:

      684626 pixels (ratio 0.75 of all image pixels) are different.

    Expected: /Users/runner/work/dazedbear.github.io/dazedbear.github.io/tests/e2e/pages/notion-article-list.spec.ts-snapshots/article-screenshot-page-test-desktop.png
    Received: /Users/runner/work/dazedbear.github.io/dazedbear.github.io/artifacts/e2e-test-results/notion-article-list-Page-t-8ebb9-sually-the-same-as-expected-page-test-desktop/article-screenshot-actual.png
        Diff: /Users/runner/work/dazedbear.github.io/dazedbear.github.io/artifacts/e2e-test-results/notion-article-list-Page-t-8ebb9-sually-the-same-as-expected-page-test-desktop/article-screenshot-diff.png

    Call log:
      - expect.toHaveScreenshot(article-screenshot.png) with timeout 60000ms
      -   verifying given screenshot expectation
      - taking page screenshot
      -   disabled all CSS animations
      - waiting for fonts to load...
      - fonts loaded
      - 684626 pixels (ratio 0.75 of all image pixels) are different.
      - waiting 100ms before taking screenshot
      - taking page screenshot
      -   disabled all CSS animations
      - waiting for fonts to load...
      - fonts loaded
      - captured a stable screenshot
      - 684626 pixels (ratio 0.75 of all image pixels) are different.


      70 |       await page.goto(pagePath)
      71 |       await page.waitForTimeout(3000) // wait for layout render script execution
    > 72 |       await expect(page).toHaveScreenshot(
         |                          ^
      73 |         `${pageName}-screenshot.png`,
      74 |         screenshotOption
      75 |       )

        at /Users/runner/work/dazedbear.github.io/dazedbear.github.io/tests/e2e/pages/notion-article-list.spec.ts:72:26
`${pageName}-screenshot.png`,
screenshotOption
Expand Down
2 changes: 1 addition & 1 deletion tests/e2e/pages/notion-single.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ notionPages.forEach((pageName) => {
page,
}) => {
await page.goto(pagePath)
await page.waitForTimeout(1500) // wait for layout render script execution
await page.waitForTimeout(3000) // wait for layout render script execution
await expect(page).toHaveScreenshot(
`${pageName}-screenshot.png`,
screenshotOption
Expand Down
2 changes: 1 addition & 1 deletion tests/e2e/pages/static.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ staticPages.forEach((pageName) => {
page,
}) => {
await page.goto(pagePath)
await page.waitForTimeout(1500) // wait for layout render script execution
await page.waitForTimeout(3000) // wait for layout render script execution
await expect(page).toHaveScreenshot(
`${pageName}-screenshot.png`,
screenshotOption
Expand Down

0 comments on commit 71d2c44

Please sign in to comment.