Fixing issue with empty string in examples causing value not to be used error, and also empty string gets turned into null (we only want that behaviour for text table)

Author: Jake Ginnivan

TestStack.BDDfy.Tests/Scanner/FluentScanner/FluentWithExamples.FluentCanBeUsedWithExamples.approved.txt

@@ -5,10 +5,11 @@ Scenario: Fluent can be used with examples
 	  Given a different method with random arg 2
 	  Given a different method with <Prop2>     
 	When method using <example string>          
+	And I use a <Multi word heading>            
 	Then all is good                            
 
 Examples: 
-| Prop 1 | Prop2 | Prop 3               |
-| 1      | foo   | ConsecutiveAssertion |
-| 2      | bar   | Initialize           |
+| Prop 1 | Prop2 | Prop 3               | Multi word heading |
+| 1      | foo   | ConsecutiveAssertion |                    |
+| 2      | bar   | Initialize           | val2               |
 

TestStack.BDDfy.Tests/Scanner/FluentScanner/FluentWithExamples.cs

@@ -17,11 +17,12 @@ public void FluentCanBeUsedWithExamples()
                 .And(_ => GivenADifferentMethodWithRandomArg(2))
                 .And(_ => GivenADifferentMethodWith(_prop2))
                 .When(_ => WhenMethodUsing__ExampleString__())
+                .When(_ => AndIUseA(multiWordHeading))
                 .Then(_ => ThenAllIsGood())
-                .WithExamples(new ExampleTable("Prop 1", "Prop2", "Prop 3")
+                .WithExamples(new ExampleTable("Prop 1", "Prop2", "Prop 3", "Multi word heading")
                 {
-                    {1, "foo", ExecutionOrder.ConsecutiveAssertion },
-                    {2, "bar", ExecutionOrder.Initialize }
+                    {1, "foo", ExecutionOrder.ConsecutiveAssertion, "" },
+                    {2, "bar", ExecutionOrder.Initialize, "val2" }
                 })
                 .BDDfy();
 
@@ -30,6 +31,12 @@ public void FluentCanBeUsedWithExamples()
             Approvals.Verify(textReporter.ToString());
         }
 
+        private void AndIUseA(string multiWordHeading)
+        {
+            multiWordHeading.ShouldBeOneOf("", "val2");
+            this.multiWordHeading.ShouldBeOneOf("", "val2");
+        }
+
         private void GivenADifferentMethodWith(string prop2)
         {
             _prop2.ShouldBeOneOf("foo", "bar");
@@ -58,6 +65,7 @@ private void GivenMethodTaking__ExampleInt__(int exampleInt)
 
         public int Prop1 { get; set; }
         private string _prop2 = null;
+        private string multiWordHeading = null;
         public ExecutionOrder Prop_3 { get; set; }
     }
 }

TestStack.BDDfy/Scanners/StepScanners/Examples/ExampleTable.cs

@@ -81,7 +81,7 @@ public static ExampleTable Parse(string table)
 
         static object[] GetValues(string row)
         {
-            return row.Split('|').Select(h => (object)h.Trim()).ToArray();
+            return row.Split('|').Select(h => h.Trim()).Select(s=>string.IsNullOrEmpty(s) ? null : (object)s).ToArray();
         }
 
         public static bool HeaderMatches(string header, string name)

TestStack.BDDfy/Scanners/StepScanners/Examples/ExampleValue.cs

@@ -32,10 +32,9 @@ public int Row
         public object GetValue(Type targetType)
         {
             var stringValue = _underlyingValue as string;
-            if (_underlyingValue == null || (_underlyingValue is string && string.IsNullOrEmpty(stringValue)))
+            if (_underlyingValue == null)
             {
-                if (targetType.IsValueType &&
-                    !(targetType.IsGenericType && targetType.GetGenericTypeDefinition() == typeof (Nullable<>)))
+                if (targetType.IsValueType && !(targetType.IsGenericType && targetType.GetGenericTypeDefinition() == typeof (Nullable<>)))
                 {
                     var valueAsString = string.IsNullOrEmpty(stringValue) ? "<null>" : string.Format("\"{0}\"", _underlyingValue);
                     throw new ArgumentException(string.Format("Cannot convert {0} to {1} (Column: '{2}', Row: {3})", valueAsString, targetType.Name, Header, Row));