fix(subchart): fix duplicated node generation on data load

Fix incorrect selection merge where causing node duplication

Ref naver#2003

src/ChartInternal/interactions/subchart.ts

@@ -195,21 +195,18 @@ export default {
 					} else {
 						const shapeUpdate = shapeChart
 							.selectAll(`.${CLASS[`chart${name}`]}`)
-							.data(targets.filter($$[`is${name}Type`].bind($$)))
-							.attr("class", chartClass);
-
-						shapeUpdate.exit().remove();
+							.attr("class", chartClass)
+							.data(targets.filter($$[`is${name}Type`].bind($$)));
 
 						const shapeEnter = shapeUpdate.enter()
 							.append("g")
 							.style("opacity", "0")
 							.attr("class", chartClass)
-							.merge(shapeUpdate);
-
-						// Bars for each data
-						shapeEnter.append("g")
+							.append("g")
 							.attr("class", shapeClass);
 
+						shapeUpdate.exit().remove();
+
 						// Area
 						v === "line" && $$.hasTypeOf("Area") &&
 							shapeEnter.append("g").attr("class", $$.getClass("areas", true));

test/interactions/subchart-spec.ts

@@ -476,4 +476,56 @@ describe("SUBCHART", () => {
 			});
 		})
 	});
+
+
+	describe("dynamic data load via .load() API", () => {
+		before(() => {
+			args = {
+				data: {
+					columns: [
+						["data1", 30, 200, 100, 400, 150, 250]
+					]
+				},
+				subchart: {
+					show: true
+				}
+			};
+		});
+
+		it("shouldn't generate duplicated nodes #1", done => {
+			// when
+			chart.load({
+				columns: [
+					["data1", 30, 20, 50, 40, 60, 50],
+					["data2", 130, 120, 150, 140, 160, 150],
+				],
+				done: function() {
+					expect(
+						this.internal.$el.subchart.main.selectAll(`.${CLASS.line}-data1`).size()
+					).to.be.equal(1);
+				}
+			});
+		});
+
+		it("shouldn't generate duplicated nodes #2", done => {
+			// when
+			chart.load({
+				columns: [
+					["data2", 130, 120, 150, 140, 160, 150],
+				],
+				unload: ["data1"],
+				done: function() {
+					const {main} = this.internal.$el.subchart;
+
+					expect(
+						main.selectAll(`.${CLASS.line}-data1`).empty()
+					).to.be.true;
+
+					expect(
+						main.selectAll(`.${CLASS.line}-data2`).size()
+					).to.be.equal(1);
+				}
+			});
+		});
+	});
 });