Merge pull request #282 from plotly/andrew/test_timing

Andrew/test timing

Author: ayjayt

tests/conftest.py

@@ -1,10 +1,11 @@
 import asyncio
 import logging
 
-import choreographer as choreo
 import logistro
 import pytest
 import pytest_asyncio
+
+import choreographer as choreo
 from choreographer import errors
 
 _logger = logistro.getLogger(__name__)
@@ -95,7 +96,7 @@ async def wrapped_test_fn(*args, **kwargs):
 
 def pytest_configure():
     # change this by command line TODO
-    pytest.default_timeout = 20  # type: ignore[reportAttributeAccessIssue]
+    pytest.default_timeout = 24  # type: ignore[reportAttributeAccessIssue]
 
 
 # pytest shuts down its capture before logging/threads finish

tests/test_browser.py

@@ -137,7 +137,10 @@ async def test_browser_send_command_with_perf(browser):
 
         # Validate timing makes sense (write_start <= write_end <= read_end)
         write_start, write_end, read_end = perf
-        assert write_start <= write_end <= read_end
+        assert write_start <= write_end
+        assert write_start <= read_end
+        # read_end is tough because it happens on a different processor
+        # and it might actually be somewhat skewed
 
         perfs.append(perf)
         await asyncio.sleep(0.1)