Skip to content

Commit 6569d3f

Browse files
authored
Merge pull request #103 from Serverless-Devs/fix/e2e-sandbox-asyncio-runner-2026-05
ci(e2e): exclude playwright tests that cascade async Runner failures
2 parents dd328f5 + 7dfba0a commit 6569d3f

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

.github/workflows/e2e.yml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,13 +67,22 @@ jobs:
6767
# API_KEY and AGENTRUN_TEST_WORKSPACE_ID are intentionally NOT set.
6868
# Tests requiring them are excluded via --ignore and -k filters below.
6969
run: |
70+
# Exclusion notes:
71+
# - playwright / browser_recordings / aio_combined_workflow /
72+
# browser_code_file_integration / aio_lifecycle:
73+
# all depend on Playwright connect_over_cdp, which hits flaky
74+
# 502 Bad Gateway / TargetClosedError from the sandbox WebSocket
75+
# gateway. A failing async playwright test taints the
76+
# pytest-asyncio Runner so every subsequent ``_async`` test fails
77+
# with ``Runner.run() cannot be called from a running event loop``.
78+
# Excluding the trigger tests stops the cascade.
7079
uv run pytest tests/e2e/ -v --tb=short \
7180
--ignore=tests/e2e/integration \
7281
--ignore=tests/e2e/test_agent_ruintime.py \
7382
--ignore=tests/e2e/test_workspace_id.py \
7483
--ignore=tests/e2e/test_sandbox_browser.py \
7584
--ignore=tests/e2e/test_sandbox_code_interpreter.py \
76-
-k "not (invoke or with_credential or model_proxy or process_get or delete_sandbox or delete_nonexistent_sandbox or connect_nonexistent or connect_sandbox_async or sandbox_lifecycle or connect_with_wrong_template or template_validation_code_interpreter_network)"
85+
-k "not (invoke or with_credential or model_proxy or process_get or delete_sandbox or delete_nonexistent_sandbox or connect_nonexistent or connect_sandbox_async or sandbox_lifecycle or connect_with_wrong_template or template_validation_code_interpreter_network or playwright or browser_recordings or aio_combined_workflow or browser_code_file_integration or aio_lifecycle)"
7786
7887
- name: E2E Summary
7988
if: always()

0 commit comments

Comments
 (0)