From b6ea783826c49bf0e90a78f165f65a5a0fbd725d Mon Sep 17 00:00:00 2001
From: netil <netil@netil.local>
Date: Wed, 8 Nov 2023 14:24:08 +0900
Subject: [PATCH] fix(tooltip): Fix tooltip.format.value arg for bar range data

- Make format function called once for range(array) data type
- Update on defaultValueFormat to handle array type value also.

Ref #3507
---
 src/ChartInternal/internals/format.ts  | 4 ++--
 src/ChartInternal/internals/tooltip.ts | 4 ++--
 test/internals/tooltip-spec.ts         | 7 +++----
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/ChartInternal/internals/format.ts b/src/ChartInternal/internals/format.ts
index f8878222b..059cfe8d3 100644
--- a/src/ChartInternal/internals/format.ts
+++ b/src/ChartInternal/internals/format.ts
@@ -50,8 +50,8 @@ export default {
 		};
 	},
 
-	defaultValueFormat(v): number|string {
-		return isValue(v) ? +v : "";
+	defaultValueFormat(v: number|number[]): number|string {
+		return isArray(v) ? v.join("~") : (isValue(v) ? +v : "");
 	},
 
 	defaultArcValueFormat(v, ratio): string {
diff --git a/src/ChartInternal/internals/tooltip.ts b/src/ChartInternal/internals/tooltip.ts
index 5bb6eb7ff..13eb4119b 100644
--- a/src/ChartInternal/internals/tooltip.ts
+++ b/src/ChartInternal/internals/tooltip.ts
@@ -198,9 +198,9 @@ export default {
 
 				value = `<b>Open:</b> ${open} <b>High:</b> ${high} <b>Low:</b> ${low} <b>Close:</b> ${close}${volume ? ` <b>Volume:</b> ${volume}` : ""}`;
 			} else if ($$.isBarRangeType(row)) {
-				const {value: [start, end], id, index} = row;
+				const {value: rangeValue, id, index} = row;
 
-				value = `${valueFormat(start, undefined, id, index)} ~ ${valueFormat(end, undefined, id, index)}`;
+				value = `${valueFormat(rangeValue, undefined, id, index)}`;
 			} else {
 				value = valueFormat(getRowValue(row), ...param);
 			}
diff --git a/test/internals/tooltip-spec.ts b/test/internals/tooltip-spec.ts
index f76ce74cf..495c8fe69 100644
--- a/test/internals/tooltip-spec.ts
+++ b/test/internals/tooltip-spec.ts
@@ -1953,7 +1953,7 @@ describe("TOOLTIP", function() {
 			util.hoverChart(chart, "mousemove", {clientX: 180, clientY: 130});
 
 			expect(chart.$.tooltip.select(".value").html())
-				.to.be.equal("1300 ~ 1339");
+				.to.be.equal("1300~1339");
 		});
 	});
 
@@ -2080,13 +2080,12 @@ describe("TOOLTIP", function() {
 			// when
 			chart.tooltip.show({x: 1});
 
-			expect(spy.callCount).to.be.equal(2);
-
+			expect(spy.callCount).to.be.equal(1);
 			expect(spy.args.every(v => v.length === 4)).to.be.true;
 			expect(spy.args.every(v => {
 				const [value, ratio, id, index]= v;
 
-				return isNumber(value) && isUndefined(ratio) && isString(id) && isNumber(index);
+				return Array.isArray(value) && isUndefined(ratio) && isString(id) && isNumber(index);
 			})).to.be.true;
 
 			spy.resetHistory();