Skip to content

Commit d6bbe10

Browse files
committed
Fix line number tests
1 parent da73bc4 commit d6bbe10

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

Tests/Public/Convert-CodeCoverage.Tests.ps1

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,15 @@
1-
Describe "Convert-CodeCoverage"
1+
Describe "Convert-CodeCoverage" {
2+
3+
$ModulePath = Join-Path (Get-Module ModuleBuilder).ModuleBase ModuleBuilder.psm1
4+
$ModuleRoot = Resolve-Path "$PSScriptRoot\..\..\Source"
5+
6+
$ModuleFiles = Get-ChildItem $ModuleRoot -File -Recurse -Filter *.ps1
7+
$ModuleSource = Get-Content $ModulePath
8+
9+
$lineNumber = Get-Random -min 2 -max $ModuleSource.Count
10+
while($ModuleSource[$lineNumber] -match "^#(END)?REGION") {
11+
$lineNumber += 5
12+
}
213

314
It 'Should extract code coverage from Pester objects and add Source conversions' {
415

@@ -16,10 +27,9 @@ Describe "Convert-CodeCoverage"
1627
}
1728
}
1829

19-
$SourceLocation = $PesterResults | Convert-CodeCoverage
30+
$SourceLocation = $PesterResults | Convert-CodeCoverage -SourceRoot $ModuleRoot
2031

2132
$SourceLocation.SourceFile | Should -Be ".\Private\CopyReadme.ps1"
22-
$SourceLocation.SourceLineNumber | Should -Be 24
23-
$SourceLocation.Function | Should -Be 'CopyReadme'
33+
$SourceLocation.Line | Should -Be 24
2434
}
2535
}

Tests/Public/Convert-LineNumber.Tests.ps1

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,11 @@ Describe "Convert-LineNumber" {
1010
for($i=0; $i -lt 5; $i++) {
1111

1212
$lineNumber = Get-Random -min 2 -max $ModuleSource.Count
13-
while($ModuleSource[$lineNumber] -match "^#(END)?REGION") {
13+
while($ModuleSource[$lineNumber] -match "^\s*$|^#(END)?REGION") {
1414
$lineNumber += 5
1515
}
1616

17-
It 'Should map line number $lineNumber in the Module to the matching line the Source' {
17+
It "Should map line number $lineNumber in the Module to the matching line the Source" {
1818
$SourceLocation = Convert-LineNumber $ModulePath $lineNumber
1919

2020
$line = (Get-Content (Join-Path $ModuleRoot $SourceLocation.SourceFile))[$SourceLocation.SourceLineNumber]
@@ -30,7 +30,6 @@ Describe "Convert-LineNumber" {
3030
$line = Select-String -Path $ModulePath 'function ParseLineNumber {' | % LineNumber
3131

3232
$SourceLocation = "At ${ModulePath}:$line char:17" | Convert-LineNumber
33-
Write-Verbose "At ${ModulePath}:$line char:17" -Verbose
3433
$SourceLocation.SourceFile | Should -Be ".\Private\ParseLineNumber.ps1"
3534
$SourceLocation.SourceLineNumber | Should -Be 1
3635
}
@@ -56,7 +55,6 @@ Describe "Convert-LineNumber" {
5655
}
5756

5857
$SourceLocation = $PesterMiss | Convert-LineNumber -Passthru
59-
6058
$SourceLocation.SourceFile | Should -Be ".\Private\CopyReadme.ps1"
6159
$SourceLocation.SourceLineNumber | Should -Be 24
6260
$SourceLocation.Function | Should -Be 'CopyReadme'

0 commit comments

Comments
 (0)