From 7b96dfca67eeeab8749ab73f6f34dfabe1ed469e Mon Sep 17 00:00:00 2001
From: Adrian Moldovan <3854374+adimoldovan@users.noreply.github.com>
Date: Tue, 14 May 2024 12:25:17 +0300
Subject: [PATCH] [e2e tests] Remove BuildKite reporter (#47449)

* Remove BuildKite reporter

* Changelog

* Remove BuildKite reporter from api tests
---
 .github/workflows/ci.yml                      | 34 -------------------
 .../changelog/e2e-remove-buildkite-reporter   |  4 +++
 .../tests/api-core-tests/playwright.config.js |  1 -
 .../tests/e2e-pw/playwright.config.js         |  1 -
 4 files changed, 4 insertions(+), 36 deletions(-)
 create mode 100644 plugins/woocommerce/changelog/e2e-remove-buildkite-reporter

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 0c3f4c923f1..43d8a56311c 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -116,29 +116,12 @@ jobs:
                   install: '${{ matrix.projectName }}...'
                   build: '${{ matrix.projectName }}'
 
-            - name: Get commit message
-              id: get_commit_message
-              run: |
-                  if [[ "${{ github.event_name }}" == "push" ]]; then
-                    COMMIT_MESSAGE=`echo "${{ github.event.head_commit.message }}" | head -1`
-                  elif [[ "${{ github.event_name }}" == "pull_request" ]]; then
-                    COMMIT_MESSAGE="${{ github.event.pull_request.title }}"
-                  else
-                    COMMIT_MESSAGE="${{ github.event_name }}"
-                  fi
-                
-                  echo "COMMIT_MESSAGE=$COMMIT_MESSAGE" >> "$GITHUB_OUTPUT"
-              shell: bash
-
             - name: 'Prepare Test Environment'
               id: 'prepare-test-environment'
               if: ${{ matrix.testEnv.shouldCreate }}
               run: 'pnpm --filter="${{ matrix.projectName }}" ${{ matrix.testEnv.start }}'
 
             - name: 'Run tests'
-              env:
-                  BUILDKITE_ANALYTICS_TOKEN: ${{ secrets.BUILDKITE_ANALYTICS_TOKEN }}
-                  BUILDKITE_ANALYTICS_MESSAGE: ${{ steps.get_commit_message.outputs.COMMIT_MESSAGE }}
               run: 'pnpm --filter="${{ matrix.projectName }}" ${{ matrix.command }}'
 
             - name: 'Upload artifacts'
@@ -170,29 +153,12 @@ jobs:
                   install: '${{ matrix.projectName }}...'
                   build: '${{ matrix.projectName }}'
 
-            - name: Get commit message
-              id: get_commit_message
-              run: |
-                  if [[ "${{ github.event_name }}" == "push" ]]; then
-                    COMMIT_MESSAGE=`echo "${{ github.event.head_commit.message }}" | head -1`
-                  elif [[ "${{ github.event_name }}" == "pull_request" ]]; then
-                    COMMIT_MESSAGE="${{ github.event.pull_request.title }}"
-                  else
-                    COMMIT_MESSAGE="${{ github.event_name }}"
-                  fi
-                
-                  echo "COMMIT_MESSAGE=$COMMIT_MESSAGE" >> "$GITHUB_OUTPUT"
-              shell: bash
-
             - name: 'Prepare Test Environment'
               id: 'prepare-test-environment'
               if: ${{ matrix.testEnv.shouldCreate }}
               run: 'pnpm --filter="${{ matrix.projectName }}" ${{ matrix.testEnv.start }}'
 
             - name: 'Run tests'
-              env:
-                  BUILDKITE_ANALYTICS_TOKEN: ${{ secrets.BUILDKITE_API_ANALYTICS_TOKEN }}
-                  BUILDKITE_ANALYTICS_MESSAGE: ${{ steps.get_commit_message.outputs.COMMIT_MESSAGE }}
               run: 'pnpm --filter="${{ matrix.projectName }}" ${{ matrix.command }}'
 
             - name: 'Upload artifacts'
diff --git a/plugins/woocommerce/changelog/e2e-remove-buildkite-reporter b/plugins/woocommerce/changelog/e2e-remove-buildkite-reporter
new file mode 100644
index 00000000000..bda9047bfd7
--- /dev/null
+++ b/plugins/woocommerce/changelog/e2e-remove-buildkite-reporter
@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+
+Remove BuildKite reporter from e2e tests
diff --git a/plugins/woocommerce/tests/api-core-tests/playwright.config.js b/plugins/woocommerce/tests/api-core-tests/playwright.config.js
index be1e0c34532..6a3c9207974 100644
--- a/plugins/woocommerce/tests/api-core-tests/playwright.config.js
+++ b/plugins/woocommerce/tests/api-core-tests/playwright.config.js
@@ -48,7 +48,6 @@ const config = {
 					'./test-results/test-results.json',
 			},
 		],
-		[ 'buildkite-test-collector/playwright/reporter' ],
 	],
 	use: {
 		screenshot: 'only-on-failure',
diff --git a/plugins/woocommerce/tests/e2e-pw/playwright.config.js b/plugins/woocommerce/tests/e2e-pw/playwright.config.js
index 743dba84310..36b6b128f90 100644
--- a/plugins/woocommerce/tests/e2e-pw/playwright.config.js
+++ b/plugins/woocommerce/tests/e2e-pw/playwright.config.js
@@ -31,7 +31,6 @@ const reporter = [
 
 if ( process.env.CI ) {
 	reporter.push( [ 'github' ] );
-	reporter.push( [ 'buildkite-test-collector/playwright/reporter' ] );
 } else {
 	reporter.push( [
 		'html',