Fixed #10413, tweak addPoint function to returning point.

highcharts · Apr 23, 2019 · 860c744 · 860c744
1 parent c1c9083
commit 860c744
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 0 deletions.
diff --git a/js/parts/Dynamics.js b/js/parts/Dynamics.js
@@ -924,6 +924,7 @@ extend(Series.prototype, /** @lends Series.prototype */ {
         if (redraw) {
             chart.redraw(animation); // Animation is set anyway on redraw, #5665
         }
+        return point;
     },
 
     /**

diff --git a/samples/unit-tests/series/addpoint/demo.details b/samples/unit-tests/series/addpoint/demo.details
@@ -0,0 +1,6 @@
+---
+ resources:
+   - https://code.jquery.com/qunit/qunit-2.0.1.js
+   - https://code.jquery.com/qunit/qunit-2.0.1.css
+ js_wrap: b
+...
diff --git a/samples/unit-tests/series/addpoint/demo.html b/samples/unit-tests/series/addpoint/demo.html
@@ -0,0 +1,6 @@
+<script src="https://code.highcharts.com/highcharts.js"></script>
+
+<div id="qunit"></div>
+<div id="qunit-fixture"></div>
+
+<div id="container"></div>
diff --git a/samples/unit-tests/series/addpoint/demo.js b/samples/unit-tests/series/addpoint/demo.js
@@ -0,0 +1,15 @@
+QUnit.test('Testing addPoint() function - return point #10413', function (assert) {
+    var chart = Highcharts.chart('container', {
+        series: [{
+            data: [3, 4, 5]
+        }]
+    });
+
+    var point = chart.series[0].addPoint(8);
+
+    assert.strictEqual(
+        typeof (point),
+        'object',
+        'Point should be added to array as an object.'
+    );
+});