Skip to content

Commit a44ff1d

Browse files
Merge pull request #30 from ChrisLGardner/fix-v10
Add passthru to ensure pester results are output - fixes #28
2 parents 52a0096 + bb3af0b commit a44ff1d

File tree

4 files changed

+36
-3
lines changed

4 files changed

+36
-3
lines changed

Extension/PesterTask/PesterV10/Pester.ps1

+3-3
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ $PesterConfig = @{
9191

9292
Run = @{
9393
Path = $TestFolder
94-
Exit = $true
94+
PassThru = $true
9595
}
9696
TestResult = @{
9797
Enabled = $true
@@ -142,6 +142,6 @@ if (-not([String]::IsNullOrWhiteSpace($ScriptBlock))) {
142142

143143
$result = Invoke-Pester -Configuration ([PesterConfiguration]$PesterConfig)
144144

145-
if ($result.failedCount -ne 0) {
146-
Write-Error "Pester returned errors"
145+
if ($Result.FailedCount -gt 0) {
146+
Write-Error "Pester Failed at least one test. Please see results for details."
147147
}

pester-v10.yml

+2
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,15 @@ steps:
1717

1818
- task: PublishTestResults@2
1919
displayName: 'Publish Test Results Test-Pester.XML'
20+
condition: 'always()'
2021
inputs:
2122
testRunTitle: '$(agent.os)-v10'
2223
testResultsFormat: NUnit
2324
testResultsFiles: '$(System.DefaultWorkingDirectory)/Test-Pester.XML'
2425

2526
- task: PublishCodeCoverageResults@1
2627
displayName: 'Publish code coverage'
28+
condition: 'always()'
2729
inputs:
2830
summaryFileLocation: '$(System.DefaultWorkingDirectory)/CC-Pester.XML'
2931

pester-v9.yml

+2
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,15 @@ steps:
1717

1818
- task: PublishTestResults@2
1919
displayName: 'Publish Test Results Test-Pester.XML'
20+
condition: 'always()'
2021
inputs:
2122
testRunTitle: '$(agent.os)-v9'
2223
testResultsFormat: NUnit
2324
testResultsFiles: '$(System.DefaultWorkingDirectory)/Test-Pester.XML'
2425

2526
- task: PublishCodeCoverageResults@1
2627
displayName: 'Publish code coverage'
28+
condition: 'always()'
2729
inputs:
2830
summaryFileLocation: '$(System.DefaultWorkingDirectory)/CC-Pester.XML'
2931

testResults.xml

+29
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
<?xml version="1.0" encoding="utf-8" standalone="no"?>
2+
<test-results xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="nunit_schema_2.5.xsd" name="Pester" total="6" errors="0" failures="1" not-run="0" inconclusive="0" ignored="0" skipped="0" invalid="0" date="2020-06-07" time="17:19:09">
3+
<environment platform="Microsoft Windows 10 Enterprise|C:\WINDOWS|\Device\Harddisk0\Partition4" os-version="10.0.19041" nunit-version="2.5.8.0" machine-name="DESKTOP-T1JUALL" cwd="C:\source\github\PesterAzureDevOpsExtension" user-domain="DESKTOP-T1JUALL" clr-version="Unknown" user="Chris" />
4+
<culture-info current-culture="en-GB" current-uiculture="en-GB" />
5+
<test-suite type="TestFixture" name="Pester" executed="True" result="Failure" success="False" time="0.1278" asserts="0" description="Pester">
6+
<results>
7+
<test-suite type="TestFixture" name="C:\source\github\PesterAzureDevOpsExtension\Extension\tests\Calculator\Add-Numbers.Tests.ps1" executed="True" result="Failure" success="False" time="0.1278" asserts="0" description="C:\source\github\PesterAzureDevOpsExtension\Extension\tests\Calculator\Add-Numbers.Tests.ps1">
8+
<results>
9+
<test-suite type="TestFixture" name="Add-Numbers" executed="True" result="Failure" success="False" time="0.1126" asserts="0" description="Add-Numbers">
10+
<results>
11+
<test-case description="adds positive numbers" name="Add-Numbers.adds positive numbers" time="0.0027" asserts="0" success="True" result="Success" executed="True" />
12+
<test-case description="adds negative numbers" name="Add-Numbers.adds negative numbers" time="0.0057" asserts="0" success="True" result="Success" executed="True" />
13+
<test-case description="adds one negative number to positive number" name="Add-Numbers.adds one negative number to positive number" time="0.0029" asserts="0" success="True" result="Success" executed="True" />
14+
<test-case description="concatenates strings if given strings" name="Add-Numbers.concatenates strings if given strings" time="0.0085" asserts="0" success="True" result="Success" executed="True" />
15+
<test-case description="should not be 0" name="Add-Numbers.should not be 0" time="0.003" asserts="0" success="True" result="Success" executed="True" />
16+
<test-case description="Should fail" name="Add-Numbers.Should fail" time="0.0235" asserts="0" success="False" result="Failure" executed="True">
17+
<failure>
18+
<message>Expected $false, but got $true.
19+
at $true | Should -Be $false, C:\source\github\PesterAzureDevOpsExtension\Extension\tests\Calculator\Add-Numbers.Tests.ps1:28</message>
20+
<stack-trace>at &lt;ScriptBlock&gt;, C:\source\github\PesterAzureDevOpsExtension\Extension\tests\Calculator\Add-Numbers.Tests.ps1:28</stack-trace>
21+
</failure>
22+
</test-case>
23+
</results>
24+
</test-suite>
25+
</results>
26+
</test-suite>
27+
</results>
28+
</test-suite>
29+
</test-results>

0 commit comments

Comments
 (0)