Skip to content

tests e2e with Playwright#917

Draft
MarionGiusti wants to merge 2 commits into
masterfrom
marion_e2e_playwright
Draft

tests e2e with Playwright#917
MarionGiusti wants to merge 2 commits into
masterfrom
marion_e2e_playwright

Conversation

@MarionGiusti

Copy link
Copy Markdown
Collaborator

No description provided.

@MarionGiusti MarionGiusti marked this pull request as draft June 22, 2026 11:09
@MarionGiusti MarionGiusti force-pushed the marion_e2e_playwright branch 2 times, most recently from 70bc55d to 51b70e1 Compare June 23, 2026 14:07
@MarionGiusti MarionGiusti changed the title Marion e2e playwright tests e2e playwright Jun 23, 2026
@MarionGiusti MarionGiusti changed the title tests e2e playwright tests e2e with playwright Jun 23, 2026
@MarionGiusti MarionGiusti changed the title tests e2e with playwright tests e2e with Playwright Jun 23, 2026
@MarionGiusti MarionGiusti force-pushed the marion_e2e_playwright branch from 51b70e1 to 1cba4c6 Compare June 23, 2026 14:39
@@ -0,0 +1,27 @@
name: Playwright Tests

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The file name should not mention "playwright". The workflow is actually "e2e". "Playwright" is just the tool we use for that.

Comment thread playwright.config.ts Outdated
Comment thread playwright.config.ts Outdated
@pgiraud

pgiraud commented Jun 25, 2026

Copy link
Copy Markdown
Member

Errors like: xxxx pixels (ratio 0.0x of all image pixels) are different. in failing tests are most probably due to tiny difference in the PlanNode.vue node type (nodeName) text due to rendered font. To be investigated.

Comment thread tests/e2e/plan.spec.ts

test('switches to dark mode', async ({ page }) => {
await page.getByTitle('Switch to dark mode').click();
await expect(page).toHaveScreenshot('plan_darkMode.png');

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not sure if an argument is needed here.

@MarionGiusti MarionGiusti force-pushed the marion_e2e_playwright branch from 1cba4c6 to 5cfb456 Compare June 25, 2026 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants