Merge pull request #157 from nils-a/release/1.0.4

Release/1.0.4
cake-contrib · Sep 25, 2021 · 9a6c3c1 · 9a6c3c1
2 parents ff5e93a + 4072964
commit 9a6c3c1
Show file tree

Hide file tree

Showing 80 changed files with 505 additions and 534 deletions.
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
diff --git a/.github/renovate.json b/.github/renovate.json
@@ -0,0 +1,10 @@
+{
+  "$schema": "https://docs.renovatebot.com/renovate-schema.json",
+  "extends": [ "github>nils-a/renovate-config" ],
+  "packageRules": [
+    {
+      "matchPackageNames": ["cake.tool", "Cake.Core"],
+      "enabled": false
+    }
+  ]
+}
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
@@ -13,6 +13,7 @@ on:
     branches: [develop]
   schedule:
     - cron: '0 15 * * 6'
+  workflow_dispatch:
 
 jobs:
   analyze:
@@ -57,6 +58,18 @@ jobs:
         target: DotNetCore-Build
         cake-version: 0.38.5
         cake-bootstrap: true
+      env:
+        COMPlus_DbgEnableMiniDump: 1
+        COMPlus_DbgMiniDumpType: 1
+        COMPlus_DbgMiniDumpName: BuildArtifacts/coredump.dmp
 
     - name: Perform CodeQL Analysis
       uses: github/codeql-action/analyze@v1
+
+    - name: Upload CoreDump
+      uses: actions/upload-artifact@v2
+      if: failure()
+      with:
+        if-no-files-found: warn
+        name: CoreDump
+        path: BuildArtifacts/coredump.dmp
diff --git a/.github/workflows/dependabot-cake.yml b/.github/workflows/dependabot-cake.yml
diff --git a/demo/frosting/build/Build.csproj b/demo/frosting/build/Build.csproj
@@ -5,7 +5,7 @@
     <RunWorkingDirectory>$(MSBuildProjectDirectory)..</RunWorkingDirectory>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Cake.Frosting" Version="1.0.0" />
+    <PackageReference Include="Cake.Frosting" Version="1.2.0" />
   </ItemGroup>
   <ItemGroup>
     <Reference Include="Cake.7zip, Version=0.8.0.0, Culture=neutral, PublicKeyToken=null">

diff --git a/src/.idea/.idea.Cake.7zip/.idea/indexLayout.xml b/src/.idea/.idea.Cake.7zip/.idea/indexLayout.xml
diff --git a/src/.idea/.idea.Cake.7zip/.idea/riderModule.iml b/src/.idea/.idea.Cake.7zip/.idea/riderModule.iml
diff --git a/src/Cake.7zip.Tests/Cake.7zip.Tests.csproj b/src/Cake.7zip.Tests/Cake.7zip.Tests.csproj
@@ -11,23 +11,22 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Cake.Testing" Version="1.1.0" />
-        <PackageReference Include="coverlet.msbuild" Version="3.0.3">
+        <PackageReference Include="Cake.Testing" Version="1.2.0" />
+        <PackageReference Include="coverlet.msbuild" Version="3.1.0">
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
           <PrivateAssets>all</PrivateAssets>
         </PackageReference>
-        <PackageReference Include="FluentAssertions" Version="5.10.3" />
-        <PackageReference Include="FluentAssertions.Analyzers" Version="0.11.4" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.9.4" />
+        <PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" Version="5.0.3">
+          <PrivateAssets>all</PrivateAssets>
+          <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+        </PackageReference>
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.11.0" />
         <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0">
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
           <PrivateAssets>all</PrivateAssets>
         </PackageReference>
         <PackageReference Include="Moq" Version="4.16.1" />
-        <PackageReference Include="Text.Analyzers" Version="2.6.4">
-            <PrivateAssets>all</PrivateAssets>
-            <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-        </PackageReference>
+        <PackageReference Include="Shouldly" Version="4.0.3" />
         <PackageReference Include="xunit" Version="2.4.1" />
         <PackageReference Include="xunit.analyzers" Version="0.10.0" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">
@@ -39,4 +38,4 @@
     <ItemGroup>
         <ProjectReference Include="..\Cake.7zip\Cake.7zip.csproj" />
     </ItemGroup>
-</Project>
+</Project>
diff --git a/src/Cake.7zip.Tests/Commands/AbstractOutputCommandTests.cs b/src/Cake.7zip.Tests/Commands/AbstractOutputCommandTests.cs
@@ -1,10 +1,10 @@
 using Cake.SevenZip.Commands;
 using Cake.SevenZip.Parsers;
 
-using FluentAssertions;
-
 using Moq;
 
+using Shouldly;
+
 using Xunit;
 
 namespace Cake.SevenZip.Tests.Commands
@@ -24,7 +24,7 @@ public void AbstractOutputCommand_sets_raw_output()
 
             ((ICanParseOutput)partialMock).SetRawOutput(expected);
 
-            actual.Should().BeEquivalentTo(expected);
+            actual.ShouldBeEquivalentTo(expected);
         }
 
         [Fact]
@@ -39,7 +39,7 @@ public void AbstractOutputCommand_sets_nothing_when_called_with_null()
 
             ((ICanParseOutput)partialMock).SetRawOutput(null);
 
-            actual.Should().BeNull();
+            actual.ShouldBeNull();
         }
 
         [Fact]
@@ -56,7 +56,7 @@ public void AbstractOutputCommand_uses_parser_to_set_non_raw_output()
 
             ((ICanParseOutput)partialMock).SetRawOutput(new string[] { });
 
-            actual.Should().Be(expected.Object);
+            actual.ShouldBe(expected.Object);
         }
     }
 }
diff --git a/src/Cake.7zip.Tests/Commands/BenchmarkCommandTests.cs b/src/Cake.7zip.Tests/Commands/BenchmarkCommandTests.cs
@@ -5,7 +5,7 @@
 using Cake.SevenZip.Commands;
 using Cake.SevenZip.Parsers;
 
-using FluentAssertions;
+using Shouldly;
 
 using Xunit;
 
@@ -19,8 +19,8 @@ public void OutputCommand_uses_its_own_parser<T>(BaseOutputCommand<T> command, T
             where T : IOutput
         {
             var actual = command.OutputParser;
-            actual.Should().NotBeNull();
-            actual.Should().BeOfType(expectedParserType);
+            actual.ShouldNotBeNull();
+            actual.ShouldBeOfType(expectedParserType);
         }
 
         private class TestData : IEnumerable<object[]>

diff --git a/src/Cake.7zip.Tests/Commands/CommandExtensionTests.cs b/src/Cake.7zip.Tests/Commands/CommandExtensionTests.cs
@@ -2,7 +2,7 @@
 
 using Cake.SevenZip.Commands;
 
-using FluentAssertions;
+using Shouldly;
 
 using Xunit;
 
@@ -25,7 +25,7 @@ public void Extension_RequireNotNull_throws_for_null()
             // ReSharper disable once ExpressionIsAlwaysNull
             Action action = () => o.RequireNotNull("boom");
 
-            action.Should().Throw<ArgumentException>().WithMessage("boom");
+            action.ShouldThrow<ArgumentException>().Message.ShouldBe("boom");
         }
 
         [Fact]
@@ -52,11 +52,11 @@ public void Extension_RequireNotEmpty_throws_for_empty_enumerables()
         {
             Action action = () => (new object[] { }).RequireNotEmpty("boom");
 
-            action.Should().Throw<ArgumentException>().WithMessage("boom");
+            action.ShouldThrow<ArgumentException>().Message.ShouldBe("boom");
         }
 
         [Fact]
-        public void Extension_RequireNotEmpty_does_not_throw_for_enumerble_alternatives()
+        public void Extension_RequireNotEmpty_does_not_throw_for_enumerable_alternatives()
         {
             // ReSharper disable once CoVariantArrayConversion
             (new object[] { }).RequireNotEmpty("boom", new[] { new[] { "" } }); // two arrays, because of the "params-array"

diff --git a/src/Cake.7zip.Tests/Commands/GeneralOutputCommandTests.cs b/src/Cake.7zip.Tests/Commands/GeneralOutputCommandTests.cs
@@ -5,7 +5,7 @@
 using Cake.SevenZip.Commands;
 using Cake.SevenZip.Parsers;
 
-using FluentAssertions;
+using Shouldly;
 
 using Xunit;
 
@@ -19,8 +19,8 @@ public void OutputCommand_uses_its_own_parser<T>(BaseOutputCommand<T> command, T
             where T : IOutput
         {
             var actual = command.OutputParser;
-            actual.Should().NotBeNull();
-            actual.Should().BeOfType(expectedParserType);
+            actual.ShouldNotBeNull();
+            actual.ShouldBeOfType(expectedParserType);
         }
 
         private class TestData : IEnumerable<object[]>