loculus-project / loculus

An open-source software package to power microbial genomic databases
https://loculus.org
GNU Affero General Public License v3.0
35 stars 2 forks source link

Flaky E2E test: "pages/review/index.spec.ts:53:5 › The review page › approve restricted sequences" #3018

Open corneliusroemer opened 4 days ago

corneliusroemer commented 4 days ago

Flaky, not consistently failing but there's something off here:

3) [webkit] › pages/review/index.spec.ts:53:5 › The review page › approve restricted sequences ───

    Test timeout of 60000ms exceeded.

    Error: locator.waitFor: Test timeout of 60000ms exceeded.
    Call log:
      - waiting for locator('tr').first() to be visible

      71 |         reviewPage.page.getByText(`Search returned ${testSequenceCount} sequence`);
      72 |
    > 73 |         await reviewPage.page.locator('tr').first().waitFor();
         |                                                     ^
      74 |         const rowLocator = reviewPage.page.locator('tr').getByText('LOC').first();
      75 |         await rowLocator.click();
      76 |         await expect(reviewPage.page.getByText('Restricted sequence')).toBeVisible();

        at /home/runner/work/loculus/loculus/website/tests/pages/review/index.spec.ts:73:53
corneliusroemer commented 3 days ago

Now failed twice in a row causing failure:

  Slow test file: [chromium] › pages/search/index.spec.ts (21.2s)
  Slow test file: [chromium] › pages/sequences/accession.spec.ts (19.6s)
  Consider splitting slow test files to speed up parallel execution
  1 failed
    [chromium] › pages/review/index.spec.ts:53:5 › The review page › approve restricted sequences ──
  39 passed (2.9m)