Implement CodeCoverage.ExcludeTests

pester · Jul 1, 2024 · 48643db · 48643db
1 parent 6cefccf
commit 48643db
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 0 deletions.
diff --git a/src/functions/Coverage.ps1 b/src/functions/Coverage.ps1
@@ -167,6 +167,10 @@ function Get-CoverageInfoFromDictionary {
     $includeTests = Get-DictionaryValueFromFirstKeyFound -Dictionary $Dictionary -Key 'IncludeTests'
     $recursePaths = Get-DictionaryValueFromFirstKeyFound -Dictionary $Dictionary -Key 'RecursePaths'
 
+    # TODO: Implement or remove the IDictionary config logic from CodeCoverage
+    # Jank fix for https://github.com/pester/Pester/issues/2514 until CodeCoverage config logic is updated
+    if ($null -eq $includeTests) { $includeTests = $PesterPreference.CodeCoverage.ExcludeTests.Value -ne $true }
+
     $startLine = Convert-UnknownValueToInt -Value $startLine -DefaultValue 0
     $endLine = Convert-UnknownValueToInt -Value $endLine -DefaultValue 0
     [bool] $includeTests = Convert-UnknownValueToInt -Value $includeTests -DefaultValue 0

diff --git a/tst/Pester.RSpec.Coverage.ts.ps1 b/tst/Pester.RSpec.Coverage.ts.ps1
@@ -289,4 +289,38 @@ i -PassThru:$PassThru {
             $r.Result | Verify-Equal 'Passed'
         }
     }
+
+    b 'Coverage path resolution' {
+        t 'Excludes test files when ExcludeTests is true' {
+            # https://github.com/pester/Pester/issues/2514
+            $c = New-PesterConfiguration
+            $c.Run.Path = "$PSScriptRoot/testProjects/CoverageTestFile.Tests.ps1"
+            $c.Run.PassThru = $true
+            $c.CodeCoverage.Enabled = $true
+            $c.CodeCoverage.ExcludeTests = $true # default
+            $c.CodeCoverage.Path = "$PSScriptRoot/CoverageTestFile.ps1", "$PSScriptRoot/testProjects"
+
+            $r = Invoke-Pester -Configuration $c
+
+            $r.Result | Verify-Equal 'Passed'
+            $r.CodeCoverage.FilesAnalyzedCount | Verify-Equal 1
+            @($r.CodeCoverage.FilesAnalyzed) -match '\.Tests.ps1$' | Verify-Null
+        }
+
+        t 'Includes test files when ExcludeTests is false' {
+            # https://github.com/pester/Pester/issues/2514
+            $c = New-PesterConfiguration
+            $c.Run.Path = "$PSScriptRoot/testProjects/CoverageTestFile.Tests.ps1"
+            $c.Run.PassThru = $true
+            $c.CodeCoverage.Enabled = $true
+            $c.CodeCoverage.ExcludeTests = $false
+            $c.CodeCoverage.Path = "$PSScriptRoot/CoverageTestFile.ps1", "$PSScriptRoot/testProjects"
+
+            $r = Invoke-Pester -Configuration $c
+
+            $r.Result | Verify-Equal 'Passed'
+            $r.CodeCoverage.FilesAnalyzedCount | Verify-Equal 4
+            @($r.CodeCoverage.FilesAnalyzed) -match '\.Tests.ps1$' | Verify-NotNull
+        }
+    }
 }