Fix tests

Author: SteveL-MSFT

dsc/tests/dsc_extension_discover.tests.ps1

@@ -4,9 +4,8 @@
 Describe 'Discover extension tests' {
     BeforeAll {
         $oldPath = $env:PATH
-        $separator = [System.IO.Path]::PathSeparator
         $toolPath = Resolve-Path -Path "$PSScriptRoot/../../extensions/test/discover"
-        $env:PATH = "$toolPath$separator$oldPath"
+        $env:PATH = $toolPath + [System.IO.Path]::PathSeparator + $oldPath
     }
 
     AfterAll {
@@ -75,7 +74,7 @@ Describe 'Discover extension tests' {
         Set-Content -Path "$TestDrive/test.dsc.extension.json" -Value $extension_json
         Copy-Item -Path "$toolPath/discover.ps1" -Destination $TestDrive | Out-Null
         Copy-Item -Path "$toolPath/resources" -Destination $TestDrive -Recurse | Out-Null
-        $env:DSC_RESOURCE_PATH = $TestDrive
+        $env:DSC_RESOURCE_PATH = "$TestDrive$separator$oldPath"
         try {
             $out = dsc extension list | ConvertFrom-Json
             $out.Count | Should -Be 1

dsc/tests/dsc_resource_input.tests.ps1

@@ -85,7 +85,7 @@ Describe 'tests for resource input' {
     }
 '@
         $oldPath = $env:DSC_RESOURCE_PATH
-        $env:DSC_RESOURCE_PATH = $TestDrive
+        $env:DSC_RESOURCE_PATH = $TestDrive + [System.IO.Path]::PathSeparator + $env:PATH
         Set-Content $TestDrive/EnvVarInput.dsc.resource.json -Value $manifest
     }
 

dsc/tests/dsc_set.tests.ps1

@@ -147,7 +147,7 @@ changedProperties:
 
         $oldPath = $env:DSC_RESOURCE_PATH
         try {
-            $env:DSC_RESOURCE_PATH = $TestDrive
+            $env:DSC_RESOURCE_PATH = $TestDrive + [System.IO.Path]::PathSeparator + $env:PATH
             $out = '{ "test": true }' | dsc resource set -r Test/SetNoTest -f - --output-format $format | Out-String
             $LASTEXITCODE | Should -Be 0
             $out.Trim() | Should -BeExactly $expected
@@ -160,7 +160,7 @@ changedProperties:
     It 'set can be used on a resource that does not implement test' {
         $oldPath = $env:DSC_RESOURCE_PATH
         try {
-            $env:DSC_RESOURCE_PATH = $TestDrive
+            $env:DSC_RESOURCE_PATH = $TestDrive + [System.IO.Path]::PathSeparator + $env:PATH
             $out = '{ "test": true }' | dsc resource set -r Test/SetNoTest -f - | ConvertFrom-Json
             $LASTEXITCODE | Should -Be 0
             $out.BeforeState.test | Should -Be $true

dsc_lib/locales/en-us.toml

@@ -72,6 +72,7 @@ couldNotReadSetting = "Could not read 'resourcePath' setting"
 appendingEnvPath = "Appending PATH to resourcePath"
 originalPath = "Original PATH: %{path}"
 failedGetEnvPath = "Failed to get PATH environment variable"
+failedJoinEnvPath = "Failed to join PATH environment variable"
 exeHomeAlreadyInPath = "Exe home is already in path: %{path}"
 addExeHomeToPath = "Adding exe home to path: %{path}"
 usingResourcePath = "Using Resource Path: %{path}"

dsc_lib/src/discovery/command_discovery.rs

@@ -155,7 +155,7 @@ impl CommandDiscovery {
             if let Ok(new_path) = env::join_paths(paths.clone()) {
                 env::set_var("PATH", new_path);
             } else {
-                return Err(DscError::Operation(t!("discovery.commandDiscovery.failedSetEnvPath").to_string()));
+                return Err(DscError::Operation(t!("discovery.commandDiscovery.failedJoinEnvPath").to_string()));
             }
         } else {
             // if exe home is not already in PATH env var then add it to env var and list of searched paths

runcommandonset/tests/runcommandonset.get.tests.ps1

@@ -2,15 +2,6 @@
 # Licensed under the MIT License.
 
 Describe 'tests for runcommandonset get' {
-    BeforeAll {
-        $oldPath = $env:DSC_RESOURCE_PATH
-        $env:DSC_RESOURCE_PATH = Join-Path $PSScriptRoot ".."
-    }
-
-    AfterAll {
-        $env:DSC_RESOURCE_PATH = $oldPath
-    }
-
     It 'Input passed for executable, arguments, and exit code' {
         $json = @"
         {
@@ -21,6 +12,7 @@ Describe 'tests for runcommandonset get' {
 "@
 
         $result = $json | dsc resource get -r Microsoft.DSC.Transitional/RunCommandOnSet -f - | ConvertFrom-Json
+        $LASTEXITCODE | Should -Be 0
         $result.actualState.arguments | Should -BeExactly @('bar', 'baz')
         $result.actualState.executable | Should -BeExactly 'foo'
         $result.actualState.exitCode | Should -BeExactly 5

runcommandonset/tests/runcommandonset.set.tests.ps1

@@ -2,21 +2,12 @@
 # Licensed under the MIT License.
 
 Describe 'tests for runcommandonset set' {
-    BeforeAll {
-        $oldPath = $env:DSC_RESOURCE_PATH
-        $env:DSC_RESOURCE_PATH = Join-Path $PSScriptRoot ".."
-    }
-
     AfterEach {
         if (Test-Path $TestDrive/output.txt) {
             Remove-Item -Path $TestDrive/output.txt
         }
     }
 
-    AfterAll {
-        $env:DSC_RESOURCE_PATH = $oldPath
-    }
-
     It 'Input for executable and arguments can be sent to the resource' {
         $input_json = @"
         {