wrapper/tailwindcss
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Retry UI test assertions (#14165)

Browse source 
We noticed that some of the tests depending on the `hover:` variant were
flaky. After some investigation, we found that injected elements had the
`:hover` state without us explicitly hovering over the element.

To avoid this, we now set up an explicit placeholder element to move the
mouse to before running the tests.
This commit is contained in:
Philipp Spiess 2024-08-12 12:44:26 +02:00 • committed by GitHub
parent fbf877aa0a
commit 8cace0d9b5
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 10 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

10
packages/tailwindcss/tests/ui.spec.ts
View file

@ -305,6 +305,14 @@ async function render(page: Page, content: string) {
}
`)
// We noticed that some of the tests depending on the `hover:` variant were
// flaky. After some investigation, we found that injected elements had the
// `:hover` state without us explicitly hovering over the element.
//
// To avoid this, we now set up an explicit placeholder element to move the
// mouse to before running the tests.
content = `<div id="mouse-park" class="size-12"></div>${content}`
await page.setContent(content)
await page.addStyleTag({
content: optimizeCss(
@ -312,6 +320,8 @@ async function render(page: Page, content: string) {
),
})
await page.locator('#mouse-park').hover()
return {
getPropertyValue(selector: string | [string, string], property: string) {
return getPropertyValue(