Merge pull request #423 from chef/ashique/test-hab-pipelines

Fixing the buildkite pipelines for hab

.expeditor/buildkite/artifact.habitat.test.ps1

@@ -76,15 +76,8 @@ Write-Host "+++ Testing $Plan"
 Push-Location $project_root
 
 try {
-    Write-Host "Running unit tests..."
-    hab pkg exec "${pkg_ident}" rake unit
-
-    If ($lastexitcode -ne 0) {
-        Write-Host "Rake unit tests failed!" -ForegroundColor Red
-        Exit $lastexitcode
-    } else {
-        Write-Host "Rake unit tests passed!" -ForegroundColor Green
-    }
+    $scriptPath = Join-Path $project_root "habitat/tests/test.ps1"
+    & $scriptPath $pkg_ident
 }
 finally {
     # Ensure we always return to the original directory

.expeditor/buildkite/artifact.habitat.test.sh

@@ -68,4 +68,4 @@ export PATH
 echo "PATH is $PATH"
 
 echo "--- :mag_right: Testing $PLAN"
-${project_root}/habitat/tests/test.sh "$pkg_ident" || error 'failures during test of executables'
+"${project_root}/habitat/tests/test.sh" "$pkg_ident"

.expeditor/config.yml

@@ -3,18 +3,12 @@
 
 # Slack channel in Chef Software slack to send notifications about build failures, etc
 slack:
-  notify_channel: chef-infra-notify
+  notify_channel: chef-ws-notify
 
 # This publish is triggered by the `built_in:publish_rubygems` artifact_action.
 rubygems:
   - chef-vault
 
-release_branches:
-  - workstation-LTS:
-      version_constraint: 4.*
-  - main:
-      version_constraint: 4.*
-
 github:
   # This deletes the GitHub PR branch after successfully merged into the release branch
   delete_branch_on_merge: true
@@ -27,6 +21,12 @@ github:
   major_bump_labels:
     - "Expeditor: Bump Version Major"
 
+release_branches:
+  - workstation-LTS:
+      version_constraint: 4.*
+  - main:
+      version_constraint: 4.*
+
 changelog:
   rollup_header: Changes not yet released to rubygems.org
 
@@ -46,6 +46,11 @@ subscriptions:
             - "Expeditor: Skip All"
       - built_in:build_gem:
           only_if: built_in:bump_version
+      - trigger_pipeline:habitat/test:
+          only_if: built_in:bump_version
+          ignore_labels:
+            - "Expeditor: Skip Habitat"
+            - "Expeditor: Skip All"
       - trigger_pipeline:habitat/build:
           only_if: built_in:bump_version
           ignore_labels:

.expeditor/habitat-test.pipeline.yml

@@ -9,7 +9,7 @@ expeditor:
 
 steps:
 
-  - label: ":linux: Validate Habitat Builds of Test Kitchen"
+  - label: ":linux: Validate Habitat Builds of Chef-vault"
     commands:
       - .expeditor/buildkite/artifact.habitat.test.sh
     expeditor:
@@ -18,7 +18,7 @@ steps:
           image: ruby:3.1
           privileged: true
 
-  - label: ":windows: Validate Habitat Builds of Test Kitchen"
+  - label: ":windows: Validate Habitat Builds of Chef-vault"
     commands:
       - .expeditor/buildkite/artifact.habitat.test.ps1
     expeditor:

.expeditor/run_windows_tests.ps1

@@ -18,4 +18,4 @@ If ($lastexitcode -ne 0) { Exit $lastexitcode }
 Write-Output "--- Bundle Execute"
 
 bundle exec rake 
-If ($lastexitcode -ne 0) { Exit $lastexitcode }
+If ($lastexitcode -ne 0) { Exit $lastexitcode }

habitat/tests/test.ps1

@@ -0,0 +1,22 @@
+param (
+    [Parameter()]
+    [string]$PackageIdentifier = $(throw "Usage: test.ps1 [test_pkg_ident] e.g. test.ps1 ci/user-windows/1.0.0/20190812103929")
+)
+
+
+Write-Host "--- :fire: Smokish test"
+# Pester the Package
+$help_message=hab pkg exec "${pkg_ident}" -- chef-vault -h
+$original_message="Usage: chef-vault"
+
+Write-Host "Checking the help message of the package"
+Write-Host "Expected: $original_message"
+Write-Host "Actual: $help_message"
+if ($help_message -like "*$original_message*")
+{
+    Write "Chef-vault is working fine"
+}
+else {
+    Write-Error "chef-vault binary doesn't return the correct usage message "
+    throw "Chef-vault windows pipeline not working for hab pkg"
+}

habitat/tests/test.sh

@@ -3,6 +3,7 @@
 set -euo pipefail
 
 project_root="$(git rev-parse --show-toplevel)"
+pkg_ident="$1"
 
 # print error message followed by usage and exit
 error () {