Merge pull request #42 from MehdiK/fixing-null-error

Fixes #41 - html report failing when there's no story

Author: mwhelan

Samples/TestStack.BDDfy.Samples/CanWorkWithoutAStory.cs

@@ -0,0 +1,25 @@
+using NUnit.Framework;
+
+namespace TestStack.BDDfy.Samples
+{
+    public class CanWorkWithoutAStory
+    {
+        public void Given_no_story_is_provided()
+        {
+        }
+
+        public void When_we_BDDfy_the_class()
+        {
+        }
+
+        public void Then_the_namespace_is_used_in_the_report()
+        {
+        }
+
+        [Test]
+        public void Execute()
+        {
+            this.BDDfy();
+        }
+    }
+}

Samples/TestStack.BDDfy.Samples/TestStack.BDDfy.Samples.csproj

@@ -51,6 +51,7 @@
     <Compile Include="Atm\HtmlReportConfig.cs" />
     <Compile Include="BDDfyConfiguration.cs" />
     <Compile Include="BDDfy_Rocks.cs" />
+    <Compile Include="CanWorkWithoutAStory.cs" />
     <Compile Include="TicTacToe\Game.cs" />
     <Compile Include="TicTacToe\Helpers.cs" />
     <Compile Include="CustomTextReporter.cs" />
@@ -88,4 +89,4 @@
   <Target Name="AfterBuild">
   </Target>
   -->
-</Project>
+</Project>

TestStack.BDDfy/Processors/Reporters/Html/HtmlReporter.cs

@@ -33,7 +33,6 @@ public void Process(IEnumerable<Story> stories)
 
         void WriteOutHtmlReport(IEnumerable<Story> stories)
         {
-            const string error = "There was an error compiling the html report: ";
             var viewModel = new HtmlReportViewModel(_configuration, stories);
             ShouldTheReportUseCustomization(viewModel);
             string report;
@@ -44,7 +43,7 @@ void WriteOutHtmlReport(IEnumerable<Story> stories)
             }
             catch (Exception ex)
             {
-                report = error + ex.Message;
+                report = ex.Message + ex.StackTrace;
             }
 
             _writer.OutputReport(report, _configuration.OutputFileName, _configuration.OutputPath);

TestStack.BDDfy/Processors/StoryCache.cs

@@ -16,7 +16,10 @@ public void Process(Story story)
         {
             foreach (var scenario in story.Scenarios)
             {
-                scenario.TestObject = null;
+                // ToDo: should uncomment this and find a way to generate file reports with null TestObject.
+                // Currently it fails when there is no story as the namespace is resolved through TestObject
+                // scenario.TestObject = null;
+
                 foreach (var step in scenario.Steps)
                     step.StepAction = null;
             }