add scatterpolar hover label tests

Author: etpinard

src/traces/scatterpolar/hover.js

@@ -19,8 +19,6 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
     var newPointData = scatterPointData[0];
 
     // hovering on fill case
-    // TODO do we need to constrain the scatter point data further (like for
-    // ternary subplots) or not?
     if(newPointData.index === undefined) {
         return scatterPointData;
     }

test/jasmine/tests/scatterpolar_test.js

@@ -1,7 +1,15 @@
-// var Plotly = require('@lib');
-// var Lib = require('@src/lib');
+var Plotly = require('@lib');
+var Lib = require('@src/lib');
 var ScatterPolar = require('@src/traces/scatterpolar');
 
+var createGraphDiv = require('../assets/create_graph_div');
+var destroyGraphDiv = require('../assets/destroy_graph_div');
+var fail = require('../assets/fail_test');
+var mouseEvent = require('../assets/mouse_event');
+
+var customAssertions = require('../assets/custom_assertions');
+var assertHoverLabelContent = customAssertions.assertHoverLabelContent;
+
 describe('Test scatterpolar trace defaults:', function() {
     var traceOut;
 
@@ -31,14 +39,80 @@ describe('Test scatterpolar trace defaults:', function() {
     });
 });
 
-describe('Test scatterpolar calc:', function() {
+describe('Test scatterpolar hover:', function() {
+    var gd;
+
+    afterEach(destroyGraphDiv);
 
-});
+    function run(specs) {
+        gd = createGraphDiv();
 
-describe('Test scatterpolar hover:', function() {
-    // gd;
+        var fig = Lib.extendDeep(
+            {width: 700, height: 500},
+            specs.mock || require('@mocks/polar_scatter.json')
+        );
 
-    it('', function() {
+        if(specs.patch) {
+            fig = specs.patch(fig);
+        }
 
+        var pos = specs.pos || [200, 200];
+
+        return Plotly.plot(gd, fig).then(function() {
+            mouseEvent('mousemove', pos[0], pos[1]);
+            assertHoverLabelContent(specs);
+        });
+    }
+
+    [{
+        desc: 'base',
+        nums: 'r: 4.022892\nθ: 128.342°',
+        name: 'Trial 3'
+    }, {
+        desc: '(no labels - out of sector)',
+        patch: function(fig) {
+            fig.layout.polar.sector = [15, 75];
+            return fig;
+        },
+        pos: [144, 350],
+        nums: '',
+        name: ''
+    }, {
+        desc: 'on a `thetaunit: radians` polar subplot',
+        patch: function(fig) {
+            fig.layout.polar.angularaxis.thetaunit = 'radians';
+            return fig;
+        },
+        nums: 'r: 4.022892\nθ: 2.239991',
+        name: 'Trial 3'
+    }, {
+        desc: 'on work on log radial axis',
+        patch: function(fig) {
+            fig.layout.polar.radialaxis.type = 'log';
+            return fig;
+        },
+        nums: 'r: 1.108937\nθ: 115.4969°',
+        name: 'Trial 3'
+    }, {
+        desc: 'on fills',
+        mock: require('@mocks/polar_fills.json'),
+        pos: [300, 230],
+        nums: 'trace 2',
+        name: ''
+    }, {
+        desc: 'on category axes',
+        mock: require('@mocks/polar_categories.json'),
+        patch: function(fig) {
+            fig.data.forEach(function(t) { t.fill = 'none'; });
+            return fig;
+        },
+        pos: [465, 90],
+        nums: 'r: 4\nθ: d',
+        name: 'angular cate...'
+    }]
+    .forEach(function(specs) {
+        it('should generate correct hover labels ' + specs.desc, function(done) {
+            run(specs).catch(fail).then(done);
+        });
     });
 });