Merge branch 'release/0.25.0' into main

* release/0.25.0:
  (build) Updated version and release notes.
  GH-1997: Add support for --trace command line option for NUnit3 settings + runner
  (GH-1995) Make In-Proc NuGet addin/tool installtion default
  Adds missing .NET CLI dependencies/restore switches
  (GH-1937) Speed up in-process NuGet
  Update MSBuildVerbosityExtensions.cs
  (GH-1992) Update to .NET Core 1.0.9 / SDK 1.1.7
  GH-1987: Normalize exception handling in PropertyToken for invalid parameter index
  [ci skip] Get more Open Source Helpers
  Removed unnecessary parameter
  Fix writing start and end progress messages
  (GH-1974) Added System.Collections work around for Mono
  (GH-1689) Add documentation that Chocolatey licensed is required
  (build) Updated Cake tool to version 0.24.0

Author: devlead

README.md

@@ -2,6 +2,7 @@
 
 [![NuGet](https://img.shields.io/nuget/v/Cake.svg)](https://www.nuget.org/packages/Cake) [![MyGet](https://img.shields.io/myget/cake/vpre/Cake.svg?label=myget)](https://www.myget.org/gallery/cake) [![Chocolatey](https://img.shields.io/chocolatey/v/Cake.portable.svg)](https://chocolatey.org/packages/cake.portable)
 [![homebrew](https://img.shields.io/homebrew/v/cake.svg)](http://braumeister.org/formula/cake)
+[![Help Contribute to Open Source](https://www.codetriage.com/cake-build/cake/badges/users.svg)](https://www.codetriage.com/cake-build/cake)
 
 [![Source Browser](https://img.shields.io/badge/Browse-Source-green.svg)](http://sourcebrowser.io/Browse/cake-build/cake)
 

ReleaseNotes.md

@@ -1,3 +1,18 @@
+### New in 0.25.0 (Released 2018/01/17)
+
+* 1995 Make In-proc NuGet addin/tool installation default
+* 1994 Get MSBuild Verbosity enum from string
+* 1988 TeamCity writing start and end progress contains invalid messages property
+* 1974 ToDictionary on Mono causes "The type 'Dictionary<,>' is defined in an assembly that is not referenced"
+* 1998 Some .NET Core commands missing no dependencies/restore
+* 1997 Add the --trace option to the NUnit3Settings class.
+* 1992 Update to .NET Runtime 1.0.9 because security issues
+* 1989 Path unnecessarily trims backslash in already normalized string
+* 1987 Confusing Error from Bad Format String to Information()
+* 1937 UseInProcessClient=true is slow
+* 1982 CodeTriage - Get more Open Source Helpers
+* 1689 ChocolateyDownload should be documented to only work in paid edition
+
 ### New in 0.24.0 (Released 2017/12/29)
 
 * 1950 Allow Cake modules to be bootstrapped by Cake in a pre-processing phase

build.ps1

@@ -31,9 +31,9 @@ Param(
     [string[]]$ScriptArgs
 )
 
-$CakeVersion = "0.23.0"
+$CakeVersion = "0.24.0"
 $DotNetChannel = "Current";
-$DotNetVersion = "1.1.5";
+$DotNetVersion = "1.1.7";
 $DotNetInstallerUri = "https://dot.net/v1/dotnet-install.ps1";
 $NugetUrl = "https://dist.nuget.org/win-x86-commandline/latest/nuget.exe"
 

build.sh

@@ -10,7 +10,7 @@ SCRIPT_DIR=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )
 TOOLS_DIR=$SCRIPT_DIR/tools
 NUGET_EXE=$TOOLS_DIR/nuget.exe
 NUGET_URL=https://dist.nuget.org/win-x86-commandline/latest/nuget.exe
-CAKE_VERSION=0.23.0
+CAKE_VERSION=0.24.0
 CAKE_EXE=$TOOLS_DIR/Cake.$CAKE_VERSION/Cake.exe
 
 # Temporarily skip verification and opt-in to new in-proc NuGet
@@ -50,7 +50,7 @@ if [ ! -d "$SCRIPT_DIR/.dotnet" ]; then
   mkdir "$SCRIPT_DIR/.dotnet"
 fi
 curl -Lsfo "$SCRIPT_DIR/.dotnet/dotnet-install.sh" https://dot.net/v1/dotnet-install.sh
-sudo bash "$SCRIPT_DIR/.dotnet/dotnet-install.sh" --version 1.1.5 --install-dir .dotnet --no-path
+sudo bash "$SCRIPT_DIR/.dotnet/dotnet-install.sh" --version 1.1.7 --install-dir .dotnet --no-path
 export PATH="$SCRIPT_DIR/.dotnet":$PATH
 export DOTNET_SKIP_FIRST_TIME_EXPERIENCE=1
 export DOTNET_CLI_TELEMETRY_OPTOUT=1

global.json

@@ -1,6 +1,6 @@
 {
     "projects": [ "src" ],
     "sdk": {
-        "version": "1.1.5"
+        "version": "1.1.7"
     }
 }

src/Cake.Common.Tests/Unit/Tools/DotNetCore/Build/DotNetCoreBuilderTests.cs

@@ -147,13 +147,14 @@ public void Should_Add_Build_Arguments()
                 var fixture = new DotNetCoreBuilderFixture();
                 fixture.Settings.NoIncremental = true;
                 fixture.Settings.NoDependencies = true;
+                fixture.Settings.NoRestore = true;
                 fixture.Project = "./src/*";
 
                 // When
                 var result = fixture.Run();
 
                 // Then
-                Assert.Equal("build \"./src/*\" --no-incremental --no-dependencies", result.Args);
+                Assert.Equal("build \"./src/*\" --no-incremental --no-dependencies --no-restore", result.Args);
             }
 
             [Fact]

src/Cake.Common.Tests/Unit/Tools/DotNetCore/Pack/DotNetCorePackerTests.cs

@@ -106,6 +106,8 @@ public void Should_Add_Settings()
                 // Given
                 var fixture = new DotNetCorePackFixture();
                 fixture.Settings.NoBuild = true;
+                fixture.Settings.NoDependencies = true;
+                fixture.Settings.NoRestore = true;
                 fixture.Settings.Configuration = "Release";
                 fixture.Settings.OutputDirectory = "./artifacts/";
                 fixture.Settings.VersionSuffix = "rc1";
@@ -118,7 +120,7 @@ public void Should_Add_Settings()
                 var result = fixture.Run();
 
                 // Then
-                Assert.Equal("pack --output \"/Working/artifacts\" --no-build --include-symbols --include-source --configuration Release --version-suffix rc1 --serviceable --verbosity Minimal", result.Args);
+                Assert.Equal("pack --output \"/Working/artifacts\" --no-build --no-dependencies --no-restore --include-symbols --include-source --configuration Release --version-suffix rc1 --serviceable --verbosity Minimal", result.Args);
             }
 
             [Fact]

src/Cake.Common.Tests/Unit/Tools/DotNetCore/Publish/DotNetCorePublisherTests.cs

@@ -105,6 +105,8 @@ public void Should_Add_Settings()
             {
                 // Given
                 var fixture = new DotNetCorePublisherFixture();
+                fixture.Settings.NoDependencies = true;
+                fixture.Settings.NoRestore = true;
                 fixture.Settings.Framework = "dnxcore50";
                 fixture.Settings.Configuration = "Release";
                 fixture.Settings.Runtime = "runtime1";
@@ -116,7 +118,7 @@ public void Should_Add_Settings()
                 var result = fixture.Run();
 
                 // Then
-                Assert.Equal("publish --output \"/Working/artifacts\" --runtime runtime1 --framework dnxcore50 --configuration Release --version-suffix rc1 --verbosity Minimal", result.Args);
+                Assert.Equal("publish --output \"/Working/artifacts\" --runtime runtime1 --framework dnxcore50 --configuration Release --version-suffix rc1 --no-dependencies --no-restore --verbosity Minimal", result.Args);
             }
 
             [Fact]

src/Cake.Common.Tests/Unit/Tools/DotNetCore/Test/DotNetCoreTesterTests.cs

@@ -108,6 +108,7 @@ public void Should_Add_Additional_Settings()
                 // Given
                 var fixture = new DotNetCoreTesterFixture();
                 fixture.Settings.NoBuild = true;
+                fixture.Settings.NoRestore = true;
                 fixture.Settings.Framework = "dnxcore50";
                 fixture.Settings.Configuration = "Release";
                 fixture.Settings.OutputDirectory = "./artifacts/";
@@ -122,7 +123,7 @@ public void Should_Add_Additional_Settings()
                 var result = fixture.Run();
 
                 // Then
-                Assert.Equal("test --settings \"/Working/demo.runsettings\" --filter \"Priority = 1\" --test-adapter-path \"/Working/custom-test-adapter\" --logger \"trx;LogFileName=/Working/logfile.trx\" --output \"/Working/artifacts\" --framework dnxcore50 --configuration Release --diag \"/Working/artifacts/logging/diagnostics.txt\" --no-build --results-directory \"/Working/tests\"", result.Args);
+                Assert.Equal("test --settings \"/Working/demo.runsettings\" --filter \"Priority = 1\" --test-adapter-path \"/Working/custom-test-adapter\" --logger \"trx;LogFileName=/Working/logfile.trx\" --output \"/Working/artifacts\" --framework dnxcore50 --configuration Release --diag \"/Working/artifacts/logging/diagnostics.txt\" --no-build --no-restore --results-directory \"/Working/tests\"", result.Args);
             }
 
             [Fact]

src/Cake.Common.Tests/Unit/Tools/MSBuild/MSBuildVerbosityExtensionsTests.cs

@@ -0,0 +1,78 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information.
+
+using System.Linq;
+using Cake.Common.Tools.MSBuild;
+using Cake.Core;
+using Cake.Core.Diagnostics;
+using Xunit;
+
+namespace Cake.Common.Tests.Unit.Tools.MSBuild
+{
+    public sealed class MSBuildVerbosityExtensionsTests
+    {
+        public sealed class GetMSBuildVerbosityMethod
+        {
+            [Theory]
+            [InlineData("quiet", Verbosity.Quiet)]
+            [InlineData("minimal", Verbosity.Minimal)]
+            [InlineData("normal", Verbosity.Normal)]
+            [InlineData("detailed", Verbosity.Verbose)]
+            [InlineData("diagnostic", Verbosity.Diagnostic)]
+            public void Should_Convert_Valid_String_To_Verbosity_Enum(string verbosityName, Verbosity expectedVerbosity)
+            {
+                // Given
+
+                // When
+                var actualVerbosity = MSBuildVerbosityExtensions.GetMSBuildVerbosity(verbosityName);
+
+                // Then
+                Assert.Equal(expectedVerbosity, actualVerbosity);
+            }
+
+            [Theory]
+            [InlineData("QUIET", Verbosity.Quiet)]
+            [InlineData("MINIMAL", Verbosity.Minimal)]
+            [InlineData("NORMAL", Verbosity.Normal)]
+            [InlineData("DETAILED", Verbosity.Verbose)]
+            [InlineData("DIAGNOSTIC", Verbosity.Diagnostic)]
+            public void Should_Convert_Valid_UpperCase_String_To_Verbosity_Enum(string verbosityName, Verbosity expectedVerbosity)
+            {
+                // Given
+
+                // When
+                var actualVerbosity = MSBuildVerbosityExtensions.GetMSBuildVerbosity(verbosityName);
+
+                // Then
+                Assert.Equal(expectedVerbosity, actualVerbosity);
+            }
+
+            [Fact]
+            public void Should_Throws_Exception_When_Parameter_Is_Null()
+            {
+                // Given
+
+                // When
+                var actualException = Assert.Throws<CakeException>(() => MSBuildVerbosityExtensions.GetMSBuildVerbosity(null));
+
+                // Then
+                Assert.Equal("Encountered unknown MSBuild build log verbosity ''. Valid values are 'quiet', 'minimal', 'normal', 'detailed' and 'diagnostic'.", actualException.Message);
+            }
+
+            [Theory]
+            [InlineData("", "Encountered unknown MSBuild build log verbosity ''. Valid values are 'quiet', 'minimal', 'normal', 'detailed' and 'diagnostic'.")]
+            [InlineData("invalid", "Encountered unknown MSBuild build log verbosity 'invalid'. Valid values are 'quiet', 'minimal', 'normal', 'detailed' and 'diagnostic'.")]
+            public void Should_Throws_Exception_When_Parameter_Has_Invalid_Value(string invalidVerbosityValue, string expectedMessage)
+            {
+                // Given
+
+                // When
+                var actualException = Assert.Throws<CakeException>(() => MSBuildVerbosityExtensions.GetMSBuildVerbosity(invalidVerbosityValue));
+
+                // Then
+                Assert.Equal(expectedMessage, actualException.Message);
+            }
+        }
+   }
+}