Test failure logging

Use exception instead of Write-Error to not hide source.
Split test and publish steps out into separate scripts.

Author: rasmusjelsgaard

.travis.yml

@@ -26,12 +26,12 @@ install:
   - "echo \"GitVersion says: ${GitVersion_Version} \""
 
 script:
-  - pwsh -c "Invoke-Psake -buildFile Build.ps1 -taskList build,test"
+  - pwsh -f "./Invoke-Tests.ps1"
 
 deploy:
   skip_cleanup: true
   provider: script
-  script: pwsh -c "Invoke-Psake -buildFile Build.ps1 -taskList build,test,publish"
+  script: pwsh -f "./Invoke-Publish.ps1"
   on:
     tags: true
     branch: master

Build.ps1

@@ -71,7 +71,7 @@ Task Test -depends Build {
 
     if ($TestResult.FailedCount -gt 0) {
         $TestResult | Format-List
-        Write-Error -Message 'One or more tests for the module failed. Failing the build.'
+        throw 'One or more tests for the module failed. Failing the build.'
     }
 }
 

Invoke-Publish.ps1

@@ -0,0 +1,2 @@
+Invoke-Psake -buildFile Build.ps1 -taskList build,test,publish
+exit ( [int]( -not $psake.build_success ) )

Invoke-Tests.ps1

@@ -0,0 +1,2 @@
+Invoke-Psake -buildFile Build.ps1 -taskList build,test
+exit ( [int]( -not $psake.build_success ) )

Tests/Get-CrumbHeader.Tests.ps1

@@ -16,7 +16,7 @@ Describe 'Get-CrumbHeader method' {
 
             $crumbHeader = Get-CrumbHeader -UserName $user -Password $pass
             Assert-MockCalled -CommandName Invoke-RestMethod -Exactly 1
-            $crumbHeader | Should -BeExactly "requestfield=headerdata"
+            $crumbHeader | Should -BeExactly "requestfield=headerdata"            
         }
     }