refactor complex reportAndWrite

Author: rostalan

smoke-tests/smoke-test.mjs

@@ -408,55 +408,67 @@ async function probeFrontendPlugins(plugins, port, pluginsRoot) {
 // Reporting
 // ---------------------------------------------------------------------------
 
+function logResultAndCollectFailures(result, failedPlugins) {
+  switch (result.status) {
+    case "skip":
+      console.log(`  SKIP  ${result.pkgName}  (${result.role})`);
+      return;
+    case "pass":
+      if (result.pluginId) {
+        console.log(
+          `  PASS  ${result.pkgName}  → /api/${result.pluginId}  (${result.http})`,
+        );
+      } else if (isFrontendRole(result.role)) {
+        console.log(`  PASS  ${result.pkgName}  (${result.role})`);
+      }
+      return;
+    case "warn":
+      console.log(
+        `  WARN  ${result.pkgName}  → /api/${result.pluginId}  (404 — pluginId guess may be wrong)`,
+      );
+      return;
+    case "fail-bundle":
+      console.log(`  FAIL  ${result.pkgName}  [bundle] ${result.detail}`);
+      failedPlugins.push(result.pkgName);
+      return;
+    case "fail-load":
+      console.log(`  FAIL  ${result.pkgName}  [load] ${result.detail}`);
+      failedPlugins.push(result.pkgName);
+      return;
+    default:
+      console.log(`  FAIL  ${result.pkgName}  ${result.error}`);
+      failedPlugins.push(result.pkgName);
+  }
+}
+
+function updateResultCounts(result, backendCounts, frontendCounts) {
+  const isFrontend = isFrontendRole(result.role);
+  if (result.status === "fail-bundle" || result.status === "fail-load") {
+    if (isFrontend) frontendCounts.fail++;
+    else backendCounts.fail++;
+    return;
+  }
+
+  if (isFrontend) {
+    frontendCounts[result.status] = (frontendCounts[result.status] ?? 0) + 1;
+    return;
+  }
+
+  backendCounts[result.status] = (backendCounts[result.status] ?? 0) + 1;
+}
+
 function reportAndWrite(results, resultsFile) {
   console.log("\n========== Smoke Test Results ==========\n");
   const failedPlugins = [];
 
   for (const r of results) {
-    switch (r.status) {
-      case "skip":
-        console.log(`  SKIP  ${r.pkgName}  (${r.role})`);
-        break;
-      case "pass":
-        if (r.pluginId) {
-          console.log(
-            `  PASS  ${r.pkgName}  → /api/${r.pluginId}  (${r.http})`,
-          );
-        } else if (isFrontendRole(r.role)) {
-          console.log(`  PASS  ${r.pkgName}  (${r.role})`);
-        }
-        break;
-      case "warn":
-        console.log(
-          `  WARN  ${r.pkgName}  → /api/${r.pluginId}  (404 — pluginId guess may be wrong)`,
-        );
-        break;
-      case "fail-bundle":
-        console.log(`  FAIL  ${r.pkgName}  [bundle] ${r.detail}`);
-        failedPlugins.push(r.pkgName);
-        break;
-      case "fail-load":
-        console.log(`  FAIL  ${r.pkgName}  [load] ${r.detail}`);
-        failedPlugins.push(r.pkgName);
-        break;
-      default:
-        console.log(`  FAIL  ${r.pkgName}  ${r.error}`);
-        failedPlugins.push(r.pkgName);
-    }
+    logResultAndCollectFailures(r, failedPlugins);
   }
 
   const be = { pass: 0, warn: 0, skip: 0, fail: 0 };
   const fe = { pass: 0, fail: 0 };
   for (const r of results) {
-    const isFe = isFrontendRole(r.role);
-    if (r.status === "fail-bundle" || r.status === "fail-load") {
-      if (isFe) fe.fail++;
-      else be.fail++;
-    } else if (isFe) {
-      fe[r.status] = (fe[r.status] ?? 0) + 1;
-    } else {
-      be[r.status] = (be[r.status] ?? 0) + 1;
-    }
+    updateResultCounts(r, be, fe);
   }
   const total = results.length;
   const totalFail = be.fail + fe.fail;