Skip to content

Commit 02552cc

Browse files
committed
[py] Fix consolidate workflows
1 parent 5c53f9a commit 02552cc

File tree

1 file changed

+8
-41
lines changed

1 file changed

+8
-41
lines changed

.github/workflows/ci-python.yml

Lines changed: 8 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -66,28 +66,15 @@ jobs:
6666
include:
6767
- os: ubuntu
6868
- os: macos
69-
with:
70-
name: Unit Tests (${{ matrix.os }})
71-
os: ${{ matrix.os }}
72-
cache-key: python-unit-test-${{ matrix.os }}
73-
run: |
74-
bazel test //py:unit
75-
76-
unit-tests-windows:
77-
name: Unit Tests
78-
needs: build
79-
uses: ./.github/workflows/bazel.yml
80-
strategy:
81-
fail-fast: false
82-
matrix:
83-
include:
8469
- os: windows
8570
with:
8671
name: Unit Tests (${{ matrix.os }})
8772
os: ${{ matrix.os }}
8873
cache-key: python-unit-test-${{ matrix.os }}
8974
run: |
90-
fsutil 8dot3name set 0
75+
if [[ "${{ runner.os }}" == "Windows" ]]; then
76+
fsutil 8dot3name set 0
77+
fi
9178
bazel test //py:unit
9279
9380
remote-tests:
@@ -120,24 +107,6 @@ jobs:
120107
os: ubuntu
121108
- browser: firefox
122109
os: ubuntu
123-
with:
124-
name: Integration Tests (${{ matrix.browser }}, ${{ matrix.os }})
125-
browser: ${{ matrix.browser }}
126-
os: ${{ matrix.os }}
127-
cache-key: py-browser-${{ matrix.browser }}
128-
run: |
129-
bazel test --local_test_jobs 1 --flaky_test_attempts 3 --pin_browsers=true \
130-
//py:common-${{ matrix.browser }}-bidi \
131-
//py:test-${{ matrix.browser }}
132-
133-
browser-tests-windows:
134-
name: Browser Tests
135-
needs: build
136-
uses: ./.github/workflows/bazel.yml
137-
strategy:
138-
fail-fast: false
139-
matrix:
140-
include:
141110
- browser: chrome
142111
os: windows
143112
- browser: edge
@@ -150,10 +119,10 @@ jobs:
150119
os: ${{ matrix.os }}
151120
cache-key: py-browser-${{ matrix.browser }}
152121
run: |
153-
fsutil 8dot3name set 0
154-
bazel test --local_test_jobs 1 --flaky_test_attempts 3 --pin_browsers=true \
155-
//py:common-${{ matrix.browser }}-bidi \
156-
//py:test-${{ matrix.browser }}
122+
if [[ "${{ runner.os }}" == "Windows" ]]; then
123+
fsutil 8dot3name set 0
124+
fi
125+
bazel test --local_test_jobs 1 --flaky_test_attempts 3 --pin_browsers=true //py:common-${{ matrix.browser }}-bidi //py:test-${{ matrix.browser }}
157126
158127
safari-tests:
159128
name: Browser Tests
@@ -171,6 +140,4 @@ jobs:
171140
os: ${{ matrix.os }}
172141
cache-key: py-browser-${{ matrix.browser }}
173142
run: |
174-
bazel test --local_test_jobs 1 --flaky_test_attempts 3 --pin_browsers=true \
175-
//py:common-${{ matrix.browser }} \
176-
//py:test-${{ matrix.browser }}
143+
bazel test --local_test_jobs 1 --flaky_test_attempts 3 --pin_browsers=true //py:common-${{ matrix.browser }} //py:test-${{ matrix.browser }}

0 commit comments

Comments
 (0)