Fixing markdown report too

Jake Ginnivan · Jake Ginnivan · commit c54b23d5ff63 · 2014-05-02T16:39:58.000+01:00
diff --git a/TestStack.BDDfy.Tests/Reporters/MarkDown/MarkDownReportBuilderTests.ShouldProduceExpectedMarkdownWithExamples.approved.txt b/TestStack.BDDfy.Tests/Reporters/MarkDown/MarkDownReportBuilderTests.ShouldProduceExpectedMarkdownWithExamples.approved.txt
@@ -10,12 +10,12 @@
 
 ### Examples: 
 
-    | sign     | action | Result | Errors                                                                                       |
-    | positive | is     | Passed |                                                                                              |
-    | negative | is not | Failed | Step: Then money &lt;action&gt; dispensed failed with exception: [Boom] [Details at 1 below] |
+    | sign     | action | Result | Errors                                                                                                      |
+    | positive | is     | Passed |                                                                                                             |
+    | negative | is not | Failed | Step: Then money &lt;action&gt; dispensed failed with exception: [Boom With New lines] [Details at 1 below] |
 
 #### Exceptions:
-    1. Boom
+    1. Boom With New lines
        at TestStack.BDDfy.Tests.Reporters.ReportTestData.GetScenarios(Boolean includeFailingScenario, Boolean includeExamples) in ...\ReportTestData.cs:line 113
 
 ## Story: Happiness
diff --git a/TestStack.BDDfy.Tests/Reporters/TextReporter/TextReporterTests.ShouldProduceExpectedTextWithExamples.approved.txt b/TestStack.BDDfy.Tests/Reporters/TextReporter/TextReporterTests.ShouldProduceExpectedTextWithExamples.approved.txt
@@ -25,12 +25,12 @@ Scenario: Example Scenario
 	Then money <action> dispensed         
 
 Examples: 
-| sign     | action | Result | Errors                                                                                 |
-| positive | is     | Passed |                                                                                        |
-| negative | is not | Failed | Step: Then money <action> dispensed failed with exception: [Boom] [Details at 1 below] |
+| sign     | action | Result | Errors                                                                                                |
+| positive | is     | Passed |                                                                                                       |
+| negative | is not | Failed | Step: Then money <action> dispensed failed with exception: [Boom With New lines] [Details at 1 below] |
 
 Exceptions:
-  1. Boom
+  1. Boom With New lines
    at TestStack.BDDfy.Tests.Reporters.ReportTestData.GetScenarios(Boolean includeFailingScenario, Boolean includeExamples) in ...\ReportTestData.cs
 
 
diff --git a/TestStack.BDDfy.Tests/Reporters/TextReporter/TextReporterTests.cs b/TestStack.BDDfy.Tests/Reporters/TextReporter/TextReporterTests.cs
@@ -29,7 +29,7 @@ public void ShouldProduceExpectedReport()
 
         [Test]
         [MethodImpl(MethodImplOptions.NoInlining)]
-        public void ShouldProduceExpectedMarkdownWithExamples()
+        public void ShouldProduceExpectedTextWithExamples()
         {
             var stories = new ReportTestData().CreateTwoStoriesEachWithOneFailingScenarioAndOnePassingScenarioWithThreeStepsOfFiveMillisecondsAndEachHasTwoExamples();
             var actual = new StringBuilder();
diff --git a/TestStack.BDDfy.Tests/TestStack.BDDfy.Tests.csproj b/TestStack.BDDfy.Tests/TestStack.BDDfy.Tests.csproj
@@ -175,7 +175,7 @@
     <Content Include="Reporters\Html\MetroReportBuilderTests.ShouldProduceExpectedHtmlWithExamples.approved.txt" />
     <Content Include="Reporters\MarkDown\MarkDownReportBuilderTests.ShouldProduceExpectedMarkdown.approved.txt" />
     <Content Include="Reporters\MarkDown\MarkDownReportBuilderTests.ShouldProduceExpectedMarkdownWithExamples.approved.txt" />
-    <Content Include="Reporters\TextReporter\TextReporterTests.ShouldProduceExpectedMarkdownWithExamples.approved.txt" />
+    <Content Include="Reporters\TextReporter\TextReporterTests.ShouldProduceExpectedTextWithExamples.approved.txt" />
     <Content Include="Reporters\TextReporter\TextReporterTests.ShouldProduceExpectedReport.approved.txt" />
   </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
diff --git a/TestStack.BDDfy/Reporters/MarkDown/MarkDownReportBuilder.cs b/TestStack.BDDfy/Reporters/MarkDown/MarkDownReportBuilder.cs
@@ -171,11 +171,10 @@ void ReportExceptions(StringBuilder report)
 
         static string FlattenExceptionMessage(string message)
         {
-            return message
+            return string.Join(" ", message
                 .Replace("\t", " ") // replace tab with one space
-                .Replace(Environment.NewLine, ", ") // replace new line with one space
-                .Trim() // trim starting and trailing spaces
-                .Replace("  ", " ")
+                .Split(new[] { "\r\n", "\n" }, StringSplitOptions.None)
+                .Select(s => s.Trim()))
                 .TrimEnd(','); // chop any , from the end
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public void ShouldProduceExpectedReport()`
`29`	`29`
`30`	`30`	`[Test]`
`31`	`31`	`[MethodImpl(MethodImplOptions.NoInlining)]`
`32`		`- public void ShouldProduceExpectedMarkdownWithExamples()`
	`32`	`+ public void ShouldProduceExpectedTextWithExamples()`
`33`	`33`	`{`
`34`	`34`	`var stories = new ReportTestData().CreateTwoStoriesEachWithOneFailingScenarioAndOnePassingScenarioWithThreeStepsOfFiveMillisecondsAndEachHasTwoExamples();`
`35`	`35`	`var actual = new StringBuilder();`
Original file line number	Diff line number	Diff line change
`@@ -171,11 +171,10 @@ void ReportExceptions(StringBuilder report)`
`171`	`171`
`172`	`172`	`static string FlattenExceptionMessage(string message)`
`173`	`173`	`{`
`174`		`- return message`
	`174`	`+ return string.Join(" ", message`
`175`	`175`	`.Replace("\t", " ") // replace tab with one space`
`176`		`- .Replace(Environment.NewLine, ", ") // replace new line with one space`
`177`		`- .Trim() // trim starting and trailing spaces`
`178`		`- .Replace(" ", " ")`
	`176`	`+ .Split(new[] { "\r\n", "\n" }, StringSplitOptions.None)`
	`177`	`+ .Select(s => s.Trim()))`
`179`	`178`	`.TrimEnd(','); // chop any , from the end`
`180`	`179`	`}`
`181`	`180`	`}`