[Findit] Modify result page to display tryjob result for test failure.

appengine/findit/templates/build_failure.html

Index: appengine/findit/templates/build_failure.html
diff --git a/appengine/findit/templates/build_failure.html b/appengine/findit/templates/build_failure.html
index a001a21489ee9e1105f56f67b3e97b66c147624e..ee079aab7d967481b8994f230aed480311b086cd 100644
--- a/appengine/findit/templates/build_failure.html
+++ b/appengine/findit/templates/build_failure.html
@@ -263,34 +263,36 @@
     function displayTryJobResult() {
       $.getJSON('try-job-result?url=' + findit.buildUrl, function(data) {
           if (! jQuery.isEmptyObject(data)) {
-            $('#try_job_status').html(data.status);
-            if (data.status == 'Error') {
-              $('#try_job_status').addClass('error');
-            } else if(data.status == 'Analyzed') {
-              $('#try_job_status').addClass('completed');
-            } else {
-              $('#try_job_status').append(', will refresh in 1 minute...');
-              $('#try_job_status').addClass('running');
-            }
-            if ('try_job_url' in data) {
-              $('#try_job_url').html(' <a href="' + data.try_job_url + '">try job url</a>');
-            }
-
-            var tryJobStr = '';
-            var repeatStatusList = ["Pending", "Analyzing"]
-            if (repeatStatusList.indexOf(data.status) > -1) {
-              $('#try_job_result').addClass('not-display');
-              setTimeout(displayTryJobResult, 60000);
-            } else if (data.status == 'Analyzed' && 'review_url' in data) {
-              tryJobStr += '<br>culprit: <a href="' + data.review_url + '">';
-              tryJobStr += data.commit_position || data.revision + '</a>';
-              $('#try_job_result').html(tryJobStr);
-              $('#try_job_result').removeClass('not-display');
-            } else {
-              $('#try_job_result').addClass('not-display');
-            }
-          } else {
-            $('#try_job').addClass('not-display');
+            var tableString = '<table><tr><th>Step</th><th>Test</th><th>Try Job</th><th>Status</th><th>Culprit</th></tr>';
+            $.each(data, function(key, value) {
+              tableString += '<tr>';
+              tableString += '<td>' + value.step + '</td>';
+              tableString += '<td>' + value.test + '</td>';
+              if (value.try_job_url) {
+                tableString += '<td><a href="' + value.try_job_url + '">' + value.try_job_build_number + '</a></td>';
+              } else {
+                tableString += '<td></td>';
+              }
+              if (value.status == 'Completed') {
+                status_class = 'completed';
+              } else if (value.status == 'Error') {
+                status_class = 'error';
+              } else {
+                status_class = 'running';

[stgao, 2016/02/02 22:51:27]

nit: Maybe add a comment for the CSS class sharing.

[chanli, 2016/02/02 23:38:08]

Done.

+              }
+              tableString += '<td><span class="' + status_class + '">' + value.status + '</span></td>';
+              if (value.review_url) {
+                tableString += '<td><a href="' + value.review_url + '">' + value.commit_position || value.revision + '</a></td>';
+              } else if (value.revision) {
+                tableString += '<td>' + value.revision + '</td>';
+              } else {
+                tableString += '<td>Not Found</td>';
+              }
+              tableString += '</tr>';
+            });
+            tableString += '</table>';
+            $('#try_job_result').html(tableString);
+            $('#try_job').removeClass('not-display');
           }
       });
     }
@@ -364,11 +366,9 @@
   </div>
   <br>
 
-  <div id='try_job'>
+  <div id='try_job' class='not-display'>
     <b> Try Job Result:</b>
-    <br>
-    status: <span id='try_job_status'></span><span id="try_job_url"></span>
-    <span id='try_job_result'></span>
+    <div id='try_job_result'></div>
     <br>
   </div>
   <br>