Skip to content

Commit 151e278

Browse files
.github+scripts: handle quotes properly for run_pytest.sh
1 parent 0f6d8c0 commit 151e278

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

.github/workflows/beam_PostCommit_Python.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,8 @@ jobs:
108108
-PuseWheelDistribution \
109109
-Pposargs="${{
110110
contains(matrix.os, 'self-hosted') &&
111-
'-m ''not require_docker_in_docker''' ||
112-
'-m ''require_docker_in_docker'''
111+
'-m (not require_docker_in_docker)' ||
112+
'-m require_docker_in_docker'
113113
}}" \
114114
-PpythonVersion=${{ matrix.python_version }} \
115115
env:

.github/workflows/beam_PreCommit_Python_ML.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ jobs:
105105
arguments: |
106106
-Pposargs="${{
107107
contains(matrix.os, 'self-hosted') &&
108-
'apache_beam/ml/ -m ''not require_docker_in_docker''' ||
109-
'apache_beam/ml/ -m ''require_docker_in_docker'''
108+
'apache_beam/ml/ -m (not require_docker_in_docker)' ||
109+
'apache_beam/ml/ -m require_docker_in_docker'
110110
}}" \
111111
-PpythonVersion=${{ matrix.python_version }}
112112
- name: Archive Python Test Results

sdks/python/scripts/run_pytest.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,8 @@ marker_for_sequential_tests="no_xdist"
7676

7777
if [[ -n $user_marker ]]; then
7878
# Combine user marker with internal markers.
79-
marker_for_parallel_tests="($user_marker) and ($marker_for_parallel_tests)"
80-
marker_for_sequential_tests="($user_marker) and ($marker_for_sequential_tests)"
79+
marker_for_parallel_tests="$user_marker and ($marker_for_parallel_tests)"
80+
marker_for_sequential_tests="$user_marker and ($marker_for_sequential_tests)"
8181
fi
8282

8383
# Parse posargs to separate pytest options from test paths.

0 commit comments

Comments
 (0)