Skip to content

[test] Assert on user-select that has the same value across browsers#28378

Merged
eps1lon merged 1 commit intomui:masterfrom
eps1lon:test/Tooltip/user-select-revert
Sep 16, 2021
Merged

[test] Assert on user-select that has the same value across browsers#28378
eps1lon merged 1 commit intomui:masterfrom
eps1lon:test/Tooltip/user-select-revert

Conversation

@eps1lon
Copy link
Member

@eps1lon eps1lon commented Sep 16, 2021

Safari normalizes revert to Revert but not text. So let's just use a value that's invariant across browsers. The test just needs a valid value anyway.

Fixes https://app.circleci.com/pipelines/github/mui-org/material-ui/52440/workflows/66f0e1ec-17e3-4ab1-b0f1-b703d0d6a35a/jobs/297054 which broke in #28372

Browser run: https://app.circleci.com/pipelines/github/mui-org/material-ui/52442/workflows/6a90301d-e8f1-4a1c-b6c5-4a7190382b9d/jobs/297071

@eps1lon eps1lon added the test label Sep 16, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Sep 16, 2021

No bundle size changes

Generated by 🚫 dangerJS against 2abaebb

@eps1lon eps1lon marked this pull request as ready for review September 16, 2021 08:50
@eps1lon eps1lon merged commit ffabc89 into mui:master Sep 16, 2021
@eps1lon eps1lon deleted the test/Tooltip/user-select-revert branch September 16, 2021 08:51
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.

2 participants