From 5d99066c27e58d179994e70f9938a4f798d74efa Mon Sep 17 00:00:00 2001
From: Andy Librian <andylibrian@gmail.com>
Date: Sun, 20 Dec 2020 07:23:57 +0700
Subject: [PATCH] UI: Show errors from the result

---
 web/src/App.vue                      | 21 +++++++++++++++++--
 web/src/components/result_detail.vue | 19 ++++++++++--------
 web/src/components/result_errors.vue | 30 ++++++++++++++++++++++++++++
 3 files changed, 60 insertions(+), 10 deletions(-)
 create mode 100644 web/src/components/result_errors.vue

diff --git a/web/src/App.vue b/web/src/App.vue
index d3c536c..5cec78f 100644
--- a/web/src/App.vue
+++ b/web/src/App.vue
@@ -6,16 +6,22 @@
   <link href="https://fonts.googleapis.com/css2?family=Lato&family=Open+Sans&family=Roboto&display=swap" rel="stylesheet"> 
 
   <Navbar :serverInfo="serverInfo" />
+
   <div class="section">
     <LaunchTest :serverInfo="serverInfo" :serverBaseUrl="serverBaseUrl" />
   </div>
+
   <div class="section" :class="{'is-hidden': !isResultVisible}">
     <ResultSummary :summary="metricsSummary"/>
   </div>
+
   <div class="section" :class="{'is-hidden': !isResultVisible}">
     <ResultDetail :workers="workers" :summary="metricsSummary" />
   </div>
 
+  <div class="section" :class="{'is-hidden': !isErrorsVisible}">
+    <ResultErrors :workers="workers" />
+  </div>
 </template>
 
 <script>
@@ -24,6 +30,7 @@
   import LaunchTest from './components/launch_test.vue'
   import ResultSummary from './components/result_summary.vue'
   import ResultDetail from './components/result_detail.vue'
+  import ResultErrors from './components/result_errors.vue'
 
   let serverBaseUrl = process.env.VUE_APP_SERVER_BASE_URL;
   if (!serverBaseUrl) {
@@ -37,6 +44,7 @@
       LaunchTest,
       ResultSummary,
       ResultDetail,
+      ResultErrors,
     },
     data: function() {
       return {
@@ -47,6 +55,7 @@
         workers: {},
         serverBaseUrl: serverBaseUrl,
         isResultVisible: false,
+        isErrorsVisible: false,
       }
     },
     created: function() {
@@ -92,7 +101,10 @@
                 const worker = obj[key];
                 if ("name" in worker && "metrics" in worker) {
                   _this.workers[worker.name] = worker;
-                  console.log("workers updated", _this.workers);
+
+                  if (!_this.isErrorsVisible && 'errors' in worker.metrics && worker.metrics.errors.length) {
+                    _this.isErrorsVisible = true;
+                  }
                 }
               }
             }
@@ -181,13 +193,14 @@
 
   .section .card {
     box-shadow: none;
-    border: 1px solid rgba(14, 18, 23, 0.08);
+    border: 1px solid rgba(14, 18, 23, 0.09);
     background: rgba(255, 255, 255, 0.9);
   }
 
   .section .card .card-header {
     background: rgba(100, 110, 128, 0.2) !important;
   }
+
   .section .card .card-header .card-header-title {
     border-bottom: 1px solid rgba(100, 110, 128, 0.1) !important;
   }
@@ -215,5 +228,9 @@
   .button.is-danger[disabled], fieldset[disabled] .button.is-danger {
     background-color: #e18888 !important;
   }
+
+  .section > .content {
+      padding: 0 1rem !important;
+  }
 </style>
 
diff --git a/web/src/components/result_detail.vue b/web/src/components/result_detail.vue
index e20c9df..f50ac70 100644
--- a/web/src/components/result_detail.vue
+++ b/web/src/components/result_detail.vue
@@ -52,16 +52,19 @@
 </template>
 
 <script>
-export default {
-  name: 'ResultDetail',
-  props: {
-    workers: Object,
-    summary: Object,
-  },
-  methods: {
+  export default {
+    name: 'ResultDetail',
+    props: {
+      workers: Object,
+      summary: Object,
+    },
+    methods: {
+    }
   }
-}
 </script>
 
 <style scoped>
+  .table {
+    color: #464646;
+  }
 </style>
diff --git a/web/src/components/result_errors.vue b/web/src/components/result_errors.vue
new file mode 100644
index 0000000..3f6da9b
--- /dev/null
+++ b/web/src/components/result_errors.vue
@@ -0,0 +1,30 @@
+<template>
+  <div class="content">
+    <div class="card">
+      <div class="card-header">
+        <strong class="card-header-title"><span class="icon"><i class="fas fa-exclamation-triangle" aria-hidden="true"></i></span> Errors</strong>
+      </div>
+      <div class="card-content">
+        <div v-for="(worker, name) in workers" :key="name">
+          <div v-for="(err, id) in worker.metrics.errors" :key="name + '-' + id">
+            <strong>{{ name }}</strong>&nbsp; {{ err }}
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script>
+  export default {
+    name: 'ResultErrors',
+    props: {
+      workers: Object,
+    },
+    methods: {
+    }
+  }
+</script>
+
+<style scoped>
+</style>