Skip to content

Commit

Permalink
fix: pester tests
Browse files Browse the repository at this point in the history
  • Loading branch information
kpoineal committed Oct 10, 2024
1 parent 05f3f2d commit 5e52a1a
Show file tree
Hide file tree
Showing 12 changed files with 71 additions and 11 deletions.
22 changes: 14 additions & 8 deletions .github/scripts/RunPesterTests.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@ if ($env:GITHUB_WORKSPACE) {
# Get all module directories under modules/wara
$moduleDirectories = Get-ChildItem -Path "$basePath/modules/wara/" -Directory

$mermaid = "``````mermaid"

foreach ($moduleDir in $moduleDirectories) {
$modulePath = "$($moduleDir.FullName)/$($moduleDir.Name).psm1"
$testsPath = "$basePath/tests/$($moduleDir.Name)"
Expand All @@ -31,6 +29,7 @@ foreach ($moduleDir in $moduleDirectories) {
$passedCount = $($result.PassedCount -eq $result.TotalCount) ? "$($result.PassedCount)" : "$($result.PassedCount)"
$failedCount = $($result.FailedCount -gt 0) ? "$($result.FailedCount)" : "$($result.FailedCount)"
$coveragePercent = $($result.CodeCoverage.CoveragePercent -ge $result.CodeCoverage.CoveragePercentTarget) ? "$([Math]::Round($result.CodeCoverage.CoveragePercent, 2))" : "$([Math]::Round($result.CodeCoverage.CoveragePercent, 2))"

$markdown = @"
# Code Coverage Report - $($moduleDir.Name).psm1
| Metric | Value |
Expand All @@ -40,14 +39,21 @@ foreach ($moduleDir in $moduleDirectories) {
| Failed Count | $failedCount |
| Coverage (%) | $coveragePercent |
| Target Coverage (%) | $($result.CodeCoverage.CoveragePercentTarget) |
"@

<# ---
title: Hello Title
config:
theme: base
themeVariables:
primaryColor: #00ff01
secondaryColor: #ff0000
---
``````mermaid
graph TD;
Passed -->|Passed| PassedCount;
Failed -->|Failed| FailedCount;
Coverage -->|Coverage| CoveragePercent;
Target -->|Target| CoveragePercentTarget;
"@
pie
title Coverage - $($moduleDir.Name).psm1
"Covered" : $([Math]::Round($result.CodeCoverage.CoveragePercent, 2))
"Uncovered" : $(100-[Math]::Round($result.CodeCoverage.CoveragePercent, 2))#>


$markdown | Out-File -FilePath "$testspath/coverage_$($moduleDir.Name).md" -Force
Expand Down
4 changes: 2 additions & 2 deletions coverage.xml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<!DOCTYPE report PUBLIC "-//JACOCO//DTD Report 1.1//EN" "report.dtd"[]>
<report name="Pester (10/09/2024 16:28:04)">
<sessioninfo id="this" start="1728491284877" dump="1728491284879" />
<report name="Pester (10/09/2024 14:04:29)">
<sessioninfo id="this" start="1728482669847" dump="1728482669859" />
<package name="utils">
<class name="utils/utils" sourcefilename="utils.psm1">
<method name="Import-WAFConfigFileData" desc="()" line="3">
Expand Down
2 changes: 1 addition & 1 deletion src/modules/wara/collector/collector.psm1
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ Function Invoke-WAFQueryLoop {
$QueryObject = Get-WAFQueryByResourceType -ObjectList $RecommendationObject -FilterList $Types.type -KeyColumn "recommendationResourceType"

$return = $QueryObject | Where-Object{$_.automationavailable -eq "arg"} | ForEach-Object {
Write-Progress -Activity "Running Queries" -Status "Running Query for $($_.recommendationResourceType)" -PercentComplete (($QueryObject.IndexOf($_) / $QueryObject.Count) * 100)
Write-Progress -Activity "Running Queries" -Status "Running Query for $($_.recommendationResourceType) - $($_.aprlGuid)" -PercentComplete (($QueryObject.IndexOf($_) / $QueryObject.Count) * 100)
Get-WAFAllAzGraphResource -query $_.query -subscriptionIds $subscriptionIds
}

Expand Down
6 changes: 6 additions & 0 deletions src/tests/advisor/coverage_advisor.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ✅ 82.35 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - advisor.psm1
"Covered" : 82.35
"Uncovered" : 17.65
6 changes: 6 additions & 0 deletions src/tests/collector/coverage_collector.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - collector.psm1
"Covered" : 0
"Uncovered" : 100
6 changes: 6 additions & 0 deletions src/tests/outage/coverage_outage.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - outage.psm1
"Covered" : 0
"Uncovered" : 100
6 changes: 6 additions & 0 deletions src/tests/retirement/coverage_retirement.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - retirement.psm1
"Covered" : 0
"Uncovered" : 100
6 changes: 6 additions & 0 deletions src/tests/runbook/coverage_runbook.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - runbook.psm1
"Covered" : 0
"Uncovered" : 100
6 changes: 6 additions & 0 deletions src/tests/scope/coverage_scope.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ✅ 89.19 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - scope.psm1
"Covered" : 89.19
"Uncovered" : 10.81
6 changes: 6 additions & 0 deletions src/tests/servicehealth/coverage_servicehealth.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - servicehealth.psm1
"Covered" : 0
"Uncovered" : 100
6 changes: 6 additions & 0 deletions src/tests/support/coverage_support.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - support.psm1
"Covered" : 0
"Uncovered" : 100
6 changes: 6 additions & 0 deletions src/tests/utils/coverage_utils.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,9 @@
| Failed Count | ✅ 0 |
| Coverage (%) | ❌ 0 |
| Target Coverage (%) | 75 |

```mermaid
pie
title Coverage - utils.psm1
"Covered" : 0
"Uncovered" : 100

0 comments on commit 5e52a1a

Please sign in to comment.