Handle tests marked as inconclusive

src/Main.ps1

@@ -747,12 +747,12 @@ function Invoke-Pester {
             }
 
             $plugins.Add((
-                # decorator plugin needs to be added after output
-                # because on teardown they will run in opposite order
-                # and that way output can consume the fixed object that decorator
-                # decorated, not nice but works
-                Get-RSpecObjectDecoratorPlugin
-            ))
+                    # decorator plugin needs to be added after output
+                    # because on teardown they will run in opposite order
+                    # and that way output can consume the fixed object that decorator
+                    # decorated, not nice but works
+                    Get-RSpecObjectDecoratorPlugin
+                ))
 
             if ($PesterPreference.TestDrive.Enabled.Value) {
                 $plugins.Add((Get-TestDrivePlugin))
@@ -1458,11 +1458,13 @@ function ConvertTo-Pester4Result {
                 "Skipped" {
                     $legacyResult.SkippedCount++
                 }
+                "Inconclusive" {
+                    $legacyResult.InconclusiveCount++
+                }
             }
         }
         $legacyResult.TotalCount = $legacyResult.TestResult.Count
         $legacyResult.PendingCount = 0
-        $legacyResult.InconclusiveCount = 0
         $legacyResult.Time = $PesterResult.Duration
 
         $legacyResult

src/Pester.Runtime.ps1

@@ -2211,6 +2211,9 @@ function PostProcess-ExecutedBlock {
                 if (-not $t.ShouldRun) {
                     $b.OwnNotRunCount++
                 }
+                elseif ($t.ShouldRun -and $t.Inconclusive) {
+                    $b.OwnInconclusiveCount++
+                }
                 elseif ($t.ShouldRun -and $t.Skipped) {
                     $b.OwnSkippedCount++
                 }
@@ -2273,7 +2276,7 @@ function PostProcess-ExecutedBlock {
                     $b.PassedCount += $child.PassedCount
                     $b.FailedCount += $child.FailedCount
                     $b.SkippedCount += $child.SkippedCount
-                    $b.InconclusiveCount += $child.OwnInconclusiveCount
+                    $b.InconclusiveCount += $child.InconclusiveCount
                     $b.NotRunCount += $child.NotRunCount
                 }
 

src/functions/Output.ps1

@@ -310,10 +310,6 @@ function Write-PesterReport {
     )
     # if(-not ($PesterState.Show | Has-Flag Summary)) { return }
 
-    if ($RunResult.Tests.ErrorRecord.FullyQualifiedErrorID -contains 'PesterTestPending') {
-        Write-PesterHostMessage '**DEPRECATED**: The Set-ItResult -Pending parameter is deprecated. It will be removed in a future version of Pester.' -ForegroundColor $ReportTheme.Warning
-    }
-
     Write-PesterHostMessage ($ReportStrings.Timing -f (Get-HumanTime ($RunResult.Duration))) -Foreground $ReportTheme.Foreground
 
     $Success, $Failure = if ($RunResult.FailedCount -gt 0) {
@@ -408,6 +404,11 @@ function Write-PesterReport {
     # & $SafeCommands['Write-Host'] ($ReportStrings.TestsPending -f $RunResult.PendingCount) -Foreground $Pending -NoNewLine
     # & $SafeCommands['Write-Host'] ($ReportStrings.TestsInconclusive -f $RunResult.InconclusiveCount) -Foreground $Inconclusive
     # }
+
+    if ($RunResult.Tests.ErrorRecord.FullyQualifiedErrorID -contains 'PesterTestPending') {
+        Write-PesterHostMessage ''
+        Write-PesterHostMessage '**DEPRECATED**: The Set-ItResult -Pending parameter is deprecated. It will be removed in a future version of Pester.' -ForegroundColor $ReportTheme.Warning
+    }
 }
 
 function Write-CoverageReport {

src/functions/Set-ItResult.ps1

@@ -29,6 +29,9 @@
     .EXAMPLE
     ```powershell
     Describe "Example" {
+        It "Inconclusive test" {
+            Set-ItResult -Inconclusive -Because "we want it to be inconclusice"
+        }
         It "Skipped test" {
             Set-ItResult -Skipped -Because "we want it to be skipped"
         }
@@ -38,9 +41,11 @@
     the output should be
 
     ```
-    [!] Skipped test is skipped, because we want it to be skipped
-    Tests completed in 0ms
-    Tests Passed: 0, Failed: 0, Skipped: 0, Pending: 0, Inconclusive 1
+    Describing Example
+      [?] Inconclusive test is inconclusive, because INCONCLUSIVE: we want it to be inconclusice 35ms (32ms|3ms)
+      [!] Skipped test is skipped, because we want it to be skipped 3ms (2ms|1ms)
+    Tests completed in 78ms
+    Tests Passed: 0, Failed: 0, Skipped: 1, Inconclusive: 1, NotRun: 0
     ```
 
     .LINK

tst/Pester.RSpec.ResultObject.ts.ps1

@@ -100,6 +100,14 @@ i -PassThru:$PassThru {
                         It "not run" -Tag "Slow" {
                             1 | Should -Be 1
                         }
+
+                        It "Set-ItResult -Inconclusive" {
+                            Set-ItResult -Inconclusive
+                        }
+
+                        It "Set-ItResult -Skipped" {
+                            Set-ItResult -Skipped
+                        }
                     }
                 }
                 $result = Invoke-Pester -Configuration @{
@@ -119,7 +127,7 @@ i -PassThru:$PassThru {
             $result | Verify-Property "Containers"
             $result.Containers.Count | Verify-Equal 2
 
-            $result.TotalCount | Verify-Equal 4
+            $result.TotalCount | Verify-Equal 6
             $result.Tests | Verify-NotNull
 
             $result.PassedCount | Verify-Equal 1
@@ -128,12 +136,15 @@ i -PassThru:$PassThru {
             $result.FailedCount | Verify-Equal 1
             $result.Failed | Verify-NotNull
 
-            $result.SkippedCount | Verify-Equal 1
+            $result.SkippedCount | Verify-Equal 2
             $result.Skipped | Verify-NotNull
 
             $result.NotRunCount | Verify-Equal 1
             $result.NotRun | Verify-NotNull
 
+            $result.InconclusiveCount | Verify-Equal 1
+            $result.Inconclusive | Verify-NotNull
+
             $result.Duration | Verify-Equal ($result.Containers[0].Duration + $result.Containers[1].Duration)
             $result.UserDuration | Verify-Equal ($result.Containers[0].UserDuration + $result.Containers[1].UserDuration)
             $result.FrameworkDuration | Verify-Equal ($result.Containers[0].FrameworkDuration + $result.Containers[1].FrameworkDuration)

tst/Pester.RSpec.TestResults.NUnit25.ts.ps1

@@ -144,6 +144,51 @@ i -PassThru:$PassThru {
 
         }
 
+        t "should write a skipped test result" {
+            $sb = {
+                Describe "Mocked Describe 1" {
+                    It "Skipped testcase" -Skip {
+                    }
+                }
+                Describe "Mocked Describe 2" {
+                    It "Skipped testcase" {
+                        Set-ItResult -Skipped
+                    }
+                }
+            }
+            $r = Invoke-Pester -Configuration ([PesterConfiguration]@{ Run = @{ ScriptBlock = $sb; PassThru = $true }; Output = @{ Verbosity = 'None' } })
+
+            $xmlResult = $r | ConvertTo-NUnitReport
+            $xmlTestSuite = $xmlResult.'test-results'.'test-suite'.'results'.'test-suite'.'results'.'test-suite'
+            $xmlTestCase1 = $xmlTestSuite.results.'test-case'[0]
+            $xmlTestCase2 = $xmlTestSuite.results.'test-case'[1]
+
+            $xmlTestCase1.name | Verify-Equal "Mocked Describe 1.Skipped testcase"
+            $xmlTestCase1.result | Verify-Equal "Ignored"
+            $xmlTestCase1.time | Verify-XmlTime $r.Containers[0].Blocks[0].Tests[0].Duration
+
+            $xmlTestCase2.name | Verify-Equal "Mocked Describe 2.Skipped testcase"
+            $xmlTestCase2.result | Verify-Equal "Ignored"
+            $xmlTestCase2.time | Verify-XmlTime $r.Containers[0].Blocks[1].Tests[0].Duration
+        }
+
+        t "should write an inconclusive test result" {
+            $sb = {
+                Describe "Mocked Describe" {
+                    It "Inconclusive testcase" {
+                        Set-ItResult -Inconclusive
+                    }
+                }
+            }
+            $r = Invoke-Pester -Configuration ([PesterConfiguration]@{ Run = @{ ScriptBlock = $sb; PassThru = $true }; Output = @{ Verbosity = 'None' } })
+
+            $xmlResult = $r | ConvertTo-NUnitReport
+            $xmlTestCase = $xmlResult.'test-results'.'test-suite'.'results'.'test-suite'.'results'.'test-suite'.'results'.'test-case'
+            $xmlTestCase.name | Verify-Equal "Mocked Describe.Inconclusive testcase"
+            $xmlTestCase.result | Verify-Equal "Inconclusive"
+            $xmlTestCase.time | Verify-XmlTime $r.Containers[0].Blocks[0].Tests[0].Duration
+        }
+
         t "should write the test summary" {
             $sb = {
                 Describe "Mocked Describe" {
@@ -162,6 +207,59 @@ i -PassThru:$PassThru {
             $xmlTestResult.time | Verify-Equal (Get-Date -Format "HH:mm:ss" $r.ExecutedAt)
         }
 
+        t "should write inconclusive count" {
+            $sb = {
+                Describe "Mocked Describe" {
+                    It "Inconclusive testcase" {
+                        Set-ItResult -Inconclusive
+                    }
+                }
+            }
+            $r = invoke-pester -container ( new-pestercontainer -ScriptBlock $sb) -passThru -Output Detailed
+            $xmlResult = $r | ConvertTo-NUnitReport
+            $xmlResult.'test-results'.inconclusive | Verify-Equal 1
+
+            $sb = {
+                Describe "Mocked Describe" {
+                    It "Inconclusive testcase 1" {
+                        Set-ItResult -Inconclusive
+                    }
+                    It "Inconclusive testcase 2" {
+                        Set-ItResult -Inconclusive
+                    }
+                }
+            }
+            $r = invoke-pester -container ( new-pestercontainer -ScriptBlock $sb) -passThru -Output Detailed
+            $xmlResult = $r | ConvertTo-NUnitReport
+            $xmlResult.'test-results'.inconclusive | Verify-Equal 2
+        }
+
+        t "should write skipped count" {
+            $sb = {
+                Describe "Mocked Describe" {
+                    It "Skipped testcase" {
+                        Set-ItResult -Skipped
+                    }
+                }
+            }
+            $r = invoke-pester -container ( new-pestercontainer -ScriptBlock $sb) -passThru -Output Detailed
+            $xmlResult = $r | ConvertTo-NUnitReport
+            $xmlResult.'test-results'.skipped | Verify-Equal 1
+
+            $sb = {
+                Describe "Mocked Describe" {
+                    It "Skipped testcase 1" -Skip {
+                    }
+                    It "Skippde testcase 2" {
+                        Set-ItResult -Skipped
+                    }
+                }
+            }
+            $r = invoke-pester -container ( new-pestercontainer -ScriptBlock $sb) -passThru -Output Detailed
+            $xmlResult = $r | ConvertTo-NUnitReport
+            $xmlResult.'test-results'.skipped | Verify-Equal 2
+        }
+
         t "should write the test-suite information" {
             $sb = {
                 Describe "Mocked Describe" {