Apply suggestions from code review

Author: schloerke

shiny/playwright/controller/_accordion.py

@@ -238,7 +238,10 @@ def expect_width(self, value: StyleValue, *, timeout: Timeout = None) -> None:
         _expect_style_to_have_value(self.loc_container, "width", value, timeout=timeout)
 
     def expect_open(
-        self, value: list[PatternOrStr], *, timeout: Timeout = None, **kwargs
+        self,
+        value: list[PatternOrStr],
+        *,
+        timeout: Timeout = None,
     ) -> None:
         expect_locator_values_in_list(
             page=self.page,
@@ -252,7 +255,7 @@ def expect_open(
             arr=value,
             key="data-value",
             timeout=timeout,
-            **kwargs,
+            alt_verify=True,
         )
 
     def expect_panels(
@@ -276,7 +279,7 @@ def expect_panels(
             arr=value,
             key="data-value",
             timeout=timeout,
-            **kwargs,
+            alt_verify=True,
         )
 
     def set(

shiny/playwright/controller/_expect.py

@@ -1,7 +1,5 @@
 from __future__ import annotations
 
-from time import sleep
-
 from playwright.sync_api import Locator, Page
 from playwright.sync_api import expect as playwright_expect
 
@@ -148,7 +146,7 @@ def expect_locator_values_in_list(
     is_checked: bool | MISSING_TYPE = MISSING,
     timeout: Timeout = None,
     key: str = "value",
-    **kwargs,
+    alt_verify: bool = False,
 ) -> None:
     """
     Expect the locator to contain the values in the list.

tests/playwright/shiny/components/accordion/test_accordion.py

@@ -90,7 +90,6 @@ def test_accordion(page: Page, local_app: ShinyAppProc) -> None:
             "Section F",
             "Section G",
         ],
-        alt_verify=True,
     )
     acc.expect_open(
         [
@@ -101,7 +100,6 @@ def test_accordion(page: Page, local_app: ShinyAppProc) -> None:
             "Section F",
             "Section G",
         ],
-        alt_verify=True,
     )
     # Should be uncommented once https://github.com/rstudio/bslib/issues/565 is fixed
     # output_txt_verbatim.expect_value(