Skip to content

Commit faf2978

Browse files
committed
fix: update Playwright workflow to support multisite configuration and improve caching keys
1 parent c49e829 commit faf2978

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

.github/workflows/playwright-test.yml

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,9 @@ jobs:
5555
- name: Checkout source code
5656
uses: actions/checkout@v4
5757

58+
- name: WordPress mode
59+
run: echo "multisite=${{ inputs.multisite }}"
60+
5861
- name: Set up PHP
5962
uses: codesnippetspro/setup-php@v2
6063
with:
@@ -90,9 +93,9 @@ jobs:
9093
path: |
9194
node_modules
9295
src/vendor
93-
key: ${{ runner.os }}-deps-${{ steps.deps-hash.outputs.deps_hash }}
96+
key: ${{ runner.os }}-${{ inputs.test-mode }}-deps-${{ steps.deps-hash.outputs.deps_hash }}
9497
restore-keys: |
95-
${{ runner.os }}-deps-
98+
${{ runner.os }}-${{ inputs.test-mode }}-deps-
9699
97100
- name: Install workflow dependencies
98101
if: steps.deps-cache.outputs.cache-hit != 'true'
@@ -105,7 +108,7 @@ jobs:
105108
path: |
106109
src/vendor
107110
node_modules
108-
key: ${{ runner.os }}-deps-${{ steps.deps-hash.outputs.deps_hash }}
111+
key: ${{ runner.os }}-${{ inputs.test-mode }}-deps-${{ steps.deps-hash.outputs.deps_hash }}
109112

110113
- name: Wait for WordPress to be reachable
111114
run: |
@@ -128,7 +131,7 @@ jobs:
128131
run: |
129132
curl -fsSL -o "${RUNNER_TEMP}/wp-cli.phar" https://raw.githubusercontent.com/wp-cli/builds/gh-pages/phar/wp-cli.phar
130133
131-
- name: Install WordPress (and optionally multisite)
134+
- name: "Install WordPress: ${{ inputs.multisite && 'Multisite' || 'Single Site' }}"
132135
env:
133136
WP_CONTAINER: ${{ job.services.wordpress.id }}
134137
WP_URL: http://localhost:8888

.github/workflows/playwright.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,15 @@ jobs:
3030
with:
3131
test-mode: 'default'
3232
project-name: 'chromium-db-snippets'
33+
multisite: ${{ github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'run-tests') && contains(github.event.pull_request.labels.*.name, 'multisite') }}
3334

3435
playwright-file-based-execution:
3536
if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'run-tests')
3637
uses: ./.github/workflows/playwright-test.yml
3738
with:
3839
test-mode: 'file-based-execution'
3940
project-name: 'chromium-file-based-snippets'
41+
multisite: ${{ github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'run-tests') && contains(github.event.pull_request.labels.*.name, 'multisite') }}
4042

4143
test-result:
4244
needs: [playwright-default, playwright-file-based-execution]

0 commit comments

Comments
 (0)