fix playwright bis (#1333)

* test CI

* skip failing tests

* fix import error

* skip collatz until fixed

* skip computedvars until fixed

* longer timeout, remove skips

* longer timeout

* longer timeout

* try something different

* sigh try again

* :(

* whitelist page for faster tests and rmeove skips

* fix tests

* misfixed the test

* fix tests again

* laxer condition in test

* fix test again

* clean up code

Author: Lendemor

pcweb/whitelist.py

@@ -12,6 +12,7 @@
 
 WHITELISTED_PAGES = []
 
+
 def _check_whitelisted_path(path: str):
     if len(WHITELISTED_PAGES) == 0:
         return True

tests/conftest.py

@@ -8,5 +8,17 @@
 @pytest.fixture(scope="session")
 def reflex_web_app():
     app_root = Path(__file__).parent.parent
+    from pcweb.whitelist import WHITELISTED_PAGES
+
+    WHITELISTED_PAGES.extend(
+        [
+            "/docs/events",
+            "/docs/vars",
+            "/docs/getting-started",
+            "/docs/library/graphing",
+            "/docs/api-reference/special-events",
+        ]
+    )
+
     with AppHarness.create(root=app_root) as harness:
         yield harness

tests/test_background_events.py

@@ -1,4 +1,3 @@
-import os
 import re
 
 import pytest
@@ -14,15 +13,14 @@ def background_events_url() -> str:
     return docs.events.background_events.path
 
 
-@pytest.mark.skipif(
-    os.environ.get("GITHUB_ACTIONS") is not None, reason="Consistently fails in CI"
-)
 def test_background_events(
     reflex_web_app: AppHarness,
     page: Page,
     background_events_url: str,
 ):
     assert reflex_web_app.frontend_url is not None
+    page.set_default_timeout(60000)
+    page.set_default_navigation_timeout(60000)
 
     page.goto(reflex_web_app.frontend_url + background_events_url)
     expect(page).to_have_url(re.compile(background_events_url))
@@ -36,7 +34,8 @@ def test_background_events(
     expect(heading).to_have_text("0 /")
 
     start_button.click()
-    expect(heading).to_have_text("4 /")
+    expect(heading).to_have_text(re.compile(r"[4-7] /"))
+
     reset_button.click()
     expect(heading).to_have_text("0 /")
-    expect(heading).to_have_text("10 /")
+    expect(heading).to_have_text("10 /", timeout=12000)

tests/test_chaining_event.py

@@ -1,4 +1,3 @@
-import os
 import re
 import time
 
@@ -15,9 +14,6 @@ def chaining_event_url() -> str:
     return docs.events.chaining_events.path
 
 
-@pytest.mark.skipif(
-    os.environ.get("GITHUB_ACTIONS") is not None, reason="Consistently fails in CI"
-)
 def test_handler_from_handler(
     reflex_web_app: AppHarness,
     page: Page,