Skip to content

Commit 8f6c099

Browse files
committed
fix one more test
1 parent c2727df commit 8f6c099

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

playwright/src/test/java/com/microsoft/playwright/TestPageAddLocatorHandler.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import com.microsoft.playwright.options.AriaRole;
2020
import com.microsoft.playwright.options.WaitForSelectorState;
2121
import org.junit.jupiter.api.Test;
22-
import org.junit.jupiter.api.Disabled;
2322

2423
import static com.microsoft.playwright.assertions.PlaywrightAssertions.assertThat;
2524
import static org.junit.jupiter.api.Assertions.*;
@@ -255,7 +254,6 @@ public void shouldWaitForHiddenByDefault() {
255254
}
256255

257256
@Test
258-
@Disabled("need to debug why this is failing")
259257
public void shouldWaitForHiddenByDefault2() {
260258
Page page = browser.newPage();
261259
page.navigate(server.PREFIX + "/input/handle-locator.html");
@@ -273,7 +271,7 @@ public void shouldWaitForHiddenByDefault2() {
273271

274272
PlaywrightException e = assertThrows(PlaywrightException.class, () -> page.locator("#target").click(new Locator.ClickOptions().setTimeout(3_000)));
275273
assertEquals(0, (int) page.evaluate("window.clicked"));
276-
assertThat(page.locator("#interstitial")).isVisible();
274+
assertTrue(page.locator("#interstitial").isVisible());
277275
assertEquals(1, called[0]);
278276
assertTrue(e.getMessage().contains("locator handler has finished, waiting for getByRole(AriaRole.BUTTON, new Page.GetByRoleOptions().setName(\"close\")) to be hidden"), e.getMessage());
279277
}

0 commit comments

Comments
 (0)