Merge release/9.0.0 into trunk

Author: mgascam

.eslintrc

@@ -21,8 +21,7 @@
 		"wcpaySettings": true,
 		"page": true,
 		"browser": true,
-		"context": true,
-		"jestPuppeteer": true
+		"context": true
 	},
 	"settings": {
 		"react": {

.github/actions/e2e-pw/run-log-tests/action.yml

@@ -8,7 +8,7 @@ runs:
           id: first_run_e2e_pw_tests
           # Use +e to trap errors when running E2E tests.
           shell: /bin/bash +e {0}
-          run: npm run test:e2e-pw-ci
+          run: npm run test:e2e-ci
         - uses: actions/upload-artifact@v4
           if: always()
           with:

.github/actions/e2e/atomic-prepare-and-run/action.yml

@@ -25,5 +25,9 @@ runs:
       run: |
         mkdir screenshots
 
+    - name: Install Playwright
+      shell: bash
+      run: npx playwright install chromium
+
     - name: "Run tests, upload screenshots & logs"
       uses: ./.github/actions/e2e/run-log-tests

.github/actions/e2e/run-log-tests/action.yml

@@ -9,19 +9,18 @@ runs:
       # Use +e to trap errors when running E2E tests.
       shell: /bin/bash +e {0}
       run: |
-        npm run test:e2e -- --json --outputFile="$E2E_RESULT_FILEPATH" --NODE_ENV="$NODE_ENV"
+        npm run test:e2e-ci
         
         if [[ -f "$E2E_RESULT_FILEPATH" ]]; then
-          E2E_NUM_FAILED_TEST_SUITES=$(cat "$E2E_RESULT_FILEPATH" | jq '.numFailedTestSuites')
+          E2E_NUM_FAILED_TEST_SUITES=$(cat "$E2E_RESULT_FILEPATH" | jq '.stats["unexpected"]')
           echo "FIRST_RUN_FAILED_TEST_SUITES=$(echo $E2E_NUM_FAILED_TEST_SUITES)" >> $GITHUB_OUTPUT
           if [[ ${E2E_NUM_FAILED_TEST_SUITES} -gt 0 ]]; then
             echo "::notice::${E2E_NUM_FAILED_TEST_SUITES} test suite(s) failed in the first run but we will try (it) them again in the second run."
             exit 0
           fi
         else
           echo "FIRST_RUN_FAILED_TEST_SUITES=0" >> $GITHUB_OUTPUT
-          echo "::notice::${E2E_RESULT_FILEPATH} doesn't exist so results are unclear and we will not try again."
-          exit 1
+          exit 0
         fi
 
     # Retry failed E2E tests
@@ -30,18 +29,17 @@ runs:
       shell: bash
       # Filter failed E2E files from the result JSON file, and re-run them.
       run: |
-        cat "$E2E_RESULT_FILEPATH" | jq '.testResults[] | select(.status == "failed") | .name' | xargs npm run test:e2e -- --NODE_ENV="$NODE_ENV"
+        npm run test:e2e-ci $(cat $E2E_RESULT_FILEPATH | jq -r '[.suites[] | (if has("suites") then .suites[] | .specs[] else .specs[] end) | select(.tests[].status == "unexpected") | .file] | unique | .[]')
 
     # Archive screenshots if any
     - name: Archive e2e test screenshots & logs
-      if: ${{ failure() }}
+      if: ${{ always() }}
       uses: actions/upload-artifact@v4
       with:
           name: wp(${{ env.E2E_WP_VERSION }})-wc(${{ env.E2E_WC_VERSION }})-${{ env.E2E_GROUP }}-${{ env.E2E_BRANCH }}
           path: |
-            screenshots
-            tests/e2e/screenshots
-            tests/e2e/docker/wordpress/wp-content/debug.log
+            playwright-report/
+            tests/e2e/test-results
             ${{ env.E2E_RESULT_FILEPATH }}
           if-no-files-found: ignore
           retention-days: 14

.github/workflows/e2e-pull-request.yml

@@ -3,9 +3,8 @@ name: E2E Tests - Pull Request
 on:
   pull_request:
     branches:
-      # - develop
-      # - trunk
-      - dev/10240-update-npm-scripts # [TODO] Remove this line after the branch is merged.
+      - develop
+      - trunk
   workflow_dispatch:
   workflow_call:
     inputs:
@@ -28,7 +27,7 @@ env:
   E2E_BLOG_TOKEN:                ${{ secrets.E2E_BLOG_TOKEN }}
   E2E_USER_TOKEN:                ${{ secrets.E2E_USER_TOKEN }}
   WC_E2E_SCREENSHOTS:            1
-  E2E_SLACK_CHANNEL:             ${{ secrets.E2E_SLACK_CHANNEL }}
+  E2E_SLACK_CHANNEL_ID:          ${{ secrets.E2E_SLACK_CHANNEL_ID }}
   E2E_SLACK_TOKEN:               ${{ secrets.E2E_SLACK_TOKEN }}
   E2E_USE_LOCAL_SERVER:          false
   E2E_RESULT_FILEPATH:           'tests/e2e/results.json'
@@ -75,5 +74,9 @@ jobs:
       - name: Setup E2E environment
         uses: ./.github/actions/e2e/env-setup
 
+      - name: Install Playwright
+        shell: bash
+        run: npx playwright install chromium
+
       - name: Run tests, upload screenshots & logs
         uses: ./.github/actions/e2e/run-log-tests

.github/workflows/e2e-pw-pull-request.yml

@@ -27,7 +27,7 @@ env:
     E2E_BLOG_TOKEN: ${{ secrets.E2E_BLOG_TOKEN }}
     E2E_USER_TOKEN: ${{ secrets.E2E_USER_TOKEN }}
     WC_E2E_SCREENSHOTS: 1
-    E2E_SLACK_CHANNEL: ${{ secrets.E2E_SLACK_CHANNEL }}
+    E2E_SLACK_CHANNEL_ID: ${{ secrets.E2E_SLACK_CHANNEL_ID }}
     E2E_SLACK_TOKEN: ${{ secrets.E2E_SLACK_TOKEN }}
     E2E_USE_LOCAL_SERVER: false
     E2E_RESULT_FILEPATH: 'tests/e2e/results.json'
@@ -71,4 +71,4 @@ jobs:
               run: npx playwright install chromium
 
             - name: Run tests, upload screenshots & logs
-              uses: ./.github/actions/e2e-pw/run-log-tests
+              uses: ./.github/actions/e2e/run-log-tests

.github/workflows/e2e-test.yml

@@ -19,7 +19,7 @@ env:
   E2E_BLOG_TOKEN:                ${{ secrets.E2E_BLOG_TOKEN }}
   E2E_USER_TOKEN:                ${{ secrets.E2E_USER_TOKEN }}
   WC_E2E_SCREENSHOTS:            1
-  E2E_SLACK_CHANNEL:             ${{ secrets.E2E_SLACK_CHANNEL }}
+  E2E_SLACK_CHANNEL_ID:          ${{ secrets.E2E_SLACK_CHANNEL_ID }}
   E2E_SLACK_TOKEN:               ${{ secrets.E2E_SLACK_TOKEN }}
   E2E_USE_LOCAL_SERVER:          false
   E2E_RESULT_FILEPATH:           'tests/e2e/results.json'
@@ -64,6 +64,10 @@ jobs:
       - name: Setup E2E environment
         uses: ./.github/actions/e2e/env-setup
 
+      - name: Install Playwright
+        shell: bash
+        run: npx playwright install chromium
+
       - name: Run tests, upload screenshots & logs
         uses: ./.github/actions/e2e/run-log-tests
 
@@ -116,5 +120,9 @@ jobs:
       - name: Setup E2E environment
         uses: ./.github/actions/e2e/env-setup
 
+      - name: Install Playwright
+        shell: bash
+        run: npx playwright install chromium
+
       - name: Run tests, upload screenshots & logs
         uses: ./.github/actions/e2e/run-log-tests

.github/workflows/e2e-tests-atomic.yml

@@ -6,7 +6,7 @@ on:
 
 env:
   WC_E2E_SCREENSHOTS: 1
-  E2E_SLACK_CHANNEL: ${{ secrets.E2E_SLACK_CHANNEL }}
+  E2E_SLACK_CHANNEL_ID: ${{ secrets.E2E_SLACK_CHANNEL_ID }}
   E2E_SLACK_TOKEN: ${{ secrets.E2E_SLACK_TOKEN }}
   E2E_WP_VERSION: 'nightly'
   E2E_WC_VERSION: 'latest'

.gitignore

@@ -85,16 +85,15 @@ local.env
 # Screenshots created locally when running e2e tests
 tests/e2e/screenshots
 
-# E2E Performance test results
+# E2E test results
 tests/e2e/reports
-
-# E2E Playwright
 /playwright-report/
 /blob-report/
-tests/e2e-pw/.auth/
-tests/e2e-pw/test-results/
-tests/e2e-pw/playwright/.cache/
-tests/e2e-pw/tests/e2e-pw/.auth/*
+tests/e2e/.auth/
+tests/e2e/test-results/
+tests/e2e/playwright/.cache/
+tests/e2e/tests/e2e/.auth/*
+
 # Slate docs
 docs/rest-api/build/*