feat: Keep bars aligned when test names are longer than the requested length.

Author: jdmarshall

lib/reporter/chart.js

@@ -116,6 +116,9 @@ function chartReport(results, options = { labelWidth: 45, printHeader: true }) {
 		process.stdout.write(styleText("bold", "\nSummary (vs. baseline):\n"));
 	}
 
+	const maxNameLength = Math.max(...results.map((r) => r.name.length));
+	const columnWidth = Math.max(maxNameLength, options.labelWidth ?? 45);
+
 	for (const result of results) {
 		let comment = "";
 
@@ -129,8 +132,6 @@ function chartReport(results, options = { labelWidth: 45, printHeader: true }) {
 			}
 		}
 
-		const columnWidth = options.labelWidth ?? 45;
-
 		drawBar(
 			result.name.padEnd(columnWidth),
 			result[primaryMetric],

test/reporter.js

@@ -73,6 +73,46 @@ describe("chartReport", () => {
 		});
 	});
 
+	describe("with long names", async (t) => {
+		let output = "";
+
+		before(async () => {
+			const originalStdoutWrite = process.stdout.write;
+			process.stdout.write = (data) => {
+				output += data;
+			};
+
+			const suite = new Suite({
+				reporter: chartReport,
+			});
+
+			suite
+				.add("single with matcher looooooooooooooooooooooooooong", () => {
+					const pattern = /[123]/g;
+					const replacements = { 1: "a", 2: "b", 3: "c" };
+					const subject = "123123123123123123123123123123123123123123123123";
+					const r = subject.replace(pattern, (m) => replacements[m]);
+					assert.ok(r);
+				})
+				.add("multiple replaces", () => {
+					const subject = "123123123123123123123123123123123123123123123123";
+					const r = subject
+						.replace(/1/g, "a")
+						.replace(/2/g, "b")
+						.replace(/3/g, "c");
+					assert.ok(r);
+				});
+			await suite.run();
+
+			process.stdout.write = originalStdoutWrite;
+		});
+
+		it("should pad out benchmark names", () => {
+			assert.ok(output.includes("oong | "));
+			assert.ok(output.includes("multiple replaces".padEnd(51)));
+		});
+	});
+
 	describe("respects reporterOptions.printHeader", async (t) => {
 		let outputWithHeader = "";
 		let outputWithoutHeader = "";