Merge pull request #137 from DigitalProductInnovationAndDevelopment/feature/opt

optimize test

Author: amandaxmqiu

.github/workflows/releasetest_sep.yaml

@@ -6,7 +6,7 @@ on:
       - main
 
 # ───────────────────────────────────────────────────────────────
-# 1. Playwright tests  → artifacts
+# 1. Playwright tests  → artifacts 
 # ───────────────────────────────────────────────────────────────
 jobs:
   test:
@@ -137,16 +137,6 @@ jobs:
           name: playwright-metrics
           path: dl
 
-      - name: Prepare metrics file
-        run: |
-          if [ -f dl/playwright-metrics-pr.json ]; then
-            cp dl/playwright-metrics-pr.json dl/playwright-metrics.json
-            echo "✅ Copied playwright-metrics-pr.json to playwright-metrics.json"
-          else
-            echo "⚠️  Warning: playwright-metrics-pr.json not found"
-            ls -la dl/
-          fi
-
       - name: Download PR HTML report
         uses: actions/download-artifact@v4
         with:
@@ -165,6 +155,26 @@ jobs:
           name: lint-summary
           path: dl
 
+      # CRITICAL: Ensure PR metrics are used for visualization
+      - name: Prepare artifact files
+        run: |
+          echo "📁 Downloaded artifacts:"
+          ls -la dl/
+          
+          # Ensure the PR metrics file is properly named
+          if [ -f dl/playwright-metrics-pr.json ]; then
+            cp dl/playwright-metrics-pr.json dl/playwright-metrics.json
+            echo "✅ Using PR metrics for visualization"
+          else
+            echo "❌ PR metrics file not found!"
+          fi
+          
+          # Ensure PR summary is properly named
+          if [ -f dl/playwright-summary-pr.json ] && [ ! -f dl/playwright-summary.json ]; then
+            cp dl/playwright-summary-pr.json dl/playwright-summary.json
+            echo "✅ Using PR summary"
+          fi
+
       # Build dashboard, post comment, deploy Pages
       - id: review
         name: Dashboard / PR comment / Pages
@@ -174,4 +184,4 @@ jobs:
           mode:               dashboard-only
           custom-artifacts-path: dl
           enable-github-pages: 'true'
-          enable-visual-comparison: 'true'
+          enable-visual-comparison: 'true'

action.yml

@@ -444,14 +444,17 @@ runs:
         '  artifacts/playwright-metrics.json > artifacts/playwright-summary-pr.json
 
       fi
-      cp artifacts/playwright-metrics-pr.json artifacts/playwright-metrics.json || true
+      # IMPORTANT: Keep a copy of PR metrics before main branch tests
+      if [ -f artifacts/playwright-metrics.json ]; then
+        cp artifacts/playwright-metrics.json artifacts/playwright-metrics-pr.json
+        echo "✅ Preserved PR metrics"
+      fi
 
   - name: Alias summary for checklist
     if: steps.modes.outputs.playwright == 'true'
     shell: bash
     run: |
       cp artifacts/playwright-summary-pr.json artifacts/playwright-summary.json || true
-      cp artifacts/playwright-metrics.json artifacts/playwright-metrics-pr.json || true
 
   # 3b ── Playwright on main branch with progress indicators
   - name: Run Playwright on main