(#88) Add instance name replacement of destination (#89)

* (#88) Add instance name replacement of destination Replace %{instance_name} with the instance name in the destination of the downloads key. Signed-off-by: Cory Knox <[email protected]> * (#88) Add test results to Azure Pipelines Add the test results to the Azure Pipelines runs. Signed-off-by: Cory Knox <[email protected]>
test-kitchen · Apr 3, 2022 · 1c45062 · 1c45062
1 parent 300c7bc
commit 1c45062
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 2 deletions.
diff --git a/.kitchen.appveyor.yml b/.kitchen.appveyor.yml
@@ -19,6 +19,8 @@ verifier:
   environment:
     API_KEY: Some key value
     PUSH_REPO: https://push.example.com
+  downloads:
+    "PesterTestResults.xml": "testresults/%{instance_name}/"
 
 suites:
   - name: default
diff --git a/.kitchen.yml b/.kitchen.yml
@@ -18,6 +18,8 @@ verifier:
   environment:
     API_KEY: Some key value
     PUSH_REPO: https://push.example.com
+  downloads:
+    "PesterTestResults.xml": "testresults/%{instance_name}/"
 
 suites:
   - name: default
diff --git a/README.md b/README.md
@@ -143,7 +143,16 @@ This is necessary in certain cases, such as when `pwsh` is installed via `snap`
 
 * `downloads`- map[string, string], defaults to `{"./PesterTestResults.xml" => "./testresults}"`.
 Files to download from SUT to local system, used to download the pester results localy.
-The key is the remote file (relative to verifier folder or absolute), the value is the directory (ends with / or \\) it should be saved to (relative to pwd or absolute).
+The key is the remote file to download, while the value is the destination.
+  * The source can:
+    * Be relative to the verifier folder (by default `$TEMP/verifier`)
+    * Be absolute on the system (e.g. `/var/tmp/file.zip` or `C:\\Windows\\Temp\\file.zip`)
+  * The destination can:
+    * Include `%{instance_name}` to indicate the Test Kitchen instance name
+    * Be a directory (ends in `/` or `\\`)
+    * Be relative to the Current Working Directory
+    * Be absolute
+
   ```yaml
   downloads:
       PesterTestResults.xml: "./output/testResults/"

diff --git a/TESTING.md b/TESTING.md
@@ -9,3 +9,4 @@ The general steps to test your changes:
 1. Build new gem: `chef gem build ./kitchen-pester.gemspec`
 1. Install built gem: `chef gem install ./kitchen-pester-<version>.gem`
 1. Test with `test-kitchen`: `kitchen test`
+1. Ensure that `PesterTestResults.xml` is created in `./testresults/default-windows-2016`
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
@@ -59,6 +59,10 @@ stages:
             displayName: Run Tests
             env:
               SPEC_OPTS: --format progress
+          - task: PublishTestResults@2
+            inputs:
+              testResultsFormat: NUnit
+              testResultsFiles: 'testresults/**/PesterTestResults.xml'
       - job: Package
         dependsOn: Validate
         condition: and(succeeded(), eq(variables['build.sourceBranch'], 'refs/heads/main'), notIn(variables['Build.Reason'], 'PullRequest'))

diff --git a/lib/kitchen/verifier/pester.rb b/lib/kitchen/verifier/pester.rb
@@ -190,6 +190,7 @@ def resolve_downloads_paths!
         config[:downloads] = config[:downloads]
           .map do |source, destination|
             source = source.to_s
+            destination = destination.gsub("%{instance_name}", instance.name)
             info("  resolving remote source's absolute path.")
             unless source.match?('^/|^[a-zA-Z]:[\\/]') # is Absolute?
               info("  '#{source}' is a relative path, resolving to: #{File.join(config[:root_path], source)}")
@@ -203,7 +204,7 @@ def resolve_downloads_paths!
             if !File.directory?(File.dirname(destination))
               FileUtils.mkdir_p(File.dirname(destination))
             else
-              info("  Directory #{File.dirname(destination)} seem to exist.")
+              info("  Directory #{File.dirname(destination)} seems to exist.")
             end
 
             [ source, destination ]