fix: sort groups in HTML report when sort mode is changed

thomhurst · thomhurst · commit bfa2b65d5efe · 2026-03-07T17:37:20.000Z
The sort toggle in the HTML report only sorted tests within each group
but never reordered the groups themselves. Added sortGroups() to also
sort groups by duration (slowest first) or name (alphabetical).

Also added an empty favicon link to prevent 404 errors when the report
is served from a remote host.
diff --git a/TUnit.Engine/Reporters/Html/HtmlReportGenerator.cs b/TUnit.Engine/Reporters/Html/HtmlReportGenerator.cs
@@ -24,6 +24,7 @@ private static void AppendHead(StringBuilder sb, ReportData data)
         sb.AppendLine("<head>");
         sb.AppendLine("<meta charset=\"UTF-8\">");
         sb.AppendLine("<meta name=\"viewport\" content=\"width=device-width, initial-scale=1.0\">");
+        sb.AppendLine("<link rel=\"icon\" href=\"data:,\">");
         sb.Append("<title>Test Report \u2014 ");
         sb.Append(WebUtility.HtmlEncode(data.AssemblyName));
         sb.AppendLine("</title>");
@@ -1532,10 +1533,20 @@ function renderSlowestSection() {
     sec.innerHTML = h;
 }
 
+function sortGroups(grps) {
+    if (sortMode === 'duration') return [...grps].sort((a,b) => {
+        const aMax = Math.max.apply(null, a.tests.map(function(t){return t.durationMs}));
+        const bMax = Math.max.apply(null, b.tests.map(function(t){return t.durationMs}));
+        return bMax - aMax;
+    });
+    if (sortMode === 'name') return [...grps].sort((a,b) => a.label.localeCompare(b.label));
+    return grps;
+}
+
 function render() {
     let total = 0;
     let html = '';
-    const displayGroups = computeDisplayGroups();
+    const displayGroups = sortGroups(computeDisplayGroups());
     const limited = displayGroups.slice(0, renderLimit);
     limited.forEach((g,gi)=>{
         const ft = sortTests(g.tests.filter(matchesFilter));
