optimize test

Author: amandaxmqiu

scripts/generate-test-city-3d.js.js scripts/generate-test-city-3d.jsscripts/generate-test-city-3d.js.js renamed to scripts/generate-test-city-3d.js

@@ -22,13 +22,44 @@ const readJSON = (filepath, defaultValue = null) => {
 
 // Extract comprehensive test data from Playwright metrics
 function extractTestData() {
-  const metrics = readJSON('playwright-metrics.json') || readJSON(path.join(ART, 'playwright-metrics.json'));
-  const history = readJSON('.test-history.json');
-  const failureAnalysis = readJSON(path.join(ART, 'test-failure-analysis.json'));
+  const possiblePaths = [
+    'playwright-metrics.json',
+    path.join(ART, 'playwright-metrics.json'),
+    path.join(ART, 'playwright-summary-pr.json'), 
+    path.join(ART, 'playwright-summary.json')   
+  ];
+  
+  let metrics = null;
+  for (const p of possiblePaths) {
+    if (fs.existsSync(p)) {
+      metrics = readJSON(p);
+      if (metrics && (metrics.suites || metrics.total)) {
+        console.log(`Found metrics at: ${p}`);
+        break;
+      }
+    }
+  }
 
+  // If no detailed metrics found, try to use summary
   if (!metrics || !metrics.suites) {
-    console.error('No test metrics found');
-    return [];
+    const summary = readJSON(path.join(ART, 'playwright-summary-pr.json'));
+    if (summary && summary.total) {
+      // Create a minimal structure from summary
+      console.log('Using summary data for visualization');
+      return [{
+        id: 'summary-tests',
+        suite: 'All Tests',
+        describe: 'Summary',
+        name: `${summary.passed} passed, ${summary.failed} failed`,
+        duration: summary.duration || 0,
+        passRate: summary.pass_rate || 0,
+        passed: summary.passed || 0,
+        failed: summary.failed || 0,
+        total: summary.total || 0,
+        lastStatus: summary.failed > 0 ? 'failed' : 'passed',
+        priority: 1
+      }];
+    }
   }
 
   const testData = [];

scripts/summary-comment.js

@@ -303,6 +303,9 @@ ${testHistory.flakyTests.length > 5 ? `\n_...and ${testHistory.flakyTests.length
 </details>
 ` : '';
 
+/* dashboard root (absolute if workflow provided it) */
+const dashboardURL = process.env.WEB_REPORT_URL || 'index.html';
+
 // Quick commands section (enhanced)
 const mdQuickCommands = quickActionsData?.commands?.length > 0 ? `
 ## ⚡ Quick Actions
@@ -327,14 +330,18 @@ ${cmd.command}
 const insights = generateInsights();
 const recommendations = generateRecommendations();
 
-/* dashboard root (absolute if workflow provided it) */
-const dashboardURL = process.env.WEB_REPORT_URL || 'index.html';
-
 /* Status summary line */
 const overallStatus = playPR.failed === 0 && codeQualityIssues === 0 ? 
   '✅ **All checks passed!**' : 
   `⚠️ **${playPR.failed} test failure(s), ${codeQualityIssues} code quality issue(s)${hasVisualChanges ? `, ${visualChangeCount} visual change(s)` : ''}**`;
 
+const joinURL = (base, path) => {
+  if (base.endsWith('/')) {
+    return base + path;
+  }
+  return base + '/' + path;
+};
+
 /* final comment body */
 const body = `
 # 🔍 GUI Test Review Summary
@@ -343,7 +350,7 @@ ${overallStatus}
 
 <div align="center">
 
-[📊 **Dashboard**](${dashboardURL}) • [🏙️ **3D Test City**](${dashboardURL}/test-city-3d.html) • [🖼️ **Visual Regression**](${dashboardURL}#visual-regression) • [⚡ **Quick Actions**](${dashboardURL}#quick-actions)
+[📊 **Dashboard**](${dashboardURL}) • [🏙️ **3D Test City**](${joinURL(dashboardURL, 'test-city-3d.html')}) • [🖼️ **Visual Regression**](${dashboardURL}#visual-regression) • [⚡ **Quick Actions**](${dashboardURL}#quick-actions)
 
 </div>