Install dotnet sdk if missing on Unix as well to fix failing Ubuntu build due to image update and Pester/PowerShell bug (PowerShell#1107)

* add unix support for installing the .net core sdk on-demand

* add logging

* update sdk

* adapt for dotnet writing to stderr

* fix parenthesis

* only set protocol if needed and set erroraction to avoid ci failur

* try fix nullreferenceexception

* use use --list-sdks

* use 2.1.403 sdk, which is at least available on Windows

* use erroractionpreference

* redirect error

* use continue

* use 2.1.400 that is available on both windows and linux and revert other changes that would break break wmf4

* remove if check that could break wmf4?

* use --list-sdks

* try different syntax to get wmf4 to work

* disable wmf4 for the moment

* use 2.1.403 and re-enable wmf4

* fix typo (wmf4)

* use .net core sdk 2.1.502

* ignore 5 pester failure on Ubuntu for AppVeyor due Pester/PowerShell bug that happens on AppVeyor environment

* Fix typo

* add more skip statements for failing appveyor tests

* Revert "add more skip statements for failing appveyor tests"

This reverts commit 064683f.

* Revert "Fix typo"

This reverts commit 61c4d59.

* Revert "ignore 5 pester failure on Ubuntu for AppVeyor due Pester/PowerShell bug that happens on AppVeyor environment"

This reverts commit 8c48e93.

* set locale

* empty

* Revert "set locale"

This reverts commit ee79467.

* simplify dotnet check

* output debug details and disable other images

* output rule names

* output more

* add more and downgrade sdk

* use 2.1.403 sdk

* 2.1.401 sdk

* 2.1.400 sdk

* Revert "simplify dotnet check"

This reverts commit 7db40c3.

* 2.1.401

* 2.1.403

* 2.1.502

* fix resource string casing of AvoidUsingUsernameAndPassWordParams to AvoidUsingUsernameAndPasswordParams

* another resource string casing optimisation

* use OrdinalIgnoreCase to be really case insensitive

* cleanup

Author: bergmeister

README.md

@@ -105,7 +105,7 @@ Note: the PSScriptAnalyzer Chocolatey package is provided and supported by the c
 
 #### Requirements
 
-* [.NET Core 2.1.401 SDK](https://www.microsoft.com/net/download/dotnet-core/2.1#sdk-2.1.401) or newer patch release
+* [.NET Core 2.1.502 SDK](https://www.microsoft.com/net/download/dotnet-core/2.1#sdk-2.1.502) or newer patch release
 * [PlatyPS 0.9.0 or greater](https://github.com/PowerShell/platyPS/releases)
 * Optionally but recommended for development: [Visual Studio 2017](https://www.visualstudio.com/downloads/)
 

Rules/Strings.Designer.cs

@@ -484,7 +484,7 @@
     <value>Function '{0}' has both Username and Password parameters. Either set the type of the Password parameter to SecureString or replace the Username and Password parameters with a Credential parameter of type PSCredential. If using a Credential parameter in PowerShell 4.0 or earlier, please define a credential transformation attribute after the PSCredential type attribute.</value>
   </data>
   <data name="AvoidUsernameAndPasswordParamsName" xml:space="preserve">
-    <value>AvoidUsingUserNameAndPassWordParams</value>
+    <value>AvoidUsingUsernameAndPasswordParams</value>
   </data>
   <data name="AvoidInvokingEmptyMembersCommonName" xml:space="preserve">
     <value>Avoid Invoking Empty Members</value>

Rules/Strings.resx

@@ -35,7 +35,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeDSCResource(Ast ast, string fileName
             List<string> expectedTargetResourceFunctionNames = new List<string>(new string[]  { "Get-TargetResource", "Set-TargetResource", "Test-TargetResource" });
 
             // Retrieve a list of Asts where the function name contains TargetResource            
-            IEnumerable<Ast> functionDefinitionAsts = (ast.FindAll(dscAst => dscAst is FunctionDefinitionAst && ((dscAst as FunctionDefinitionAst).Name.IndexOf("targetResource", StringComparison.CurrentCultureIgnoreCase) != -1), true));
+            IEnumerable<Ast> functionDefinitionAsts = (ast.FindAll(dscAst => dscAst is FunctionDefinitionAst && ((dscAst as FunctionDefinitionAst).Name.IndexOf("targetResource", StringComparison.OrdinalIgnoreCase) != -1), true));
 
             List<string> targetResourceFunctionNamesInAst = new List<string>();
             foreach (FunctionDefinitionAst functionDefinitionAst in functionDefinitionAsts)
@@ -46,7 +46,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeDSCResource(Ast ast, string fileName
             foreach (string expectedTargetResourceFunctionName in expectedTargetResourceFunctionNames)
             {
                 // If the Ast does not contain the expected functions, provide a Rule violation message
-                if (!targetResourceFunctionNamesInAst.Contains(expectedTargetResourceFunctionName, StringComparer.CurrentCultureIgnoreCase))
+                if (!targetResourceFunctionNamesInAst.Contains(expectedTargetResourceFunctionName, StringComparer.OrdinalIgnoreCase))
                 {
                     yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.UseStandardDSCFunctionsInResourceError, expectedTargetResourceFunctionName),
                         ast.Extent, GetName(), DiagnosticSeverity.Error, fileName);      

Rules/UseStandardDSCFunctionsInResource.cs

@@ -1,5 +1,5 @@
 {
     "sdk": {
-        "version": "2.1.401"
+        "version": "2.1.502"
     }
 }

global.json

@@ -32,13 +32,33 @@ function Invoke-AppVeyorInstall {
 
     # the legacy WMF4 image only has the old preview SDKs of dotnet
     $globalDotJson = Get-Content (Join-Path $PSScriptRoot '..\global.json') -Raw | ConvertFrom-Json
-    $dotNetCoreSDKVersion = $globalDotJson.sdk.version
-    # don't try to run this script on linux - we have to do the negative check because IsLinux will be defined in core, but not windows
-    if (-not ((dotnet --version).StartsWith($dotNetCoreSDKVersion)) -and ! $IsLinux ) {
-        [Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12 # https://github.com/dotnet/announcements/issues/77
-        Invoke-WebRequest 'https://dot.net/v1/dotnet-install.ps1' -OutFile dotnet-install.ps1
-        .\dotnet-install.ps1 -Version $dotNetCoreSDKVersion
-        Remove-Item .\dotnet-install.ps1
+    $requiredDotNetCoreSDKVersion = $globalDotJson.sdk.version
+    if ($PSVersionTable.PSVersion.Major -gt 4) {
+        $requiredDotNetCoreSDKVersionPresent = (dotnet --list-sdks) -match $requiredDotNetCoreSDKVersion
+    }
+    else {
+        # WMF 4 image has old SDK that does not have --list-sdks parameter
+        $requiredDotNetCoreSDKVersionPresent = (dotnet --version).StartsWith($requiredDotNetCoreSDKVersion)
+    }
+    if (-not $requiredDotNetCoreSDKVersionPresent) {
+        Write-Verbose -Verbose "Installing required .Net CORE SDK $requiredDotNetCoreSDKVersion"
+        $originalSecurityProtocol = [Net.ServicePointManager]::SecurityProtocol
+        try {
+            [Net.ServicePointManager]::SecurityProtocol = [Net.ServicePointManager]::SecurityProtocol -bor [Net.SecurityProtocolType]::Tls12
+            if ($IsLinux -or $isMacOS) {
+                Invoke-WebRequest 'https://dot.net/v1/dotnet-install.sh' -OutFile dotnet-install.sh
+                bash dotnet-install.sh --version $requiredDotNetCoreSDKVersion
+                [System.Environment]::SetEnvironmentVariable('PATH', "/home/appveyor/.dotnet$([System.IO.Path]::PathSeparator)$PATH")
+            }
+            else {
+                Invoke-WebRequest 'https://dot.net/v1/dotnet-install.ps1' -OutFile dotnet-install.ps1
+                .\dotnet-install.ps1 -Version $requiredDotNetCoreSDKVersion
+            }
+        }
+        finally {
+            [Net.ServicePointManager]::SecurityProtocol = $originalSecurityProtocol
+            Remove-Item .\dotnet-install.*
+        }
     }
 }