From 5e5ecda6e699bb1b4f9d11d4cb62de525c2bd115 Mon Sep 17 00:00:00 2001
From: Dmitry Khomutov <poisoncorpsee@gmail.com>
Date: Sat, 16 Sep 2017 22:11:53 +0700
Subject: [PATCH] Improved PHPSpec plugin GUI.

---
 public/assets/js/build-plugins/phpspec.js | 22 ++++++++++------------
 public/assets/js/build-plugins/phpunit.js |  6 +++---
 2 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/public/assets/js/build-plugins/phpspec.js b/public/assets/js/build-plugins/phpspec.js
index 68c089ee..a7c36bdc 100644
--- a/public/assets/js/build-plugins/phpspec.js
+++ b/public/assets/js/build-plugins/phpspec.js
@@ -24,12 +24,14 @@ var phpspecPlugin = ActiveBuild.UiPlugin.extend({
 
     render: function() {
 
-        return $('<table class="table table-striped" id="phpspec-data">' +
+        return $('<table class="table table-hover" id="phpspec-data">' +
             '<thead>' +
             '<tr>' +
-            '   <th>'+Lang.get('suite')+'</th>' +
-            '   <th>'+Lang.get('test')+'</th>' +
-            '   <th>'+Lang.get('result')+'</th>' +
+            '   <th>' + Lang.get('status') + '</th>' +
+            '   <th>' + Lang.get('suite') + '</th>' +
+            '   <th>' + Lang.get('test') + '</th>' +
+            '   <th>' + Lang.get('test_message') + '</th>' +
+            '   <th>' + Lang.get('codeception_time') + '</th>' +
             '</tr>' +
             '</thead><tbody></tbody></table>');
     },
@@ -55,18 +57,14 @@ var phpspecPlugin = ActiveBuild.UiPlugin.extend({
 
                 var row = $(
                     '<tr>' +
+                    '<td>' + ((test_case.status == 'passed') ? '<span class="label label-success">' + Lang.get('success') + '</span>' : '<span class="label label-danger">' + Lang.get('failed') + '</span>') + '</td>' +
                     '<td>' + test_suite.name + '</td>' +
-                    '<td title="' + Lang.get('took_n_seconds', test_case['time']) + '">' + test_case.name + '</td>' +
-                    '<td>' + (test_case.message ? test_case.message : Lang.get('ok')) + '</td>' +
+                    '<td>' + test_case.name + '</td>' +
+                    '<td>' + (test_case.message ? test_case.message : '') + '</td>' +
+                    '<td>' + test_case['time'] + '</td>' +
                     '</tr>'
                 );
 
-                if (test_case.status != 'passed') {
-                    row.addClass('danger');
-                } else {
-                    row.addClass('success');
-                }
-
                 tbody.append(row);
             }
         }
diff --git a/public/assets/js/build-plugins/phpunit.js b/public/assets/js/build-plugins/phpunit.js
index d2885cf9..980d32be 100644
--- a/public/assets/js/build-plugins/phpunit.js
+++ b/public/assets/js/build-plugins/phpunit.js
@@ -34,9 +34,9 @@ var phpunitPlugin = ActiveBuild.UiPlugin.extend({
         return $('<table class="table table-hover" id="phpunit-data">' +
             '<thead>' +
             '<tr>' +
-                '<th>'+Lang.get('status')+'</th>' +
-                '<th>'+Lang.get('test_message')+'</th>' +
-                '<th>'+Lang.get('trace')+'</th>' +
+                '<th>' + Lang.get('status') + '</th>' +
+                '<th>' + Lang.get('test_message') + '</th>' +
+                '<th>' + Lang.get('trace') + '</th>' +
             '</tr>' +
             '</thead><tbody></tbody></table>');
     },