Skip to content

Commit c42cfa6

Browse files
authored
Fixed test coverage null index issue for new modules (#21873)
1 parent e3f69e8 commit c42cfa6

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

tools/TestFx/Coverage/ValidateTestCoverage.ps1

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,14 @@ $rptData | Where-Object Module -in $testedModules | ForEach-Object {
5050
elseif ($cmdCvgD -lt 0.8) {
5151
$blCvgRow = $blData | Where-Object Module -eq $module
5252
$blCvg = $blCvgRow.CommandCoverage
53-
$blCvgD = [decimal]$blCvg.TrimEnd("%") / 100
54-
Write-Host "Last release test coverage for module $module is $blCvg."
55-
if ($cmdCvgD -lt $blCvgD) {
56-
Write-Warning "Test coverage for module $module is less than 80% and lower than the last release !"
57-
$cvgMessageBody80 = "|⚠️|Test Coverage Less Than 80%|$cmdCvg|$blCvg|Test coverage cannot be lower than the number of the last release.|`n"
58-
Set-ModuleTestStatusInPipelineResult -ModuleName "Az.$module" -Status Warning -Content ($cvgMessageHeader80 + $cvgMessageBody80)
53+
if ($null -ne $blCvg) { # null check is necessary for new module
54+
$blCvgD = [decimal]$blCvg.TrimEnd("%") / 100
55+
Write-Host "Last release test coverage for module $module is $blCvg."
56+
if ($cmdCvgD -lt $blCvgD) {
57+
Write-Warning "Test coverage for module $module is less than 80% and lower than the last release !"
58+
$cvgMessageBody80 = "|⚠️|Test Coverage Less Than 80%|$cmdCvg|$blCvg|Test coverage cannot be lower than the number of the last release.|`n"
59+
Set-ModuleTestStatusInPipelineResult -ModuleName "Az.$module" -Status Warning -Content ($cvgMessageHeader80 + $cvgMessageBody80)
60+
}
5961
}
6062
}
6163

0 commit comments

Comments
 (0)