Merge caa3ee5 into 98d64b4

.appveyor.yml

@@ -2,11 +2,10 @@ version: '{build}'
 skip_branch_with_pr: true
 image: Visual Studio 2017
 install:
-  - ps: nuget install OpenCover -OutputDirectory packages -Version 4.7.922
   - ps: dotnet tool install coveralls.net --tool-path tools
 build_script:  
   - cmd: dotnet build src/Castle.Sdk -c Release -f netstandard2.0
   - cmd: dotnet build src/Castle.Sdk -c Release -f net461
-test_script:  
-  - ps: .\packages\OpenCover.4.7.922\tools\OpenCover.Console.exe -register:user -target:dotnet.exe -targetargs:"test src/Tests/Tests.csproj" -filter:"+[Castle.Sdk*]*" -oldStyle -output:opencoverCoverage.xml
-  - ps: .\tools\csmacnz.coveralls.exe --opencover -i opencoverCoverage.xml --useRelativePaths --repoToken $env:COVERALLS_REPO_TOKEN --commitId $env:APPVEYOR_REPO_COMMIT --commitBranch $env:APPVEYOR_REPO_BRANCH --commitAuthor $env:APPVEYOR_REPO_COMMIT_AUTHOR --commitEmail $env:APPVEYOR_REPO_COMMIT_AUTHOR_EMAIL --commitMessage $env:APPVEYOR_REPO_COMMIT_MESSAGE --jobId $env:APPVEYOR_JOB_ID
+test_script:    
+  - ps: dotnet test /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:Include="[Castle.Sdk*]*" src/Tests/Tests.csproj
+  - ps: .\tools\csmacnz.coveralls.exe --opencover -i src/Tests/coverage.opencover.xml --useRelativePaths --repoToken $env:COVERALLS_REPO_TOKEN --commitId $env:APPVEYOR_REPO_COMMIT --commitBranch $env:APPVEYOR_REPO_BRANCH --commitAuthor $env:APPVEYOR_REPO_COMMIT_AUTHOR --commitEmail $env:APPVEYOR_REPO_COMMIT_AUTHOR_EMAIL --commitMessage $env:APPVEYOR_REPO_COMMIT_MESSAGE --jobId $env:APPVEYOR_JOB_ID  

src/Tests/When_converting_http_messages.cs → src/Tests/Sending/When_converting_http_messages.cs

@@ -6,7 +6,7 @@
 using Tests.SetUp;
 using Xunit;
 
-namespace Tests
+namespace Tests.Sending
 {
     public class When_converting_http_messages
     {

src/Tests/When_creating_message_sender.cs → src/Tests/Sending/When_creating_message_sender.cs

@@ -5,7 +5,7 @@
 using Tests.SetUp;
 using Xunit;
 
-namespace Tests
+namespace Tests.Sending
 {
     public class When_creating_message_sender
     {

src/Tests/When_sending_requests.cs → src/Tests/Sending/When_sending_requests.cs

@@ -10,7 +10,7 @@
 using Tests.SetUp;
 using Xunit;
 
-namespace Tests
+namespace Tests.Sending
 {
     public class When_sending_requests
     {

src/Tests/When_sending_with_no_track.cs → src/Tests/Sending/When_sending_with_no_track.cs

@@ -4,7 +4,7 @@
 using FluentAssertions;
 using Xunit;
 
-namespace Tests
+namespace Tests.Sending
 {
     public class When_sending_with_no_track
     {

src/Tests/Tests.csproj

@@ -10,14 +10,18 @@
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
-    <DebugType>full</DebugType>
+    <DebugType>portable</DebugType>
     <DebugSymbols>true</DebugSymbols>
   </PropertyGroup>
 
   <ItemGroup>
     <PackageReference Include="AutoFixture" Version="4.8.0" />
     <PackageReference Include="AutoFixture.AutoNSubstitute" Version="4.8.0" />
     <PackageReference Include="AutoFixture.Xunit2" Version="4.8.0" />
+    <PackageReference Include="coverlet.msbuild" Version="2.6.0">
+      <PrivateAssets>all</PrivateAssets>
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
+    </PackageReference>
     <PackageReference Include="FluentAssertions" Version="5.6.0" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.9.0" />
     <PackageReference Include="NSubstitute" Version="4.0.0" />

src/Tests/When_creating_configuration.cs

@@ -15,6 +15,14 @@ public void Should_always_add_Cookie_when_setting_blacklist(CastleConfiguration
             configuration.Blacklist.Should().Contain("Cookie");
         }
 
+        [Theory, AutoFakeData]
+        public void Should_preserve_Cookie_when_setting_blacklist_to_null(CastleConfiguration configuration)
+        {
+            configuration.Blacklist = null;
+
+            configuration.Blacklist.Should().Contain("Cookie");
+        }
+
         [Fact]
         public void Should_be_able_to_get_recommended_whitelist()
         {