From 449a7678539da38ca0c67ce9b41f07505071f8b4 Mon Sep 17 00:00:00 2001
From: Clint Andrew Hall <clint.hall@elastic.co>
Date: Fri, 5 Jun 2020 16:32:45 -0500
Subject: [PATCH] Enable exhaustive-deps; correct any lint warnings

---
 .eslintrc.js                                     |  1 -
 .../workpad/workpad_app/workpad_telemetry.tsx    | 16 +++++-----------
 2 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index f0b7d6864bef0..bf27022f13465 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -91,7 +91,6 @@ module.exports = {
     {
       files: ['x-pack/plugins/canvas/**/*.{js,ts,tsx}'],
       rules: {
-        'react-hooks/exhaustive-deps': 'off',
         'jsx-a11y/click-events-have-key-events': 'off',
       },
     },
diff --git a/x-pack/plugins/canvas/public/apps/workpad/workpad_app/workpad_telemetry.tsx b/x-pack/plugins/canvas/public/apps/workpad/workpad_app/workpad_telemetry.tsx
index 47b461f22ad65..3014369d94857 100644
--- a/x-pack/plugins/canvas/public/apps/workpad/workpad_app/workpad_telemetry.tsx
+++ b/x-pack/plugins/canvas/public/apps/workpad/workpad_app/workpad_telemetry.tsx
@@ -69,6 +69,7 @@ export const withUnconnectedElementsLoadedTelemetry = <P extends {}>(
 ) =>
   function ElementsLoadedTelemetry(props: ElementsLoadedTelemetryProps) {
     const { telemetryElementCounts, workpad, telemetryResolvedArgs, ...other } = props;
+    const { error, pending } = telemetryElementCounts;
 
     const [currentWorkpadId, setWorkpadId] = useState<string | undefined>(undefined);
     const [hasReported, setHasReported] = useState(false);
@@ -87,27 +88,20 @@ export const withUnconnectedElementsLoadedTelemetry = <P extends {}>(
           0
         );
 
-        if (
-          workpadElementCount === 0 ||
-          (resolvedArgsAreForWorkpad && telemetryElementCounts.pending === 0)
-        ) {
+        if (workpadElementCount === 0 || (resolvedArgsAreForWorkpad && pending === 0)) {
           setHasReported(true);
         } else {
           setHasReported(false);
         }
-      } else if (
-        !hasReported &&
-        telemetryElementCounts.pending === 0 &&
-        resolvedArgsAreForWorkpad
-      ) {
-        if (telemetryElementCounts.error > 0) {
+      } else if (!hasReported && pending === 0 && resolvedArgsAreForWorkpad) {
+        if (error > 0) {
           trackMetric(METRIC_TYPE.LOADED, [WorkpadLoadedMetric, WorkpadLoadedWithErrorsMetric]);
         } else {
           trackMetric(METRIC_TYPE.LOADED, WorkpadLoadedMetric);
         }
         setHasReported(true);
       }
-    });
+    }, [currentWorkpadId, hasReported, error, pending, telemetryResolvedArgs, workpad]);
 
     return <Component {...(other as P)} workpad={workpad} />;
   };