add opsSecPerRun to have each of the operations per second when repeatSuite is grater than 1

Author: Cesar-M-Diaz

lib/lifecycle.js

@@ -164,6 +164,8 @@ async function runBenchmark(
 
 	let totalIterations = 0;
 	let totalTimeSpent = 0;
+	const opsSecPerRun = repeatSuite > 1 ? [] : null;
+
 	for (let i = 0; i < repeatSuite; ++i) {
 		const { iterations, timeSpent } = await runBenchmarkOnce(
 			bench,
@@ -175,6 +177,12 @@ async function runBenchmark(
 			},
 			benchmarkMode,
 		);
+
+		if (opsSecPerRun) {
+			const runOpsSec = iterations / (timeSpent / timer.scale);
+			opsSecPerRun.push(runOpsSec);
+		}
+
 		totalTimeSpent += timeSpent;
 		totalIterations += iterations;
 	}
@@ -206,8 +214,11 @@ async function runBenchmark(
 		);
 	} else {
 		result.opsSec = opsSec;
+		if (opsSecPerRun) {
+			result.opsSecPerRun = opsSecPerRun;
+		}
 		debugBench(
-			`${bench.name} completed ${sampleData.length} samples with ${opsSec.toFixed(2)} ops/sec`,
+			`${bench.name} completed ${sampleData.length} samples with ${opsSecPerRun ? opsSecPerRun.map((op) => op.toFixed(2)).join(", ") : opsSec.toFixed(2)} ops/sec`,
 		);
 	}
 

test/time-mode.js

@@ -146,4 +146,54 @@ describe("Time-based Benchmarking", () => {
 			code: "ERR_INVALID_ARG_TYPE",
 		});
 	});
+
+	it("should include opsSecPerRun when repeatSuite is greater than 1", async () => {
+		const suite = new Suite({ reporter: false });
+		const repeatCount = 3;
+
+		// A very fast operation that should be consistent
+		suite.add("Repeat ops test", { repeatSuite: repeatCount }, () => {
+			// Simple operation
+			const x = 1 + 1;
+		});
+
+		const results = await suite.run();
+
+		assert.strictEqual(results.length, 1);
+		assert.ok(results[0].opsSec !== undefined, "opsSec should be defined");
+		assert.ok(
+			Array.isArray(results[0].opsSecPerRun),
+			"opsSecPerRun should be an array",
+		);
+		assert.strictEqual(
+			results[0].opsSecPerRun.length,
+			repeatCount,
+			`opsSecPerRun should have ${repeatCount} elements`,
+		);
+		for (const opsSec of results[0].opsSecPerRun) {
+			assert.ok(
+				typeof opsSec === "number" && !Number.isNaN(opsSec),
+				"each element should be a valid number",
+			);
+		}
+	});
+
+	it("should not include opsSecPerRun when repeatSuite is 1", async () => {
+		const suite = new Suite({ reporter: false });
+
+		// A very fast operation that should be consistent
+		suite.add("Single ops test", { repeatSuite: 1 }, () => {
+			// Simple operation
+			const x = 1 + 1;
+		});
+
+		const results = await suite.run();
+
+		assert.strictEqual(results.length, 1);
+		assert.ok(results[0].opsSec !== undefined, "opsSec should be defined");
+		assert.ok(
+			results[0].opsSecPerRun === undefined,
+			"opsSecPerRun should not be defined",
+		);
+	});
 });