Skip to content

Commit 190bc58

Browse files
committed
Fix xvfb-run usage error in Linux browser tests
- Ensured the xvfb-run command is on a single line to prevent bash interpretation errors from missing line continuations.
1 parent 459cf0b commit 190bc58

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

.github/actions/linux_browser_tests/action.yaml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,8 @@ runs:
5858
5959
# Wrap the execution in xvfb-run for virtual display.
6060
# We pass --gtest_output to the runner script, which we should ensure it handles.
61-
xvfb-run -a --server-args="${XVFB_SERVER_ARGS}"
62-
python3 "${RUN_BROWSER_TESTS_PY}" "${RUNNER_BINARY}"
63-
--gtest_filter="${TEST_FILTER}"
64-
--gtest_output="xml:${GITHUB_WORKSPACE}/results/cobalt_browsertests_result.xml"
65-
2>&1 | tee "${GITHUB_WORKSPACE}/results/cobalt_browsertests_log.txt"
61+
# Note: Using a single line to avoid backslash continuation issues in GitHub Actions.
62+
xvfb-run -a --server-args="${XVFB_SERVER_ARGS}" python3 "${RUN_BROWSER_TESTS_PY}" "${RUNNER_BINARY}" --gtest_filter="${TEST_FILTER}" --gtest_output="xml:${GITHUB_WORKSPACE}/results/cobalt_browsertests_result.xml" 2>&1 | tee "${GITHUB_WORKSPACE}/results/cobalt_browsertests_log.txt"
6663
6764
- name: Archive Browser Test Results
6865
if: always()

0 commit comments

Comments
 (0)