From a7b089e2e2528e38a0d3df8a675efcedd92a93c5 Mon Sep 17 00:00:00 2001
From: Phil Renaud <phil@riotindustries.com>
Date: Wed, 4 Dec 2024 10:57:40 -0500
Subject: [PATCH] Debugging run

---
 .github/workflows/test-ui.yml    | 42 ++++++++++++++++++++++++++++----
 scripts/analyze-ui-test-times.js |  8 +++---
 2 files changed, 41 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/test-ui.yml b/.github/workflows/test-ui.yml
index a52c0e7b61c..0ae783a8834 100644
--- a/.github/workflows/test-ui.yml
+++ b/.github/workflows/test-ui.yml
@@ -143,16 +143,48 @@ jobs:
         run: yarn percy build:finalize
 
   analyze-times:
-    needs: [tests, finalize]
+    # TODO: temporary comment-out with hardcoded sha
+    # needs: [tests, finalize]
     if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: ui
+
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
-      - uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
+
+      # Debug step to show environment
+      - name: Debug environment
+        run: |
+          echo "GITHUB_SHA: ${{ github.sha }}"
+          echo "GITHUB_EVENT_NAME: ${{ github.event_name }}"
+          echo "GITHUB_REF: ${{ github.ref }}"
+          echo "RUN_ID: ${{ github.run_id }}"
+          
+      # Try to list available artifacts first
+      - name: List artifacts
+        uses: actions/github-script@v7
         with:
-          name: test-results-${{ github.sha }}
+          script: |
+            const artifacts = await github.rest.actions.listWorkflowRunArtifacts({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              // run_id: context.runId
+              run_id: 12163157778
+            });
+            console.log('Available artifacts:');
+            console.log(JSON.stringify(artifacts.data, null, 2));
+
+      - name: Download current PR results
+        uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
+        with:
+          # name: test-results-${{ github.sha }}
+          name: pr-test-results-fe7ca11e9afc42bc98d79fe521155a37634bd232 # TODO: temporary hardcoded sha from previous run
           path: ui
-          
+          run-id: 12163157778
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+
       - name: Download historical results
         uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
         with:
@@ -161,7 +193,7 @@ jobs:
           merge-multiple: true
           
       - name: Analyze test times
-        run: node scripts/analyze-test-times.js
+        run: node ../scripts/analyze-ui-test-times.js
 
       - name: Comment PR
         uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
diff --git a/scripts/analyze-ui-test-times.js b/scripts/analyze-ui-test-times.js
index bad4c738372..6c1bc60927e 100644
--- a/scripts/analyze-ui-test-times.js
+++ b/scripts/analyze-ui-test-times.js
@@ -9,7 +9,7 @@ const fs = require('fs');
 
 async function analyzeTestTimes() {
   const currentResults = JSON.parse(
-    fs.readFileSync('../ui/combined-test-results.json')
+    fs.readFileSync('combined-test-results.json')
   );
 
   // Create a map of test names to their durations
@@ -25,11 +25,11 @@ async function analyzeTestTimes() {
 
   // Read each historical result file
   console.log('[analyze-test-times] Reading historical results directory...\n');
-  const historicalFiles = fs.readdirSync('../historical-results');
+  const historicalFiles = fs.readdirSync('historical-results');
   historicalFiles.forEach((file, index) => {
     console.log(`[analyze-test-times] Reading ${file} (${index + 1} of ${historicalFiles.length})...`);
     const historical = JSON.parse(
-      fs.readFileSync(`../historical-results/${file}`)
+      fs.readFileSync(`historical-results/${file}`)
     );
 
     if (historical.summary.failed === 0) {
@@ -86,7 +86,7 @@ async function analyzeTestTimes() {
 
   // Write analysis results
   fs.writeFileSync(
-    '../ui/test-time-analysis.json',
+    'test-time-analysis.json',
     JSON.stringify(analysis, null, 2)
   );