ortoni-report 4.1.0-test.0 → 4.1.0-test.3

package/dist/cli.js

@@ -688,6 +688,9 @@ async function mergeAllData(options = {}) {
   let totalDurationMax = 0;
   let mergedUserConfig = null;
   let mergedUserMeta = null;
+  const badShards = [];
+  const shardCounts = {};
+  const shardDurMap = {};
   for (const file of sortedFiles) {
     const fullPath = path3.join(folderPath, file);
     try {
@@ -701,19 +704,25 @@ async function mergeAllData(options = {}) {
         continue;
       }
       shardData.results.forEach((r) => allResults.push(r));
+      shardCounts[file] = shardData.results.length;
       if (Array.isArray(shardData.projectSet)) {
         shardData.projectSet.forEach((p) => projectSet.add(p));
       }
-      const shardDur = Number(shardData.duration);
+      const rawShardDur = shardData.duration;
       let durToAdd = 0;
-      if (!isNaN(shardDur) && shardDur > 0) {
-        durToAdd = shardDur;
+      let perTestSum = 0;
+      if (typeof rawShardDur === "number") {
+        durToAdd = rawShardDur;
       } else {
-        const perTestSum = (Array.isArray(shardData.results) ? shardData.results : []).map((t) => Number(t.duration) || 0).reduce((a, b) => a + b, 0);
+        perTestSum = Array.isArray(shardData.results) ? shardData.results.reduce(
+          (acc, t) => acc + (Number(t?.duration) || 0),
+          0
+        ) : 0;
         durToAdd = perTestSum;
       }
       totalDurationSum += durToAdd;
       if (durToAdd > totalDurationMax) totalDurationMax = durToAdd;
+      shardDurMap[file] = durToAdd;
       if (shardData.userConfig) {
         if (!mergedUserConfig) mergedUserConfig = { ...shardData.userConfig };
         else {
@@ -739,9 +748,16 @@ async function mergeAllData(options = {}) {
       }
     } catch (err) {
       console.error(`Ortoni Report: Failed to parse shard ${file}:`, err);
+      badShards.push(file);
       continue;
     }
   }
+  if (badShards.length > 0) {
+    console.warn(
+      `Ortoni Report: Completed merge with ${badShards.length} bad shard(s) skipped:`,
+      badShards
+    );
+  }
   const totalDuration = totalDurationSum;
   const saveHistoryFromOptions = typeof options.saveHistory === "boolean" ? options.saveHistory : void 0;
   const saveHistoryFromShard = mergedUserConfig && typeof mergedUserConfig.saveHistory === "boolean" ? mergedUserConfig.saveHistory : void 0;
@@ -782,11 +798,12 @@ async function mergeAllData(options = {}) {
     dbManager
   );
   const finalReportData = await htmlGenerator.generateFinalReport(
-    // filteredResults: keep same semantics (filter out skipped for display or pass as required)
+    // filteredResults: typically filter out skipped for display (keeps existing behavior)
     allResults.filter((r) => r.status !== "skipped"),
     totalDuration,
     allResults,
     projectSet
+    // pass Set<string> as original generateFinalReport expects
   );
   const fileManager = new FileManager(folderPath);
   const outputFileName = options.file || "ortoni-report.html";
@@ -794,12 +811,10 @@ async function mergeAllData(options = {}) {
     outputFileName,
     finalReportData
   );
-  console.log(`\u2705 Final merged report generated at ${outputPath}`);
-  console.log(
-    `\u2705 Shards merged: ${sortedFiles.length}, total tests merged: ${allResults.length}`
-  );
-  console.log(`\u2705 totalDuration (sum of shards) = ${totalDuration}`);
-  if (runId) console.log(`\u2705 DB runId: ${runId}`);
+  console.log(`\u2705 Final merged report generated at ${await outputPath}`);
+  console.log(`\u2705 Shards merged: ${sortedFiles.length}`);
+  console.log(`\u2705 Tests per shard:`, shardCounts);
+  console.log(`\u2705 Total tests merged ${allResults.length}`);
 }
 
 // src/utils/expressServer.ts

package/dist/cli.mjs

@@ -44,6 +44,9 @@ async function mergeAllData(options = {}) {
   let totalDurationMax = 0;
   let mergedUserConfig = null;
   let mergedUserMeta = null;
+  const badShards = [];
+  const shardCounts = {};
+  const shardDurMap = {};
   for (const file of sortedFiles) {
     const fullPath = path.join(folderPath, file);
     try {
@@ -57,19 +60,25 @@ async function mergeAllData(options = {}) {
         continue;
       }
       shardData.results.forEach((r) => allResults.push(r));
+      shardCounts[file] = shardData.results.length;
       if (Array.isArray(shardData.projectSet)) {
         shardData.projectSet.forEach((p) => projectSet.add(p));
       }
-      const shardDur = Number(shardData.duration);
+      const rawShardDur = shardData.duration;
       let durToAdd = 0;
-      if (!isNaN(shardDur) && shardDur > 0) {
-        durToAdd = shardDur;
+      let perTestSum = 0;
+      if (typeof rawShardDur === "number") {
+        durToAdd = rawShardDur;
       } else {
-        const perTestSum = (Array.isArray(shardData.results) ? shardData.results : []).map((t) => Number(t.duration) || 0).reduce((a, b) => a + b, 0);
+        perTestSum = Array.isArray(shardData.results) ? shardData.results.reduce(
+          (acc, t) => acc + (Number(t?.duration) || 0),
+          0
+        ) : 0;
         durToAdd = perTestSum;
       }
       totalDurationSum += durToAdd;
       if (durToAdd > totalDurationMax) totalDurationMax = durToAdd;
+      shardDurMap[file] = durToAdd;
       if (shardData.userConfig) {
         if (!mergedUserConfig) mergedUserConfig = { ...shardData.userConfig };
         else {
@@ -95,9 +104,16 @@ async function mergeAllData(options = {}) {
       }
     } catch (err) {
       console.error(`Ortoni Report: Failed to parse shard ${file}:`, err);
+      badShards.push(file);
       continue;
     }
   }
+  if (badShards.length > 0) {
+    console.warn(
+      `Ortoni Report: Completed merge with ${badShards.length} bad shard(s) skipped:`,
+      badShards
+    );
+  }
   const totalDuration = totalDurationSum;
   const saveHistoryFromOptions = typeof options.saveHistory === "boolean" ? options.saveHistory : void 0;
   const saveHistoryFromShard = mergedUserConfig && typeof mergedUserConfig.saveHistory === "boolean" ? mergedUserConfig.saveHistory : void 0;
@@ -138,11 +154,12 @@ async function mergeAllData(options = {}) {
     dbManager
   );
   const finalReportData = await htmlGenerator.generateFinalReport(
-    // filteredResults: keep same semantics (filter out skipped for display or pass as required)
+    // filteredResults: typically filter out skipped for display (keeps existing behavior)
     allResults.filter((r) => r.status !== "skipped"),
     totalDuration,
     allResults,
     projectSet
+    // pass Set<string> as original generateFinalReport expects
   );
   const fileManager = new FileManager(folderPath);
   const outputFileName = options.file || "ortoni-report.html";
@@ -150,12 +167,10 @@ async function mergeAllData(options = {}) {
     outputFileName,
     finalReportData
   );
-  console.log(`\u2705 Final merged report generated at ${outputPath}`);
-  console.log(
-    `\u2705 Shards merged: ${sortedFiles.length}, total tests merged: ${allResults.length}`
-  );
-  console.log(`\u2705 totalDuration (sum of shards) = ${totalDuration}`);
-  if (runId) console.log(`\u2705 DB runId: ${runId}`);
+  console.log(`\u2705 Final merged report generated at ${await outputPath}`);
+  console.log(`\u2705 Shards merged: ${sortedFiles.length}`);
+  console.log(`\u2705 Tests per shard:`, shardCounts);
+  console.log(`\u2705 Total tests merged ${allResults.length}`);
 }
 
 // src/cli.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ortoni-report",
-  "version": "4.1.0-test.0",
+  "version": "4.1.0-test.3",
   "description": "Playwright Report By LetCode with Koushik",
   "scripts": {
     "tsc": "tsc",