additional code coverage

Author: Gurpreet Singh

.github/workflows/build.yml

@@ -73,7 +73,8 @@ jobs:
           dotnet test 
           --configuration Release 
           --collect:"XPlat Code Coverage" 
-          --results-directory ../coverage
+          --results-directory ../coverage `
+          --settings ./default.runsettings
 
       - name: Generate coverage report
         uses: danielpalme/[email protected]

coverage-by-samples.ps1

@@ -0,0 +1,11 @@
+dotnet test ./src/Samples/TestStack.BDDfy.Samples/TestStack.BDDfy.Samples.csproj `
+    --collect:"XPlat Code Coverage" `
+    --results-directory ./TestResults `
+    --settings ./src/default.runsettings
+
+reportgenerator `
+    -reports:./TestResults/**/coverage.cobertura.xml `
+    -targetdir:./CoverageReport `
+    -reporttypes:Html
+
+Remove-Item -Recurse -Force ./TestResults

coverage-by-tests.ps1

@@ -0,0 +1,11 @@
+dotnet test ./src/TestStack.BDDfy.Tests/TestStack.BDDfy.Tests.csproj `
+    --collect:"XPlat Code Coverage" `
+    --results-directory ./TestResults `
+    --settings ./src/default.runsettings
+
+reportgenerator `
+    -reports:./TestResults/**/coverage.cobertura.xml `
+    -targetdir:./CoverageReport `
+    -reporttypes:Html
+
+Remove-Item -Recurse -Force ./TestResults

coverage-report.ps1 coverage.ps1coverage-report.ps1 renamed to coverage.ps1

@@ -1,6 +1,7 @@
 dotnet test ./src `
     --collect:"XPlat Code Coverage" `
-    --results-directory ./TestResults
+    --results-directory ./TestResults `
+    --settings ./src/default.runsettings
 
 reportgenerator `
     -reports:./TestResults/**/coverage.cobertura.xml `

src/Samples/TestStack.BDDfy.Samples/CanRunAsyncVoidSteps.cs

@@ -0,0 +1,18 @@
+using System.Threading.Tasks;
+using TestStack.BDDfy.Configuration;
+using Xunit;
+
+namespace TestStack.BDDfy.Samples
+{
+    public class CanRunAsyncVoidSteps
+    {
+        [Fact]
+        internal void Run() => this.BDDfy();
+        internal void SetUp() => Configurator.AsyncVoidSupportEnabled = false;
+        internal void TearDown() => Configurator.AsyncVoidSupportEnabled = true;
+
+        internal async void GivenNonAsyncStep() => await Task.CompletedTask;
+        internal async void WhenSomethingHappens() => await Task.CompletedTask;
+        internal async void ThenAssertSomething()  => await Task.CompletedTask;
+    }
+}

src/TestStack.BDDfy.Tests/Concurrency/TestCollectionName.cs

@@ -2,5 +2,6 @@
 {
     internal static class TestCollectionName { 
         public const string ModifiesConfigurator = "ModifiesConfigurator";
+        public const string Approvals = "Approvals";
     }
 }

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

@@ -0,0 +1,26 @@
+using System;
+using System.Reflection;
+using Shouldly;
+using TestStack.BDDfy.Processors;
+using Xunit;
+
+namespace TestStack.BDDfy.Tests.Configuration;
+
+public class ExceptionResolverTests
+{
+    [Fact]
+    public void WhenTargetInvocationException_ResolveRoot_WhenInnerNotAvailable()
+    {
+        var ex = new TargetInvocationException(null);
+        var resolved = ExceptionResolver.Resolve(ex);
+        resolved.ShouldBeOfType<TargetInvocationException>();
+    }
+
+    [Fact]
+    public void WhenNotTargetInvocationException_ResolveRoot_EvenWhenInnerIsAvailable()
+    {
+        var ex = new System.InvalidCastException("error", new AmbiguousMatchException());
+        var resolved = ExceptionResolver.Resolve(ex);
+        resolved.ShouldBeOfType<InvalidCastException>();
+    }
+}

src/TestStack.BDDfy.Tests/HumanizerTests.cs

@@ -88,5 +88,14 @@ public void ReportsIllegalExampleStepNames(string stepName, string expectedStepT
             exception.ShouldNotBeNull();
             exception.ShouldBeOfType<ArgumentException>();
         }
+
+        [Theory]
+        [InlineData("")]
+        [InlineData(" ")]
+        [InlineData(null)]
+        internal void HumanizeWithNullOrEmptyInput_ReturnsTheSame(string providedInput)
+        {
+            Humanizer.Humanize(providedInput).ShouldBe(providedInput);
+        }
     }
 }

src/TestStack.BDDfy.Tests/Reporters/Html/ClassicReportBuilderTests.ShouldProduceExpectedHtml.approved.txt

@@ -159,7 +159,7 @@ body{margin:0;padding:0;padding-bottom:40px;max-width:100%;background-color:#fff
       </div>
     </div>
     <div class='footer'>Powered by <a href='https://github.com/TestStack/TestStack.BDDfy'>BDDfy</a> framework</div>
-    <script type='text/javascript' src='http://code.jquery.com/jquery-2.1.0.min.js'></script>
+    <script type='text/javascript' src='https://code.jquery.com/jquery-2.1.0.min.js'></script>
     <script>
 $(function(){$(".canToggle").each(function(){var n=$("#"+$(this).data("toggle-target"));n.hide();$(this).click(function(){n.toggle(200)})});$(".expandAll").click(function(){$(".steps").css("display","")});$(".collapseAll").click(function(){$(".steps").css("display","none")});$("ul.resultSummary li:not('.storySummary'):not('.scenarioSummary')").append("<input type='checkbox' class='cbx_toggle' checked/>");$(".cbx_toggle").click(function(){var n=$(this),t=n.closest("li").attr("class");$("#testResult div.scenario ."+t).parent().toggle(n.is(":checked"))})});
 /*

src/TestStack.BDDfy.Tests/Reporters/Html/ClassicReportBuilderTests.cs

@@ -1,12 +1,13 @@
-#if Approvals
 using System;
 using System.Runtime.CompilerServices;
 using TestStack.BDDfy.Reporters;
 using TestStack.BDDfy.Reporters.Html;
+using TestStack.BDDfy.Tests.Concurrency;
 using Xunit;
 
 namespace TestStack.BDDfy.Tests.Reporters.Html
 {
+    [Trait("Category", TestCollectionName.Approvals)]
     public class ClassicReportBuilderTests
     {
         [Fact]
@@ -38,5 +39,4 @@ public void ShouldProduceExpectedHtmlWithExamples()
             ReportApprover.Approve(model, sut);
         }
     }
-}
-#endif
+}