additional coverage

Author: Gurpreet Singh

.gitignore

@@ -16,7 +16,8 @@ obj/
 packages/*
 PackageBuild/*
 Build/*
-TestResult.xml
+TestResult*
+CoverageReport
 TestStack.BDDfy.sln.ide/graph
 _NCrunch_TestStack.BDDfy/
 TestStack.BDDfy.sln.ide/

src/Samples/TestStack.BDDfy.Samples/Atm/AccountHasInsufficientFund.cs

@@ -10,43 +10,43 @@ public class AccountHasInsufficientFund
 
         // You can override step text using executable attributes
         [Given("Given the Account Balance is $10")]
-        void GivenTheAccountBalanceIs10()
+        internal void GivenTheAccountBalanceIs10()
         {
             _card = new Card(true, 10);
         }
 
-        void And_given_the_Card_is_valid()
+        internal void And_given_the_Card_is_valid()
         {
         }
 
-        void AndGivenTheMachineContainsEnoughMoney()
+        internal void AndGivenTheMachineContainsEnoughMoney()
         {
             _atm = new Atm(100);
         }
 
         [When("When the Account Holder requests $20")]
-        void WhenTheAccountHolderRequests20()
+        internal void WhenTheAccountHolderRequests20()
         {
             _atm.RequestMoney(_card, 20);
         }
 
-        void Then_the_ATM_should_not_dispense_any_Money()
+        internal void Then_the_ATM_should_not_dispense_any_Money()
         {
             _atm.DispenseValue.ShouldBe(0);
         }
 
-        void And_the_ATM_should_say_there_are_Insufficient_Funds()
+        internal void And_the_ATM_should_say_there_are_Insufficient_Funds()
         {
             _atm.Message.ShouldBe(DisplayMessage.InsufficientFunds);
         }
 
         [AndThen("And the Account Balance should be $20")]
-        void AndTheAccountBalanceShouldBe20()
+        internal void AndTheAccountBalanceShouldBe20()
         {
             _card.AccountBalance.ShouldBe(10);
         }
 
-        void And_the_Card_should_be_returned()
+        internal void And_the_Card_should_be_returned()
         {
             _atm.CardIsRetained.ShouldBe(false);
         }
@@ -56,5 +56,12 @@ public void Verify()
         {
             this.BDDfy<AccountHolderWithdrawsCash>();
         }
+
+        [Fact]
+        public void VerifyLazy()
+        {
+            var engine = this.LazyBDDfy<AccountHolderWithdrawsCash>();
+            engine.Run();
+        }
     }
 }

src/TestStack.BDDfy.Tests/Configuration/TestRunnerTests.cs

@@ -2,11 +2,11 @@
 using System.Linq;
 using Shouldly;
 using TestStack.BDDfy.Configuration;
+using TestStack.BDDfy.Tests.Concurrency;
 using Xunit;
 
 namespace TestStack.BDDfy.Tests.Configuration
 {
-    [Collection("ExclusiveAccessToConfigurator")]
     public class TestRunnerTests
     {
         public class ScenarioWithFailingThen
@@ -33,6 +33,9 @@ public void PassingAndThen()
             }
         }
 
+
+        [Collection(TestCollectionName.ModifiesConfigurator)]
+
         public class When_StopExecutionOnFailingThen_IsSetToTrue
         {
             [Fact]

src/TestStack.BDDfy/BDDfyExtensions.cs

@@ -50,12 +50,20 @@ public static Engine LazyBDDfy<TStory>(this object testObject, string scenarioTi
         /// <param name="scenarioTitle">Overrides the default scenario title and is displayed in the reports.</param>
         /// <param name="caller">Caller (populated by [CallerMemberName])</param>
         /// <returns></returns>
-        public static Story BDDfy(this object testObject, string scenarioTitle = null, [System.Runtime.CompilerServices.CallerMemberName] string caller = null)
+        public static Story BDDfy(
+            this object testObject, 
+            string scenarioTitle = null, 
+            [System.Runtime.CompilerServices.CallerMemberName] 
+            string caller = null)
         {
             return InternalLazyBDDfy(testObject, scenarioTitle ?? Configurator.Humanizer.Humanize(caller)).Run();
         }
 
-        public static Engine LazyBDDfy(this object testObject, string scenarioTitle = null, [System.Runtime.CompilerServices.CallerMemberName] string caller = null)
+        public static Engine LazyBDDfy(
+            this object testObject, 
+            string scenarioTitle = null, 
+            [System.Runtime.CompilerServices.CallerMemberName] 
+            string caller = null)
         {
             return InternalLazyBDDfy(testObject, scenarioTitle ?? Configurator.Humanizer.Humanize(caller));
         }
@@ -68,22 +76,30 @@ public static Engine LazyBDDfy(this object testObject, string scenarioTitle = nu
         /// <param name="scenarioTitle">Overrides the default scenario title and is displayed in the reports.</param>
         /// <param name="caller">Caller (populated by [CallerMemberName])</param>
         /// <returns></returns>
-        public static Story BDDfy<TStory>(this object testObject, string scenarioTitle = null, [System.Runtime.CompilerServices.CallerMemberName] string caller = null)
-            where TStory : class
+        public static Story BDDfy<TStory>(
+            this object testObject,
+            string scenarioTitle = null,
+            [System.Runtime.CompilerServices.CallerMemberName]
+            string caller = null)
+        where TStory : class
         {
             return InternalLazyBDDfy(testObject, scenarioTitle ?? Configurator.Humanizer.Humanize(caller), typeof(TStory)).Run();
         }
 
-        public static Engine LazyBDDfy<TStory>(this object testObject, string scenarioTitle = null, [System.Runtime.CompilerServices.CallerMemberName] string caller = null)
-            where TStory : class
+        public static Engine LazyBDDfy<TStory>(
+            this object testObject,
+            string scenarioTitle = null,
+            [System.Runtime.CompilerServices.CallerMemberName]
+            string caller = null)
+        where TStory : class
         {
             return InternalLazyBDDfy(testObject, scenarioTitle ?? Configurator.Humanizer.Humanize(caller), typeof(TStory));
         }
 #endif
 
         static Engine InternalLazyBDDfy(
-            object testObject, 
-            string scenarioTitle, 
+            object testObject,
+            string scenarioTitle,
             Type explicitStoryType = null)
         {
             var testContext = TestContext.GetContext(testObject);

src/TestStack.BDDfy/Properties/Annotations.cs

@@ -331,6 +331,7 @@ public enum ImplicitUseTargetFlags
     /// which should not be removed and so is treated as used
     /// </summary>
     [MeansImplicitUse]
+    [ExcludeFromCodeCoverage]
     public sealed class PublicAPIAttribute : Attribute
     {
         public PublicAPIAttribute() { }