Account for unknown status test cases in threshold (#176)

* Account for 'unknownStatus' test cases

* Increment version

Author: tatlax3636

pom.xml

@@ -4,7 +4,7 @@
     <artifactId>api-audit</artifactId>
     <packaging>jar</packaging>
     <name>${project.groupId}:${project.artifactId}</name>
-    <version>3.7.25-SNAPSHOT</version>
+    <version>3.7.26-SNAPSHOT</version>
     <description>Hygieia Audit Rest API Layer</description>
     <url>https://github.com/Hygieia/${repository.name}</url>
 

src/main/java/com/capitalone/dashboard/evaluator/FeatureTestResultEvaluator.java

@@ -47,7 +47,9 @@ public class FeatureTestResultEvaluator extends Evaluator<TestResultsAuditRespon
     private static final String TEST_CASE_SUCCESS_COUNT = "successTestCaseCount";
     private static final String TEST_CASE_FAILURE_COUNT = "failureTestCaseCount";
     private static final String TEST_CASE_SKIPPED_COUNT = "skippedTestCaseCount";
+    private static final String TEST_CASE_UNKNOWN_COUNT = "unknownStatusTestCaseCount";
     private static final String TEST_CASE_TOTAL_COUNT = "totalTestCaseCount";
+
     private static final String PRIORITY_HIGH = "High";
 
 
@@ -153,6 +155,7 @@ private TestResultsAuditResponse updateTestResultAuditStatuses(List<TestCapabili
         }
 
         double testCasePassPercent = this.getTestCasePassPercent(testCapabilities);
+
         if (testCasePassPercent < threshold) {
             testResultsAuditResponse.addAuditStatus(TestResultAuditStatus.TEST_RESULT_AUDIT_FAIL);
         } else {
@@ -180,13 +183,16 @@ private double getTestCasePassPercent(List<TestCapability> testCapabilities) {
             double testCaseSkipCount = testCapabilities.stream().mapToDouble(testCapability ->
                     testCapability.getTestSuites().parallelStream().mapToDouble(TestSuite::getSkippedTestCaseCount).sum()
             ).sum();
+            double testCaseUnkownCount = testCapabilities.stream().mapToDouble(testCapability ->
+                    testCapability.getTestSuites().parallelStream().mapToDouble(TestSuite::getUnknownStatusCount).sum()
+            ).sum();
 
-            if(totalTestCaseCount == 0 || testCaseSkipCount == totalTestCaseCount){
+            if(totalTestCaseCount == 0 || (testCaseSkipCount + testCaseUnkownCount) == totalTestCaseCount){
                 return 100.0;
             }
 
-            return (testCaseSuccessCount/(totalTestCaseCount - testCaseSkipCount)) * 100;
-            
+            return (testCaseSuccessCount/(totalTestCaseCount - testCaseSkipCount - testCaseUnkownCount)) * 100;
+
         }catch(Exception e){
             LOGGER.error("Could not get 'testCasePassPercent', setting to 0.0%");
             return 0.0;
@@ -215,11 +221,15 @@ protected HashMap getFeatureTestResult(TestResult testResult) {
                     testCapability.getTestSuites().parallelStream().mapToInt(TestSuite::getFailedTestCaseCount).sum()).sum();
             int testCaseSkippedCount = testCapabilities.stream().mapToInt(testCapability ->
                     testCapability.getTestSuites().parallelStream().mapToInt(TestSuite::getSkippedTestCaseCount).sum()).sum();
+            int testCaseUnknownCount = testCapabilities.stream().mapToInt(testCapability ->
+                    testCapability.getTestSuites().parallelStream().mapToInt(TestSuite::getUnknownStatusCount).sum()).sum();
 
             featureTestResultMap.put(TEST_CASE_TOTAL_COUNT, totalTestCaseCount);
             featureTestResultMap.put(TEST_CASE_SUCCESS_COUNT, testCaseSuccessCount);
             featureTestResultMap.put(TEST_CASE_FAILURE_COUNT, testCaseFailureCount);
             featureTestResultMap.put(TEST_CASE_SKIPPED_COUNT, testCaseSkippedCount);
+            featureTestResultMap.put(TEST_CASE_UNKNOWN_COUNT, testCaseUnknownCount);
+
         }catch(Exception e){
             LOGGER.error("Exception occurred while processing testResult " + testResult.getDescription(), e);
         }