Improved Html reporting, much happier with this. Except the styles, they still blow

Author: Jake Ginnivan

TestStack.ConventionTests/Convention.cs

@@ -4,6 +4,7 @@
     using System.Collections.Generic;
     using System.Reflection;
     using System.Text.RegularExpressions;
+    using TestStack.ConventionTests.ConventionData;
     using TestStack.ConventionTests.Internal;
     using TestStack.ConventionTests.Reporting;
 
@@ -50,7 +51,7 @@ static void Execute<TDataSource>(IConvention<TDataSource> convention, TDataSourc
             IResultsProcessor[] processors, ITestResultProcessor resultProcessor)
             where TDataSource : IConventionData
         {
-            var dataDescription = string.Format("{0} in {1}", ToSentenceCase(data.GetType().Name), data.Description);
+            var dataDescription = string.Format("{0} in {1}", data.GetType().GetSentenceCaseName(), data.Description);
             var context = new ConventionContext(dataDescription, Formatters, processors, resultProcessor);
             context.Execute(convention, data);
         }
@@ -76,10 +77,5 @@ static void Init(Assembly assembly)
             defaultProcessors[defaultProcessors.Length - 1] = new ThrowOnFailureResultsProcessor();
             defaultApprovalProcessors[defaultProcessors.Length - 1] = new ApproveResultsProcessor();
         }
-
-        static string ToSentenceCase(string str)
-        {
-            return Regex.Replace(str, "[a-z][A-Z]", m => m.Value[0] + " " + char.ToLower(m.Value[1]));
-        }
     }
 }

TestStack.ConventionTests/ConventionData/TypeExtensions.cs

@@ -4,6 +4,7 @@
     using System.Collections.Generic;
     using System.Linq;
     using System.Reflection;
+    using System.Text.RegularExpressions;
 
     public static class TypeExtensions
     {
@@ -66,5 +67,10 @@ public static bool ClosesInterface(this Type t, Type openGeneric)
         {
             return t.GetClosedInterfacesOf(openGeneric).Any();
         }
+
+        internal static string GetSentenceCaseName(this Type type)
+        {
+            return Regex.Replace(type.Name, "[a-z][A-Z]", m => m.Value[0] + " " + char.ToLower(m.Value[1]));
+        }
     }
 }

TestStack.ConventionTests/Reporting/HtmlConventionResultsReporter.cs

@@ -5,6 +5,7 @@
     using System.Linq;
     using System.Text;
     using System.Web.UI;
+    using TestStack.ConventionTests.ConventionData;
     using TestStack.ConventionTests.Internal;
 
     public class HtmlConventionResultsReporter : GroupedByDataTypeConventionResultsReporterBase
@@ -61,28 +62,35 @@ protected override string Process(IConventionFormatContext context, IEnumerable<
                         conventionResult.DataDescription.Replace("'", string.Empty).Replace(" ", string.Empty).Replace(".", string.Empty);
                     html.AddAttribute("style", "margin-left:20px;");
                     html.RenderBeginTag(HtmlTextWriterTag.H4);
-
-                    html.AddAttribute("class", "menu-toggle");
-                    html.AddAttribute("data-toggle", "collapse");
-                    html.AddAttribute("data-target", "." + targetId);
-                    html.RenderBeginTag(HtmlTextWriterTag.A);
-                    html.AddAttribute("class", "icon-angle-down");
-                    html.RenderBeginTag(HtmlTextWriterTag.I);
-                    html.RenderEndTag();
-                    html.RenderEndTag();
                     html.Write(conventionResult.ConventionTitle);
                     html.RenderEndTag();
-                    html.AddAttribute("class", targetId + " collapse");
-                    html.AddAttribute("style", "margin-left:20px;");
-                    html.RenderBeginTag(HtmlTextWriterTag.Div);
-                    html.RenderBeginTag(HtmlTextWriterTag.Ul);
-                    foreach (var o in conventionResult.Data)
+                    if (conventionResult.Data.Any())
                     {
-                        html.RenderBeginTag(HtmlTextWriterTag.Li);
-                        html.Write(context.FormatDataAsHtml(o));
+                        html.AddAttribute("style", "margin-left:20px;");
+                        html.RenderBeginTag(HtmlTextWriterTag.Div);
+                            html.AddAttribute("class", "menu-toggle");
+                            html.AddAttribute("data-toggle", "collapse");
+                            html.AddAttribute("data-target", "." + targetId);
+                            html.RenderBeginTag(HtmlTextWriterTag.A);
+                                html.AddAttribute("class", "icon-angle-down");
+                                html.RenderBeginTag(HtmlTextWriterTag.I);
+                                html.RenderEndTag();
+                                html.Write("With the exception of the following {0}: ", conventionResult.DataType.GetSentenceCaseName());
+                            html.RenderEndTag();
+                        html.AddAttribute("class", targetId + " collapse");
+                        html.AddAttribute("style", "margin-left:20px;");
+                        html.RenderBeginTag(HtmlTextWriterTag.Div);
+                            html.RenderBeginTag(HtmlTextWriterTag.Ul);
+                            foreach (var o in conventionResult.Data)
+                            {
+                                html.RenderBeginTag(HtmlTextWriterTag.Li);
+                                html.Write(context.FormatDataAsHtml(o));
+                                html.RenderEndTag();
+                            }
+                            html.RenderEndTag();
                         html.RenderEndTag();
                     }
-                    html.RenderEndTag();
+                    
                     html.RenderEndTag();
                 }
                 html.RenderEndTag();