From 8e9ee29c5aade44716691999cfe8d635374174d1 Mon Sep 17 00:00:00 2001
From: Uladzislau Lasitsa <Uladzislau_Lasitsa@epam.com>
Date: Tue, 1 Jun 2021 18:42:47 +0300
Subject: [PATCH 1/2] Don't show _all for goal and gauge in tooltip

---
 .../components/tooltip/_pointseries_tooltip_formatter.js      | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.js b/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.js
index cb8a8f72c51727..5e1f0bfbb44644 100644
--- a/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.js
+++ b/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.js
@@ -31,6 +31,7 @@ export function pointSeriesTooltipFormatter() {
 
     const details = [];
     const isGauge = config.get('gauge', false);
+    const chartType = config.get('type', undefined);
     const isPercentageMode = config.get(isGauge ? 'gauge.percentageMode' : 'percentageMode', false);
     const isSetColorRange = config.get('setColorRange', false);
 
@@ -44,7 +45,8 @@ export function pointSeriesTooltipFormatter() {
       });
     }
 
-    if (datum.x !== null && datum.x !== undefined) {
+    // For goal and gauge we have only one value for x - '_all'. It doesn't have sense to show it
+    if (datum.x !== null && datum.x !== undefined && !['goal', 'gauge'].includes(chartType)) {
       addDetail(data.xAxisLabel, data.xAxisFormatter(datum.x));
     }
 

From 196f9c4ae0543e57253d4b497bd770888cda4f37 Mon Sep 17 00:00:00 2001
From: Uladzislau Lasitsa <Uladzislau_Lasitsa@epam.com>
Date: Wed, 2 Jun 2021 13:42:32 +0300
Subject: [PATCH 2/2] add unit test

---
 .../_pointseries_tooltip_formatter.test.js    | 23 +++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.test.js b/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.test.js
index 5c0548ea399b75..a207b1f4360b6a 100644
--- a/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.test.js
+++ b/src/plugins/vis_type_vislib/public/vislib/components/tooltip/_pointseries_tooltip_formatter.test.js
@@ -96,4 +96,27 @@ describe('tooltipFormatter', function () {
     const $rows = $el.find('tr');
     expect($rows.length).toBe(3);
   });
+
+  it('renders correctly for gauge/goal visualizations', function () {
+    const event = _.cloneDeep(baseEvent);
+    let type = 'gauge';
+    event.config.get = (name) => {
+      const config = {
+        setColorRange: false,
+        gauge: false,
+        percentageMode: false,
+        type,
+      };
+      return config[name];
+    };
+
+    let $el = $(tooltipFormatter(event, uiSettings));
+    let $rows = $el.find('tr');
+    expect($rows.length).toBe(2);
+
+    type = 'goal';
+    $el = $(tooltipFormatter(event, uiSettings));
+    $rows = $el.find('tr');
+    expect($rows.length).toBe(2);
+  });
 });