chore: update duration formatting logic

Author: Ma11hewThomas

README.md

@@ -243,7 +243,7 @@ Formats a test duration value in milliseconds into a human-readable format. Perf
 
 #### `formatTestMessage`
 
-Core formatting logic for duration in milliseconds / const formatDurationMs = (durationMs: number): string => { if (Number.isNaN(durationMs) || durationMs < 0) { return "not captured"; } if (durationMs < 1) { return "1ms"; } else if (durationMs < 1000) { return `${Math.floor(durationMs)}ms`; } else if (durationMs < 60000) { return `${(durationMs / 1000).toFixed(1)}s`; } else if (durationMs < 3600000) { const minutes = Math.floor(durationMs / 60000); const seconds = Math.floor((durationMs % 60000) / 1000); return seconds > 0 ? `${minutes}m ${seconds}s` : `${minutes}m`; } else { const hours = Math.floor(durationMs / 3600000); const minutes = Math.floor((durationMs % 3600000) / 60000); return minutes > 0 ? `${hours}h ${minutes}m` : `${hours}h`; } }; /** Converts ANSI-formatted test messages into HTML and replaces newlines with `<br>` tags. Specifically designed for formatting the `test.message` property in CTRF reports. Ideal for rendering multi-line console messages with colors in a human-friendly HTML format. This helper formats test messages so they behave well with markdown and regular HTML content.
+Converts ANSI-formatted test messages into HTML and replaces newlines with `<br>` tags. Specifically designed for formatting the `test.message` property in CTRF reports. Ideal for rendering multi-line console messages with colors in a human-friendly HTML format. This helper formats test messages so they behave well with markdown and regular HTML content.
 
 **Parameters:**
 

__tests__/helpers/ctrf.test.ts

@@ -459,7 +459,7 @@ describe("CTRF Helpers", () => {
 
 		it("should handle negative duration (stop before start)", () => {
 			const result = formatDurationFromTimesHelper.fn(2000, 1000);
-			expect(result).toBe("not captured");
+			expect(result).toBe("1ms");
 		});
 	});
 

package-lock.json

@@ -1,6 +1,6 @@
 {
 	"name": "handlebars-helpers-ctrf",
-	"version": "0.0.4-next.9",
+	"version": "0.0.4-next.11",
 	"description": "A collection of Handlebars helpers for working with Common Test Report Format",
 	"type": "module",
 	"main": "dist/index.js",

package.json

@@ -307,7 +307,7 @@ export const formatDurationFromTimesHelper: Helper = {
 		const startNum = typeof start === "number" ? start : 0;
 		const stopNum = typeof stop === "number" ? stop : 0;
 
-		if (startNum === 0 && stopNum === 0) {
+		if (start === 0 && stop === 0) {
 			return "not captured";
 		}
 
@@ -316,7 +316,21 @@ export const formatDurationFromTimesHelper: Helper = {
 		}
 
 		const durationMs = stopNum - startNum;
-		return formatDurationMs(durationMs);
+		if (durationMs < 1) {
+			return `1ms`;
+		} else if (durationMs < 1000) {
+			return `${Math.floor(durationMs)}ms`;
+		} else if (durationMs < 60000) {
+			return `${(durationMs / 1000).toFixed(1)}s`;
+		} else if (durationMs < 3600000) {
+			const minutes = Math.floor(durationMs / 60000);
+			const seconds = Math.floor((durationMs % 60000) / 1000);
+			return seconds > 0 ? `${minutes}m ${seconds}s` : `${minutes}m`;
+		} else {
+			const hours = Math.floor(durationMs / 3600000);
+			const minutes = Math.floor((durationMs % 3600000) / 60000);
+			return minutes > 0 ? `${hours}h ${minutes}m` : `${hours}h`;
+		}
 	},
 };
 
@@ -336,33 +350,26 @@ export const formatDurationHelper: Helper = {
 	category: "CTRF",
 	fn: (duration: unknown) => {
 		const durationNum = typeof duration === "number" ? duration : 0;
-		return formatDurationMs(durationNum);
-	},
-};
+		if (Number.isNaN(durationNum) || durationNum < 0) {
+			return "not captured";
+		}
 
-/**
- * Core formatting logic for duration in milliseconds
- */
-const formatDurationMs = (durationMs: number): string => {
-	if (Number.isNaN(durationMs) || durationMs < 0) {
-		return "not captured";
-	}
-
-	if (durationMs < 1) {
-		return "1ms";
-	} else if (durationMs < 1000) {
-		return `${Math.floor(durationMs)}ms`;
-	} else if (durationMs < 60000) {
-		return `${(durationMs / 1000).toFixed(1)}s`;
-	} else if (durationMs < 3600000) {
-		const minutes = Math.floor(durationMs / 60000);
-		const seconds = Math.floor((durationMs % 60000) / 1000);
-		return seconds > 0 ? `${minutes}m ${seconds}s` : `${minutes}m`;
-	} else {
-		const hours = Math.floor(durationMs / 3600000);
-		const minutes = Math.floor((durationMs % 3600000) / 60000);
-		return minutes > 0 ? `${hours}h ${minutes}m` : `${hours}h`;
-	}
+		if (durationNum < 1) {
+			return `1ms`;
+		} else if (durationNum < 1000) {
+			return `${Math.floor(durationNum)}ms`;
+		} else if (durationNum < 60000) {
+			return `${(durationNum / 1000).toFixed(1)}s`;
+		} else if (durationNum < 3600000) {
+			const minutes = Math.floor(durationNum / 60000);
+			const seconds = Math.floor((durationNum % 60000) / 1000);
+			return seconds > 0 ? `${minutes}m ${seconds}s` : `${minutes}m`;
+		} else {
+			const hours = Math.floor(durationNum / 3600000);
+			const minutes = Math.floor((durationNum % 3600000) / 60000);
+			return minutes > 0 ? `${hours}h ${minutes}m` : `${hours}h`;
+		}
+	},
 };
 
 /**