Skip to content

Conversation

@mj12albert
Copy link
Member

@mj12albert mj12albert commented Dec 2, 2025

Background: mui/base-ui#2688 (comment)

It's more straightforward to use plain playwright instead of mixing it with testing library

@mj12albert mj12albert added the test label Dec 2, 2025
@mui-bot
Copy link

mui-bot commented Dec 2, 2025

Netlify deploy preview

https://deploy-preview-47410--material-ui.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 53eb885

@mj12albert mj12albert marked this pull request as ready for review December 2, 2025 17:06
@mj12albert mj12albert requested review from a team, Janpot and siriwatknp December 2, 2025 17:06
async function renderFixture(fixturePath: string) {
await page.goto(`${baseUrl}/e2e/${fixturePath}#no-dev`);
await page.goto(`${BASE_URL}/e2e/${fixturePath}#no-dev`);
await page.waitForSelector('[data-testid="testcase"]:not([aria-busy="true"])');
Copy link
Member

Choose a reason for hiding this comment

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

fyi: on regression tests we found this to be prone to race conditions as it doesn't check which test case is actually rendering

Copy link
Member

@Janpot Janpot left a comment

Choose a reason for hiding this comment

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

ok for me

@mj12albert mj12albert merged commit e3d9f3c into mui:master Dec 8, 2025
24 of 27 checks passed
@mj12albert mj12albert deleted the test/e2e-playwright branch December 8, 2025 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants