From 7d69f7a087960ee231776932540a48437eb5ba82 Mon Sep 17 00:00:00 2001
From: Yury Semikhatsky <yurys@chromium.org>
Date: Mon, 28 Nov 2022 16:01:51 -0800
Subject: [PATCH] chore: roll driver to 1.28.1 (#1133)

---
 .../playwright/TestSelectorsRole.java         | 45 +++++++++++--------
 scripts/CLI_VERSION                           |  2 +-
 2 files changed, 28 insertions(+), 19 deletions(-)

diff --git a/playwright/src/test/java/com/microsoft/playwright/TestSelectorsRole.java b/playwright/src/test/java/com/microsoft/playwright/TestSelectorsRole.java
index 5995137c..909b8429 100644
--- a/playwright/src/test/java/com/microsoft/playwright/TestSelectorsRole.java
+++ b/playwright/src/test/java/com/microsoft/playwright/TestSelectorsRole.java
@@ -153,26 +153,35 @@ public class TestSelectorsRole extends TestBase {
 
   @Test
   void shouldSupportExpanded() {
-    page.setContent("<button>Hi</button>\n" +
-      "    <button aria-expanded=\"true\">Hello</button>\n" +
-      "    <button aria-expanded=\"false\">Bye</button>");
+    page.setContent("<div role=\"treeitem\">Hi</div>\n" +
+      "    <div role=\"treeitem\" aria-expanded=\"true\">Hello</div>\n" +
+      "    <div role=\"treeitem\" aria-expanded=\"false\">Bye</div>");
     assertEquals(asList(
-      "<button aria-expanded=\"true\">Hello</button>"
-    ), page.locator("role=button[expanded]").evaluateAll("els => els.map(e => e.outerHTML)"));
+      "<div role=\"treeitem\">Hi</div>",
+      "<div role=\"treeitem\" aria-expanded=\"true\">Hello</div>",
+      "<div role=\"treeitem\" aria-expanded=\"false\">Bye</div>"
+    ), page.locator("role=treeitem").evaluateAll("els => els.map(e => e.outerHTML)"));
     assertEquals(asList(
-      "<button aria-expanded=\"true\">Hello</button>"
-    ), page.locator("role=button[expanded=true]").evaluateAll("els => els.map(e => e.outerHTML)"));
-    assertEquals(asList(
-      "<button aria-expanded=\"true\">Hello</button>"
-    ), page.getByRole(AriaRole.BUTTON, new Page.GetByRoleOptions().setExpanded(true)).evaluateAll("els => els.map(e => e.outerHTML)"));
-    assertEquals(asList(
-      "<button>Hi</button>",
-      "<button aria-expanded=\"false\">Bye</button>"
-    ), page.locator("role=button[expanded=false]").evaluateAll("els => els.map(e => e.outerHTML)"));
-    assertEquals(asList(
-      "<button>Hi</button>",
-      "<button aria-expanded=\"false\">Bye</button>"
-    ), page.getByRole(AriaRole.BUTTON, new Page.GetByRoleOptions().setExpanded(false)).evaluateAll("els => els.map(e => e.outerHTML)"));
+      "<div role=\"treeitem\">Hi</div>",
+      "<div role=\"treeitem\" aria-expanded=\"true\">Hello</div>",
+      "<div role=\"treeitem\" aria-expanded=\"false\">Bye</div>"
+    ), page.getByRole(AriaRole.TREEITEM).evaluateAll("els => els.map(e => e.outerHTML)"));
+
+    assertEquals(asList("<div role=\"treeitem\" aria-expanded=\"true\">Hello</div>"),
+      page.locator("role=treeitem[expanded]").evaluateAll("els => els.map(e => e.outerHTML)"));
+    assertEquals(asList("<div role=\"treeitem\" aria-expanded=\"true\">Hello</div>"),
+      page.locator("role=treeitem[expanded=true]").evaluateAll("els => els.map(e => e.outerHTML)"));
+    assertEquals(asList("<div role=\"treeitem\" aria-expanded=\"true\">Hello</div>"),
+      page.getByRole(AriaRole.TREEITEM, new Page.GetByRoleOptions().setExpanded(true)).evaluateAll("els => els.map(e => e.outerHTML)"));
+
+    assertEquals(asList("<div role=\"treeitem\" aria-expanded=\"false\">Bye</div>"),
+      page.locator("role=treeitem[expanded=false]").evaluateAll("els => els.map(e => e.outerHTML)"));
+    assertEquals(asList("<div role=\"treeitem\" aria-expanded=\"false\">Bye</div>"),
+      page.getByRole(AriaRole.TREEITEM, new Page.GetByRoleOptions().setExpanded(false)).evaluateAll("els => els.map(e => e.outerHTML)"));
+
+    // Workaround for expanded='none'.
+    assertEquals(asList("<div role=\"treeitem\">Hi</div>"),
+      page.locator("[role=treeitem]:not([aria-expanded])").evaluateAll("els => els.map(e => e.outerHTML)"));
   }
 
   @Test
diff --git a/scripts/CLI_VERSION b/scripts/CLI_VERSION
index cfc73071..450a687b 100644
--- a/scripts/CLI_VERSION
+++ b/scripts/CLI_VERSION
@@ -1 +1 @@
-1.28.0
+1.28.1