From 47107f67889d42a94cd9c7c283a30671d1c78fd4 Mon Sep 17 00:00:00 2001
From: deniz kusefoglu <denizk@gmail.com>
Date: Thu, 11 Jan 2018 14:45:24 -0800
Subject: [PATCH] Fix slack config save and test button behaviors

---
 ui/src/kapacitor/components/config/SlackConfig.js | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/ui/src/kapacitor/components/config/SlackConfig.js b/ui/src/kapacitor/components/config/SlackConfig.js
index a8c3d719f..462373317 100644
--- a/ui/src/kapacitor/components/config/SlackConfig.js
+++ b/ui/src/kapacitor/components/config/SlackConfig.js
@@ -12,10 +12,6 @@ class SlackConfig extends Component {
 
   handleSubmit = e => {
     e.preventDefault()
-    if (this.state.testEnabled) {
-      this.props.onTest()
-      return
-    }
     const properties = {
       url: this.url.value,
       channel: this.channel.value,
@@ -68,7 +64,6 @@ class SlackConfig extends Component {
             className="btn btn-primary"
             type="submit"
             disabled={this.state.testEnabled}
-            onClick={this.enableTest}
           >
             <span className="icon checkmark" />
             Save Changes
@@ -76,7 +71,7 @@ class SlackConfig extends Component {
           <button
             className="btn btn-primary"
             disabled={!this.state.testEnabled}
-            onClick={this.enableTest}
+            onClick={this.props.onTest}
           >
             <span className="icon pulse-c" />
             Send Test Alert
@@ -99,7 +94,7 @@ SlackConfig.propTypes = {
     }).isRequired,
   }).isRequired,
   onSave: func.isRequired,
-  onTest: func,
+  onTest: func.isRequired,
   enabled: bool.isRequired,
 }