Portal Bug - non proper display of Non Exec TCs

The non executed test cases have not any verdict,
assigned by the dovetail project (e.g. PASS, SKIP, FAIL).
In that case, the portal cannot recognize those TCs as failed ones
and they are out of the 'Not Passed' list in the Test Result Overview.

The problem is focused on NFVI portal.

Therefore, the controller has been updated, removing the check for the
failed TCs.
If the TC is not belonged either to Pass or Skip list then this TC is
pushed in the Not Pass list.
So, the failed as well as the non executed TCs appear under
'Non Passed' list.

JIRA: DOVETAIL-773

Change-Id: I71e639c168bbda1065e12b063e22f17875536a9a
Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>

1 files changed, 6 insertions, 6 deletions

diff --git a/3rd_party/static/testapi-ui/components/results-report/resultsReportController.js b/3rd_party/static/testapi-ui/components/results-report/resultsReportController.js
index 0c99962..7db6cb9 100644
--- a/3rd_party/static/testapi-ui/components/results-report/resultsReportController.js
+++ b/3rd_party/static/testapi-ui/components/results-report/resultsReportController.js
@@ -148,7 +148,7 @@
                                 ctrl.data.mandatory[name].skip += 1;
                                 ctrl.statistics.mandatory.skip += 1;
                                 ctrl.statistics.skip += 1;
-                            }else if(ctrl.case_list_fail.indexOf(sub_case) > -1){
+                            }else {
                                 ctrl.data.mandatory[name].fail += 1;
                                 ctrl.statistics.mandatory.fail += 1;
                                 ctrl.statistics.fail += 1;
@@ -173,7 +173,7 @@
                                 ctrl.data.optional[name].skip += 1;
                                 ctrl.statistics.optional.skip += 1;
                                 ctrl.statistics.skip += 1;
-                            }else if(ctrl.case_list_fail.indexOf(sub_case) > -1){
+                            }else {
                                 ctrl.data.optional[name].fail += 1;
                                 ctrl.statistics.optional.fail += 1;
                                 ctrl.statistics.fail += 1;
@@ -262,7 +262,7 @@
                         case_list.push(result.case_name);
                     } else if(ele.benchmark.data.sla_skip == 1){
                         case_list_skip.push(result.case_name);
-                    } else if(ele.benchmark.data.sla_fail == 1){
+                    } else {
                         case_list_fail.push(result.case_name);
                     }
                 }
@@ -291,7 +291,7 @@
                     case_list.push(result.case_name);
                 } else if(result.criteria == 'SKIP'){
                     case_list_skip.push(result.case_name);
-                } else if(result.criteria == 'FAIL') {
+                } else {
                     case_list_fail.push(result.case_name);
                 }
             }
@@ -308,7 +308,7 @@
             else if(result.sub_testcase.length == 0 && result.result == "SKIP") {
                 case_list_skip.push(result.name);
             }
-            else if(result.sub_testcase.length == 0 && result.result == "FAIL") {
+            else if(result.sub_testcase.length == 0) {
                 case_list_fail.push(result.name);
             }
             else {
@@ -317,7 +317,7 @@
                         case_list.push(subtest.name);
                     } else if(subtest.result == "SKIP") {
                         case_list_skip.push(subtest.name);
-                    } else if(subtest.result == "FAIL") {
+                    } else {
                         case_list_fail.push(subtest.name);
                     }
                 });