release test

Author: amandaxmqiu

.github/workflows/releasetest.yaml

@@ -2,12 +2,12 @@ name: Split-Module GUI Test Pipeline
 
 on:
   pull_request:
-    branches: [main]
+    branches: [ main ]
 
-# ───────────────────────────────────────────────────────────────
-# 1. Playwright tests     → artefacts (no Pages)
-# ───────────────────────────────────────────────────────────────
 jobs:
+# ───────────────────────────────────────────
+# 1. Playwright tests  → artifacts only
+# ───────────────────────────────────────────
   test:
     runs-on: ubuntu-latest
     steps:
@@ -18,7 +18,7 @@ jobs:
           cache: npm
       - run: npm install
 
-      # Playwright only – dashboard & lint disabled here
+      # Playwright only (dashboard off here)
       - name: GUI Test – Playwright only
         uses: DigitalProductInnovationAndDevelopment/Code-Reviews-of-GUI-Tests@v1.0.12
         with:
@@ -28,24 +28,24 @@ jobs:
           enable-dashboard: 'false'
           enable-github-pages: 'false'
 
-      # build playwright-summary AFTER the action moved metrics → artifacts/
+      # Build correct summary JSON — reads *after* action moved metrics
       - name: Build playwright-summary-pr.json
         run: |
           node - <<'EOF'
           const fs = require('fs');
           const rpt = JSON.parse(fs.readFileSync('artifacts/playwright-metrics.json','utf8'));
           const out = {
-            total:     rpt.stats.tests,
-            passed:    rpt.stats.passed,
-            failed:    rpt.stats.failed ?? rpt.stats.unexpected ?? 0,
-            skipped:   rpt.stats.skipped ?? 0,
-            duration:  rpt.stats.duration,
-            pass_rate: +((rpt.stats.passed / rpt.stats.tests) * 100).toFixed(2)
+            total:    rpt.stats.tests,
+            passed:   rpt.stats.passed,
+            failed:   rpt.stats.failed,
+            skipped:  rpt.stats.skipped,
+            duration: rpt.stats.duration,
+            pass_rate: +(rpt.stats.tests ? (rpt.stats.passed / rpt.stats.tests) * 100 : 0).toFixed(2)
           };
           fs.writeFileSync('artifacts/playwright-summary-pr.json', JSON.stringify(out, null, 2));
           EOF
 
-      # upload three artefacts used by dashboard job
+      # Upload three artifacts for dashboard job
       - uses: actions/upload-artifact@v4
         with:
           name: playwright-summary
@@ -61,9 +61,9 @@ jobs:
           name: playwright-report
           path: artifacts/pr-report/
 
-# ───────────────────────────────────────────────────────────────
-# 2. ESLint / Prettier
-# ───────────────────────────────────────────────────────────────
+# ───────────────────────────────────────────
+# 2. ESLint / Prettier  → lint-summary.json
+# ───────────────────────────────────────────
   lint:
     runs-on: ubuntu-latest
     steps:
@@ -79,18 +79,18 @@ jobs:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           mode: lint-only
 
-      # upload lint-summary.json so dashboard can read real numbers
-      - name: Upload lint summary
+      # Upload lint summary for dashboard
+      - name: Upload lint-summary.json
         uses: actions/upload-artifact@v4
         with:
           name: lint-summary
           path: artifacts/lint-summary.json
 
-# ───────────────────────────────────────────────────────────────
-# 3. Dashboard / checklist / flow-chart / PR comment / Pages
-# ───────────────────────────────────────────────────────────────
+# ───────────────────────────────────────────
+# 3. Dashboard / PR comment / Pages deploy
+# ───────────────────────────────────────────
   dashboard:
-    needs: [test, lint]
+    needs: [ test, lint ]
     runs-on: ubuntu-latest
 
     permissions:
@@ -106,33 +106,25 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - name: Install npm if repo has package.json
+      - name: Install npm only if needed
         run: |
           if [ -f package.json ]; then
             npm ci 2>/dev/null || npm install
           fi
 
-      # download all artefacts produced in previous jobs
+      # Download all artifacts
       - uses: actions/download-artifact@v4
-        with:
-          name: playwright-summary
-          path: dl
+        with: { name: playwright-summary, path: dl }
       - uses: actions/download-artifact@v4
-        with:
-          name: playwright-metrics
-          path: dl
+        with: { name: playwright-metrics, path: dl }
       - uses: actions/download-artifact@v4
-        with:
-          name: playwright-report
-          path: dl/pr-report
+        with: { name: playwright-report,  path: dl/pr-report }
       - uses: actions/download-artifact@v4
-        with:
-          name: lint-summary
-          path: dl
+        with: { name: lint-summary,       path: dl }
 
-      # build dashboard, flow-chart, checklist, PR comment, deploy Pages
+      # Build dashboard (flow-chart + checklist inside)
       - id: review
-        name: Build dashboard & deploy
+        name: Dashboard / PR comment / Pages deploy
         uses: DigitalProductInnovationAndDevelopment/Code-Reviews-of-GUI-Tests@v1.0.12
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}