add tests for hover/click event data keys (also rename hover_label_test to hover_test)

Author: emilykl

test/jasmine/tests/click_test.js

@@ -110,6 +110,23 @@ describe('Test click interactions:', function() {
             expect(contextPassthroughs).toBe(0);
         });
 
+        function checkData() {
+            expect(Object.keys(futureData).sort()).toEqual([
+                'event', 'points', 'xaxes', 'yaxes', 'xvals', 'yvals', 'xPixel', 'yPixel'
+            ].sort());
+
+            expect(futureData.event).not.toBe(null);
+            checkPointData();
+            // xvals, yvals, xaxes, and yaxes should all be undefined since clickanywhere is not enabled
+            expect(futureData.xvals).toBe(undefined);
+            expect(futureData.yvals).toBe(undefined);
+            expect(futureData.xaxes).toBe(undefined);
+            expect(futureData.yaxes).toBe(undefined);
+            // However, xPixel and yPixel should be defined and match the click position
+            expect(futureData.xPixel).toEqual(pointPos[0]);
+            expect(futureData.yPixel).toEqual(pointPos[1]);
+        }
+
         function checkPointData() {
             expect(futureData.points.length).toEqual(1);
             expect(clickPassthroughs).toBe(2);
@@ -131,14 +148,14 @@ describe('Test click interactions:', function() {
             expect(evt.clientY).toEqual(pointPos[1]);
         }
 
-        it('should contain the correct fields', function() {
+        it('should contain the correct fields with the correct values', function() {
             click(pointPos[0], pointPos[1]);
-            checkPointData();
+            checkData();
         });
 
         it('should work with a sloppy click (shift < minDrag before mouseup)', function() {
             click(pointPos[0], pointPos[1], {slop: [4, 4]});
-            checkPointData();
+            checkData();
         });
 
         it('works with fixedrange axes', function(done) {

test/jasmine/tests/hover_label_test.js test/jasmine/tests/hover_test.jstest/jasmine/tests/hover_label_test.js renamed to test/jasmine/tests/hover_test.js

@@ -1827,6 +1827,42 @@ describe('hover info', function () {
                 })
                 .then(done, done.fail);
         });
+
+        it('should emit event data with the expected keys and values', function (done) {
+            var hoverData;
+            gd.on('plotly_hover', function (d) {
+                hoverData = d;
+            });
+
+            Promise.resolve()
+                .then(function () {
+                    _hoverNatural(gd, 250, 200);
+                })
+                .then(function () {
+                    expect(Object.keys(hoverData).sort()).toEqual(
+                        ['event', 'points', 'xaxes', 'yaxes', 'xvals', 'yvals', 'xPixel', 'yPixel'].sort()
+                    );
+
+                    expect(hoverData.event).not.toBe(null);
+                    var pt = hoverData.points[0];
+                    expect(Object.keys(pt).sort()).toEqual([
+                        'data', 'fullData', 'curveNumber', 'pointNumber', 'pointIndex',
+                        'bbox', 'label', 'value',
+                        'x', 'y', 'xaxis', 'yaxis', 'xPixel', 'yPixel'
+                    ].sort());
+                    expect(pt.curveNumber).toEqual(0);
+                    expect(pt.pointNumber).toEqual(1);
+                    expect(pt.x).toEqual(2);
+                    expect(pt.y).toEqual(3);
+                    expect(hoverData.xvals).toEqual([2.2045454545454546]);
+                    expect(hoverData.yvals).toEqual([0.28708133971291905]);
+                    expect(hoverData.xaxes).toEqual([gd._fullLayout.xaxis]);
+                    expect(hoverData.yaxes).toEqual([gd._fullLayout.yaxis]);
+                    expect(hoverData.xPixel).toEqual(330);
+                    expect(hoverData.yPixel).toEqual(300);
+                })
+                .then(done, done.fail);
+        });
     });
 
     describe('overflowing hover labels', function () {