From 449122095ea8de15a012e8b335da4dda721d822e Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Mon, 14 Sep 2020 11:49:29 +0100
Subject: [PATCH] Show notification when testing workflow rather than feedback
 box

---
 .../workflow/SetupPanel/SetupPanel.svelte     | 43 +++----------------
 1 file changed, 6 insertions(+), 37 deletions(-)

diff --git a/packages/builder/src/components/workflow/SetupPanel/SetupPanel.svelte b/packages/builder/src/components/workflow/SetupPanel/SetupPanel.svelte
index 2c1d1928e3..e7419a05df 100644
--- a/packages/builder/src/components/workflow/SetupPanel/SetupPanel.svelte
+++ b/packages/builder/src/components/workflow/SetupPanel/SetupPanel.svelte
@@ -1,5 +1,4 @@
 <script>
-  import { fade } from "svelte/transition"
   import { getContext } from "svelte"
   import { backendUiStore, workflowStore } from "builderStore"
   import { notifier } from "builderStore/store/notifications"
@@ -10,7 +9,6 @@
   const { open, close } = getContext("simple-modal")
 
   let selectedTab = "SETUP"
-  let testResult
 
   $: workflow =
     $workflowStore.selectedWorkflow && $workflowStore.selectedWorkflow.workflow
@@ -25,14 +23,17 @@
 
   function deleteWorkflowBlock() {
     workflowStore.actions.deleteWorkflowBlock($workflowStore.selectedBlock)
-    notifier.info("Workflow block deleted.")
   }
 
   async function testWorkflow() {
     const result = await workflowStore.actions.trigger({
       workflow: $workflowStore.selectedWorkflow.workflow,
     })
-    testResult = "Workflow passed"
+    if (result.status === 200) {
+      notifier.success(`Workflow ${workflow.name} triggered successfully.`)
+    } else {
+      notifier.danger(`Failed to trigger workflow ${workflow.name}.`)
+    }
   }
 
   async function saveWorkflow() {
@@ -49,10 +50,7 @@
     <span
       class="hoverable"
       class:selected={selectedTab === 'SETUP'}
-      on:click={() => {
-        selectedTab = 'SETUP'
-        testResult = null
-      }}>
+      on:click={() => (selectedTab = 'SETUP')}>
       Setup
     </span>
   </header>
@@ -72,15 +70,6 @@
           <b>{workflow.name}</b>
         </div>
       </div>
-      {#if testResult}
-        <button
-          transition:fade|local
-          class:passed={true}
-          class:failed={false}
-          class="test-result">
-          {testResult}
-        </button>
-      {/if}
       <Button secondary wide on:click={testWorkflow}>Test Workflow</Button>
       <div class="buttons">
         <Button
@@ -159,24 +148,4 @@
     font-weight: normal;
     color: var(--ink);
   }
-
-  .test-result {
-    border: none;
-    width: 100%;
-    border-radius: 3px;
-    height: 32px;
-    font-size: 14px;
-    font-weight: 500;
-    color: var(--white);
-    text-align: center;
-    margin-bottom: 10px;
-  }
-
-  .passed {
-    background: var(--green);
-  }
-
-  .failed {
-    background: var(--red);
-  }
 </style>