From 622831eee57a61c85d2c69862bce9d2d19f090df Mon Sep 17 00:00:00 2001
From: Anton Sankov <a.sankov96@gmail.com>
Date: Wed, 18 Aug 2021 19:50:42 +0300
Subject: [PATCH] wip: Add individual test case health UI

to test run page
---
 tcms/testruns/static/testruns/js/get.js   | 54 +++++++++++++++++++++--
 tcms/testruns/templates/testruns/get.html |  5 +++
 2 files changed, 55 insertions(+), 4 deletions(-)

diff --git a/tcms/testruns/static/testruns/js/get.js b/tcms/testruns/static/testruns/js/get.js
index 4ea7b998ae..924a27d4da 100644
--- a/tcms/testruns/static/testruns/js/get.js
+++ b/tcms/testruns/static/testruns/js/get.js
@@ -7,6 +7,7 @@ const permissions = {
   removeComment: false
 }
 const autocompleteCache = {}
+const testCaseIds = []
 
 $(document).ready(() => {
   permissions.removeTag = $('#test_run_pk').data('perm-remove-tag') === 'True'
@@ -149,9 +150,45 @@ $(document).ready(() => {
     }
 
     jsonRPC('TestExecution.filter', rpcQuery, testExecutions => {
+      testExecutions.forEach(te => testCaseIds.push(te.case))
+
       drawPercentBar(testExecutions)
-      renderTestExecutions(testExecutions)
       renderAdditionalInformation(testRunId)
+
+      console.log(testCaseIds)
+
+      jsonRPC('Testing.individual_test_case_health', { case_id__in: testCaseIds }, result => {
+        console.log(result)
+
+        const testCaseHealth = {}
+        let caseId = 0
+        result.forEach(r => {
+          let positive = 0
+          let negative = 0
+          let allCount = 0
+          if (r.status__weight > 0) {
+            positive++
+          } else if (r.status__weight < 0) {
+            negative++
+          }
+          allCount++
+
+          if (r.case_id !== caseId) {
+            caseId = r.case_id
+            testCaseHealth[caseId] = {
+              completion_rate: allCount > 0 ? ((positive + negative) / allCount) : 0,
+              failure_rate: allCount > 0 ? (negative / allCount) : 0
+            }
+            positive = 0
+            negative = 0
+            allCount = 0
+          }
+        })
+
+        console.log(testCaseHealth)
+        renderTestExecutions(testExecutions, testCaseHealth)
+      })
+
     })
   })
 
@@ -357,7 +394,7 @@ function renderCountPerStatusList (statusCount) {
   }
 }
 
-function renderTestExecutions (testExecutions) {
+function renderTestExecutions (testExecutions, testCaseHealth) {
   // sort executions by sortkey
   testExecutions.sort(function (te1, te2) {
     return te1.sortkey - te2.sortkey
@@ -365,7 +402,7 @@ function renderTestExecutions (testExecutions) {
   const container = $('#test-executions-container')
 
   testExecutions.forEach(testExecution => {
-    container.append(renderTestExecutionRow(testExecution))
+    container.append(renderTestExecutionRow(testExecution, testCaseHealth))
   })
 
   bindEvents()
@@ -629,7 +666,7 @@ function renderHistoryEntry (historyEntry) {
   return template
 }
 
-function renderTestExecutionRow (testExecution) {
+function renderTestExecutionRow (testExecution, testCaseHealth={}) {
   // refresh the internal data structure b/c some fields are used
   // to render the expand area and may have changed via bulk-update meanwhile
   testExecution.status__name = $('#test_run_pk').data(`trans-execution-status-${testExecution.status}`)
@@ -647,6 +684,15 @@ function renderTestExecutionRow (testExecution) {
   template.find('.test-execution-info-link').attr('href', `/case/${testExecution.case}/`)
   template.find('.test-execution-tester').html(testExecution.tested_by__username || '-')
   template.find('.test-execution-asignee').html(testExecution.assignee__username || '-')
+  // TODO: this is just for visualization of how we will be using the API
+  // this will probably be replaced by a visual icon
+  const testExecutionCaseHealth = testCaseHealth[testExecution.case]
+  // this may be empty if we are reloading the execution after an update
+  // in this case it's unlikely that the health of the test case has changed
+  if (testExecutionCaseHealth) {
+    template.find('.test-case-completion-rate').html(testExecutionCaseHealth.completion_rate)
+    template.find('.test-case-failure-rate').html(testExecutionCaseHealth.failure_rate)
+  }
 
   const testExecutionStatus = allExecutionStatuses[testExecution.status]
   template.find('.test-execution-status-icon').addClass(testExecutionStatus.icon).css('color', testExecutionStatus.color)
diff --git a/tcms/testruns/templates/testruns/get.html b/tcms/testruns/templates/testruns/get.html
index 4261697842..a7c8bcaf3d 100644
--- a/tcms/testruns/templates/testruns/get.html
+++ b/tcms/testruns/templates/testruns/get.html
@@ -407,6 +407,11 @@ <h5><span class="test-executions-count"></span> {% trans 'records' %}</h5>
                                                     <div title="{% trans 'Bugs' %}" class="list-view-pf-additional-info-item js-bugs hidden">
                                                         <span class="fa fa-bug" style="color:#cc0000"></span>
                                                     </div>
+                                                    <div title="{% trans 'Test Case Health' %}" class="list-view-pf-additional-info-item">
+                                                        <!-- <span class="fa fa-bug" style="color:#cc0000"></span> -->
+                                                        CR: <span class="test-case-completion-rate"></span> 
+                                                        FR:  <span class="test-case-failure-rate"></span>
+                                                    </div>
                                                 </div>
                                                 <div class="list-view-pf-additional-info" style="width: 30%;">
                                                     <div title="{% trans 'Status' %}" class="list-view-pf-additional-info-item">