Add number of failed tests, number of passed tests and test pass rate

MathieuBuisson · May 6, 2017 · c5a8471 · c5a8471
1 parent 4f6e19f
commit c5a8471
Show file tree

Hide file tree

Showing 4 changed files with 77 additions and 0 deletions.
diff --git a/PSCodeHealth/Private/New-PSCodeHealthReport.ps1 b/PSCodeHealth/Private/New-PSCodeHealthReport.ps1
@@ -97,6 +97,9 @@ Function New-PSCodeHealthReport {
         'ScriptAnalyzerInformation'     = ($ScriptAnalyzerInformation | Measure-Object).Count
         'ScriptAnalyzerFindingsAverage' = [math]::Round(($FunctionHealthRecord.ScriptAnalyzerFindings | Measure-Object -Average).Average, 2)
         'NumberOfTests'                 = If ( $TestResult ) { $TestResult.TotalCount } Else { 0 }
+        'NumberOfFailedTests'           = If ( $TestResult ) { $TestResult.FailedCount } Else { 0 }
+        'NumberOfPassedTests'           = If ( $TestResult ) { $TestResult.PassedCount } Else { 0 }
+        'TestsPassRate'                 = If ($TestResult.TotalCount) { [math]::Round(($TestResult.PassedCount / $TestResult.TotalCount) * 100, 2) } Else { 0 }
         'TestCoverage'                  = $CodeCoveragePerCent
         'CommandsMissedTotal'           = $CommandsMissed
         'ComplexityAverage'             = [math]::Round(($FunctionHealthRecord.Complexity | Measure-Object -Average).Average, 2)

diff --git a/Tests/Unit/Private/New-PSCodeHealthReport.Tests.ps1 b/Tests/Unit/Private/New-PSCodeHealthReport.Tests.ps1
@@ -52,6 +52,15 @@ Describe 'New-PSCodeHealthReport' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 0
             }
@@ -117,6 +126,15 @@ Describe 'New-PSCodeHealthReport' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 51
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 7
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 44
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 86.27
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 81.48
             }
@@ -177,6 +195,15 @@ Describe 'New-PSCodeHealthReport' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 0
             }
@@ -239,6 +266,15 @@ Describe 'New-PSCodeHealthReport' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 0
             }
@@ -294,6 +330,15 @@ Describe 'New-PSCodeHealthReport' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 0
             }
@@ -350,6 +395,15 @@ Describe 'New-PSCodeHealthReport' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should BeNullOrEmpty
             }

diff --git a/Tests/Unit/Public/Invoke-PSCodeHealth2.Tests.ps1 b/Tests/Unit/Public/Invoke-PSCodeHealth2.Tests.ps1
@@ -48,6 +48,15 @@ Describe 'Invoke-PSCodeHealth (again)' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 0
             }
@@ -108,6 +117,15 @@ Describe 'Invoke-PSCodeHealth (again)' {
             It 'Should return an object with the expected property "NumberOfTests"' {
                 $Result.NumberOfTests | Should Be 0
             }
+            It 'Should return an object with the expected property "NumberOfFailedTests"' {
+                $Result.NumberOfFailedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "NumberOfPassedTests"' {
+                $Result.NumberOfPassedTests | Should Be 0
+            }
+            It 'Should return an object with the expected property "TestsPassRate"' {
+                $Result.TestsPassRate | Should Be 0
+            }
             It 'Should return an object with the expected property "TestCoverage"' {
                 $Result.TestCoverage | Should Be 0
             }

diff --git a/Tests/Unit/TestData/MockObjects.json b/Tests/Unit/TestData/MockObjects.json
@@ -521,6 +521,8 @@
         {
             "NumberOfTests": {
                 "TotalCount": 51,
+                "FailedCount": 7,
+                "PassedCount": 44,
                 "CodeCoverage": {
                     "NumberOfCommandsAnalyzed": 27,
                     "NumberOfCommandsMissed": 5,