Skip to content

chore(e2e): deflake known rare console error about CORS policy - #6917

Open
corneliusroemer wants to merge 1 commit into
mainfrom
e2e-deflake-cors
Open

chore(e2e): deflake known rare console error about CORS policy#6917
corneliusroemer wants to merge 1 commit into
mainfrom
e2e-deflake-cors

Conversation

@corneliusroemer

@corneliusroemer corneliusroemer commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Rare console errors that are not due to clear underlying problems. Possibly related to LAPIS or SILO dropping requests when switching to new dataVersion.

Console errors like this will no longer fail reducing flakiness:

Error: Unexpected console error: Access to XMLHttpRequest at 'http://10.1.1.126:8080/ebola-sudan/sample/aggregated' from origin 'http://localhost:3000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

🚀 Preview: Add preview label to enable

Rare console errors that are not due to clear underlying problems. Possibly related to LAPIS or SILO dropping requests when switching to new dataVersion.

Console errors like this will no longer fail reducing flakiness:

```
Error: Unexpected console error: Access to XMLHttpRequest at 'http://10.1.1.126:8080/ebola-sudan/sample/aggregated' from origin 'http://localhost:3000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
```

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to reduce Playwright integration-test flakiness by treating a specific rare CORS-related console error as “harmless”, so it no longer fails tests when it occurs during runs.

Changes:

  • Adds an additional substring to the console-warning filter allowlist so a known CORS console error doesn’t fail Chromium-based e2e tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread integration-tests/tests/fixtures/console-warnings.fixture.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e E2E test related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants