Skip to content

Commit fd8d5e5

Browse files
committed
[ci] Update python tests
Signed-off-by: Viet Nguyen Duc <[email protected]>
1 parent d411615 commit fd8d5e5

File tree

2 files changed

+24
-10
lines changed

2 files changed

+24
-10
lines changed

.github/workflows/bazel.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ on:
7070
jobs:
7171
bazel:
7272
name: ${{ inputs.name }}
73-
runs-on: ${{ inputs.os == 'macos' && 'macos-13' || format('{0}-latest', inputs.os) }}
73+
runs-on: ${{ format('{0}-latest', inputs.os) }}
7474
env:
7575
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
7676
SEL_M2_USER: ${{ secrets.SEL_M2_USER }}

.github/workflows/ci-python.yml

Lines changed: 23 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -81,19 +81,33 @@ jobs:
8181
name: Remote Tests
8282
needs: build
8383
uses: ./.github/workflows/bazel.yml
84+
strategy:
85+
fail-fast: false
86+
matrix:
87+
include:
88+
- browser: chrome
89+
- browser: firefox
8490
with:
8591
name: Integration Tests (remote)
86-
browser: firefox
87-
cache-key: py-remote
92+
browser: ${{ matrix.browser }}
93+
cache-key: py-remote-${{ matrix.browser }}
8894
run: bazel test --local_test_jobs 1 --flaky_test_attempts 3 //py:test-remote
8995

90-
safari-tests:
91-
name: Safari Tests
96+
browser-tests:
97+
name: Browser Tests
9298
needs: build
9399
uses: ./.github/workflows/bazel.yml
100+
strategy:
101+
fail-fast: false
102+
matrix:
103+
include:
104+
- browser: safari
105+
os: macos
106+
- browser: edge
107+
os: ubuntu
94108
with:
95-
name: Integration Tests (safari)
96-
browser: safari
97-
os: macos
98-
cache-key: py-safari
99-
run: bazel test --local_test_jobs 1 --flaky_test_attempts 3 //py:test-safari
109+
name: Integration Tests (${matrix.browser})
110+
browser: ${{ matrix.browser }}
111+
os: ${{ matrix.os }}
112+
cache-key: py-browser-${{ matrix.browser }}
113+
run: bazel test --local_test_jobs 1 --flaky_test_attempts 3 //py:test-${{ matrix.browser }}

0 commit comments

Comments
 (0)