refactor: Improved coverage to 100%.

Author: hennadiilu

src/Heleonix.Testing.NUnit/Bdd/StoryAttribute.cs

@@ -64,7 +64,7 @@ public sealed class StoryAttribute : FixtureAttribute
         /// </value>
         protected override IDictionary<string, object> Properties => new Dictionary<string, object>
         {
-            { TestPropertiesHelper.OutputPropertyName(nameof(this.TestName)), $"{this.TestName}" },
+            { TestPropertiesHelper.OutputPropertyName(nameof(this.TestName)), this.TestName },
         };
 
         /// <summary>

test/Heleonix.Testing.NUnit.Tests/Aaa/ComponentTestAttributeTests.cs

@@ -23,7 +23,7 @@ public static class ComponentTestAttributeTests
         /// <summary>
         /// Tests the <see cref="ComponentTestAttribute.TestName"/>.
         /// </summary>
-        [Test(Description = "When a component type is provided Should return the test name")]
+        [Test(Description = "When a component type is provided Should return the test name and corresponding properties")]
         public static void TestName1()
         {
             // Arrange
@@ -34,14 +34,19 @@ public static void TestName1()
                 "TestName",
                 BindingFlags.Instance | BindingFlags.NonPublic).GetValue(componentTestAttribute) as string;
 
+            var properties = componentTestAttribute.GetType().GetProperty(
+                "Properties",
+                BindingFlags.Instance | BindingFlags.NonPublic).GetValue(componentTestAttribute) as IDictionary<string, object>;
+
             // Assert
             Assert.That(testName, Is.EqualTo("Int32"));
+            Assert.That(properties["Heleonix.Testing.NUnit.Internal.Output.Type"], Is.EqualTo("Int32"));
         }
 
         /// <summary>
         /// Tests the <see cref="ComponentTestAttribute.TestName"/>.
         /// </summary>
-        [Test(Description = "When a component type is not provided Should return null")]
+        [Test(Description = "When a component type is not provided Should return null and corresponding properties")]
         public static void TestName2()
         {
             // Arrange
@@ -52,8 +57,13 @@ public static void TestName2()
                 "TestName",
                 BindingFlags.Instance | BindingFlags.NonPublic).GetValue(componentTestAttribute) as string;
 
+            var properties = componentTestAttribute.GetType().GetProperty(
+               "Properties",
+               BindingFlags.Instance | BindingFlags.NonPublic).GetValue(componentTestAttribute) as IDictionary<string, object>;
+
             // Assert
             Assert.That(testName, Is.Null);
+            Assert.That(properties["Heleonix.Testing.NUnit.Internal.Output.Type"], Is.Null);
         }
     }
 }

test/Heleonix.Testing.NUnit.Tests/Bdd/StoryAttributeTests.cs

@@ -20,46 +20,24 @@ public static class StoryAttributeTests
         /// <summary>
         /// Tests the <see cref="StoryAttribute.TestName"/>.
         /// </summary>
-        [Test(Description = "When an id and summary are provided Should return the test name")]
+        [Test(Description = "When an id and summary are provided Should return the test name and corresponding properties")]
         public static void TestName()
         {
             // Arrange
-            var componentTestAttribute = new StoryAttribute { Id = "ID-123", Summary = "Some summary" };
+            var storyTestAttribute = new StoryAttribute { Id = "ID-123", Summary = "Some summary" };
 
             // Act
-            var testName = componentTestAttribute.GetType().GetProperty(
+            var testName = storyTestAttribute.GetType().GetProperty(
                 nameof(TestName),
-                BindingFlags.Instance | BindingFlags.NonPublic).GetValue(componentTestAttribute) as string;
+                BindingFlags.Instance | BindingFlags.NonPublic).GetValue(storyTestAttribute) as string;
 
-            // Assert
-            Assert.That(testName, Is.EqualTo("ID-123: Some summary"));
-        }
-
-        /// <summary>
-        /// Tests the <see cref="StoryAttribute.Properties"/>.
-        /// </summary>
-        [Test(Description = "When an AsA, IWant, SoThat are provided Should return properties with those statements")]
-        public static void Properties()
-        {
-            // Arrange
-            var componentTestAttribute = new StoryAttribute
-            {
-                Id = "ID-1",
-                Summary = "Summary",
-                AsA = "PO",
-                IWant = "Everything",
-                SoThat = "I'm ok",
-            };
-
-            // Act
-            var properties = componentTestAttribute.GetType().GetProperty(
-                nameof(Properties),
-                BindingFlags.Instance | BindingFlags.NonPublic).GetValue(componentTestAttribute)
-                    as IDictionary<string, object>;
+            var properties = storyTestAttribute.GetType().GetProperty(
+                "Properties",
+                BindingFlags.Instance | BindingFlags.NonPublic).GetValue(storyTestAttribute) as IDictionary<string, object>;
 
             // Assert
-            Assert.That(properties.Count, Is.EqualTo(1));
-            Assert.That(properties["Heleonix.Testing.NUnit.Internal.Output.TestName"], Is.EqualTo("ID-1: Summary"));
+            Assert.That(testName, Is.EqualTo("ID-123: Some summary"));
+            Assert.That(properties["Heleonix.Testing.NUnit.Internal.Output.TestName"], Is.EqualTo("ID-123: Some summary"));
         }
     }
 }