From 896c919a8f8cf78f50274cbdb77b78d1b4e8be02 Mon Sep 17 00:00:00 2001
From: Shahzad <shahzad.muhammad@elastic.co>
Date: Mon, 2 Nov 2020 13:12:56 +0100
Subject: [PATCH] [Uptime] Remove custom handling of license enabling (#82019)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
---
 .../__snapshots__/license_info.test.tsx.snap  | 42 +++++++----------
 .../__snapshots__/ml_flyout.test.tsx.snap     | 25 +++++-----
 .../components/monitor/ml/license_info.tsx    | 46 ++++---------------
 3 files changed, 35 insertions(+), 78 deletions(-)

diff --git a/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/license_info.test.tsx.snap b/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/license_info.test.tsx.snap
index 68e3606d880d6..edf89caaff557 100644
--- a/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/license_info.test.tsx.snap
+++ b/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/license_info.test.tsx.snap
@@ -26,24 +26,21 @@ Array [
       <p>
         In order to access duration anomaly detection, you have to be subscribed to an Elastic Platinum license.
       </p>
-      <span>
-        <a
-          class="euiButton euiButton--primary"
-          href="/app/management/stack/license_management/home"
-          rel="noopener noreferrer"
-          target="_blank"
+      <a
+        class="euiButton euiButton--primary"
+        href="/app/management/stack/license_management/home"
+        rel="noreferrer"
+      >
+        <span
+          class="euiButtonContent euiButton__content"
         >
           <span
-            class="euiButtonContent euiButton__content"
+            class="euiButton__text"
           >
-            <span
-              class="euiButton__text"
-            >
-              Start free 14-day trial
-            </span>
+            Start free 14-day trial
           </span>
-        </a>
-      </span>
+        </span>
+      </a>
     </div>
   </div>,
   <div
@@ -64,19 +61,12 @@ exports[`ShowLicenseInfo shallow renders without errors 1`] = `
     <p>
       In order to access duration anomaly detection, you have to be subscribed to an Elastic Platinum license.
     </p>
-    <span
-      onClick={[Function]}
-      onKeyDown={[Function]}
+    <EuiButton
+      color="primary"
+      href="/app/management/stack/license_management/home"
     >
-      <EuiButton
-        color="primary"
-        href="/app/management/stack/license_management/home"
-        isLoading={false}
-        target="_blank"
-      >
-        Start free 14-day trial
-      </EuiButton>
-    </span>
+      Start free 14-day trial
+    </EuiButton>
   </EuiCallOut>
   <EuiSpacer />
 </Fragment>
diff --git a/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/ml_flyout.test.tsx.snap b/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/ml_flyout.test.tsx.snap
index ac1afccd1feae..fd59b14520ce1 100644
--- a/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/ml_flyout.test.tsx.snap
+++ b/x-pack/plugins/uptime/public/components/monitor/ml/__tests__/__snapshots__/ml_flyout.test.tsx.snap
@@ -146,24 +146,21 @@ exports[`ML Flyout component shows license info if no ml available 1`] = `
               <p>
                 In order to access duration anomaly detection, you have to be subscribed to an Elastic Platinum license.
               </p>
-              <span>
-                <a
-                  class="euiButton euiButton--primary"
-                  href="/app/management/stack/license_management/home"
-                  rel="noopener noreferrer"
-                  target="_blank"
+              <a
+                class="euiButton euiButton--primary"
+                href="/app/management/stack/license_management/home"
+                rel="noreferrer"
+              >
+                <span
+                  class="euiButtonContent euiButton__content"
                 >
                   <span
-                    class="euiButtonContent euiButton__content"
+                    class="euiButton__text"
                   >
-                    <span
-                      class="euiButton__text"
-                    >
-                      Start free 14-day trial
-                    </span>
+                    Start free 14-day trial
                   </span>
-                </a>
-              </span>
+                </span>
+              </a>
             </div>
           </div>
           <div
diff --git a/x-pack/plugins/uptime/public/components/monitor/ml/license_info.tsx b/x-pack/plugins/uptime/public/components/monitor/ml/license_info.tsx
index 2461875d502b7..d9616dc588cf0 100644
--- a/x-pack/plugins/uptime/public/components/monitor/ml/license_info.tsx
+++ b/x-pack/plugins/uptime/public/components/monitor/ml/license_info.tsx
@@ -3,18 +3,15 @@
  * or more contributor license agreements. Licensed under the Elastic License;
  * you may not use this file except in compliance with the Elastic License.
  */
-import React, { useContext, useState, useEffect } from 'react';
+import React, { useContext, useEffect } from 'react';
 import { EuiCallOut, EuiButton, EuiSpacer } from '@elastic/eui';
-import { useDispatch, useSelector } from 'react-redux';
+import { useDispatch } from 'react-redux';
 import { UptimeSettingsContext } from '../../../contexts';
 import * as labels from './translations';
 import { getMLCapabilitiesAction } from '../../../state/actions';
-import { hasMLFeatureSelector } from '../../../state/selectors';
 
 export const ShowLicenseInfo = () => {
   const { basePath } = useContext(UptimeSettingsContext);
-  const [loading, setLoading] = useState<boolean>(false);
-  const hasMlFeature = useSelector(hasMLFeatureSelector);
 
   const dispatch = useDispatch();
 
@@ -22,29 +19,6 @@ export const ShowLicenseInfo = () => {
     dispatch(getMLCapabilitiesAction.get());
   }, [dispatch]);
 
-  useEffect(() => {
-    let retryInterval: any;
-    if (loading) {
-      retryInterval = setInterval(() => {
-        dispatch(getMLCapabilitiesAction.get());
-      }, 5000);
-    } else {
-      clearInterval(retryInterval);
-    }
-
-    return () => {
-      clearInterval(retryInterval);
-    };
-  }, [dispatch, loading]);
-
-  useEffect(() => {
-    setLoading(false);
-  }, [hasMlFeature]);
-
-  const startLicenseTrial = () => {
-    setLoading(true);
-  };
-
   return (
     <>
       <EuiCallOut
@@ -55,16 +29,12 @@ export const ShowLicenseInfo = () => {
         iconType="help"
       >
         <p>{labels.START_TRAIL_DESC}</p>
-        <span onClick={startLicenseTrial} onKeyDown={() => {}}>
-          <EuiButton
-            color="primary"
-            isLoading={loading}
-            target="_blank"
-            href={basePath + `/app/management/stack/license_management/home`}
-          >
-            {labels.START_TRAIL}
-          </EuiButton>
-        </span>
+        <EuiButton
+          color="primary"
+          href={basePath + `/app/management/stack/license_management/home`}
+        >
+          {labels.START_TRAIL}
+        </EuiButton>
       </EuiCallOut>
       <EuiSpacer />
     </>