Autofix lint errors

waxlamp · waxlamp · commit d5ad8341dc94 · 2021-08-16T13:25:00.000-04:00
diff --git a/src/plots/smith/layout_defaults.js b/src/plots/smith/layout_defaults.js
@@ -40,9 +40,7 @@ function handleDefaults(contIn, contOut, coerce, opts) {
         axOut._id = axOut._name = axName;
         axOut._attr = opts.id + '.' + axName;
         axOut._traceIndices = subplotData.map(function(t) { return t._expandedIndex; });
-
-        var dataAttr = constants.axisName2dataArray[axName];
-        var axType = handleAxisTypeDefaults(axIn, axOut, coerceAxis, subplotData, dataAttr, opts);
+        axOut.type = 'linear';
 
         var visible = coerceAxis('visible');
         setConvert(axOut, contOut, layoutOut);
@@ -123,11 +121,6 @@ function handleDefaults(contIn, contOut, coerce, opts) {
     }
 }
 
-function handleAxisTypeDefaults(axIn, axOut) {
-    axOut.type = 'linear';
-    return axOut.type;
-}
-
 module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
     handleSubplotDefaults(layoutIn, layoutOut, fullData, {
         type: constants.name,
diff --git a/src/plots/smith/set_convert.js b/src/plots/smith/set_convert.js
@@ -121,12 +121,12 @@ function fromRadians(v, unit) {
     return unit === 'degrees' ? rad2deg(v) : v;
 }
 
-function setConvertAngular(ax, smithLayout) {
+function setConvertAngular(ax) {
     var _d2c = ax.d2c;
     var _c2d = ax.c2d;
 
-    ax.d2c = function (v, unit) { return toRadians(_d2c(v), unit); };
-    ax.c2d = function (v, unit) { return _c2d(fromRadians(v, unit)); };
+    ax.d2c = function(v, unit) { return toRadians(_d2c(v), unit); };
+    ax.c2d = function(v, unit) { return _c2d(fromRadians(v, unit)); };
 
     // override makeCalcdata to handle thetaunit and special theta0/dtheta logic
     ax.makeCalcdata = function(trace, coord) {
diff --git a/src/plots/smith/smith.js b/src/plots/smith/smith.js
@@ -1,9 +1,7 @@
 'use strict';
 
 var d3 = require('@plotly/d3');
-var tinycolor = require('tinycolor2');
 
-var Registry = require('../../registry');
 var Lib = require('../../lib');
 var strRotate = Lib.strRotate;
 var strTranslate = Lib.strTranslate;
@@ -404,7 +402,9 @@ proto.updateRadialAxis = function(fullLayout, smithLayout) {
 
         // circular grid lines
         Axes.drawGrid(gd, ax, {
-            vals: realStops.map(v => ({x: v})),
+            vals: realStops.map(function(v) {
+                return {x: v};
+            }),
             layer: layers['radial-grid'],
             path: gridPathFn,
             transFn: Lib.noop,
diff --git a/src/traces/scattersmith/attributes.js b/src/traces/scattersmith/attributes.js
@@ -4,7 +4,6 @@ var hovertemplateAttrs = require('../../plots/template_attributes').hovertemplat
 var texttemplateAttrs = require('../../plots/template_attributes').texttemplateAttrs;
 var extendFlat = require('../../lib/extend').extendFlat;
 var scatterAttrs = require('../scatter/attributes');
-var baseAttrs = require('../../plots/attributes');
 var lineAttrs = scatterAttrs.line;
 
 module.exports = {
diff --git a/test/jasmine/tests/scattersmith_test.js b/test/jasmine/tests/scattersmith_test.js
@@ -1,7 +1,7 @@
 var Plotly = require('@lib/index');
 var ScatterSmith = require('@src/traces/scattersmith');
 var Lib = require('@src/lib');
-const { assertHoverLabelContent } = require('../assets/custom_assertions');
+var assertHoverLabelContent = require('../assets/custom_assertions').assertHoverLabelContent;
 
 var createGraphDiv = require('../assets/create_graph_div');
 var destroyGraphDiv = require('../assets/destroy_graph_div');
@@ -41,7 +41,7 @@ describe('Test scattersmith trace defaults:', function() {
     });
 });
 
-describe('Test scattersmith hover', function () {
+describe('Test scattersmith hover', function() {
     var gd;
 
     afterEach(destroyGraphDiv);
@@ -60,7 +60,7 @@ describe('Test scattersmith hover', function () {
 
         var pos = specs.pos || [200, 200];
 
-        return Plotly.newPlot(gd, fig).then(function () {
+        return Plotly.newPlot(gd, fig).then(function() {
             mouseEvent('mousemove', pos[0], pos[1]);
             assertHoverLabelContent(specs);
         });
@@ -71,9 +71,9 @@ describe('Test scattersmith hover', function () {
         nums: '0.1 + 0.1j',
         pos: [80, 175],
         name: 'trace 0'
-    }].forEach(function (specs) {
-        it('should generate correct hover labels ' + specs.desc, function (done) {
+    }].forEach(function(specs) {
+        it('should generate correct hover labels ' + specs.desc, function(done) {
             run(specs).then(done, done.fail);
         });
-    })
+    });
 });
diff --git a/test/jasmine/tests/smith_test.js b/test/jasmine/tests/smith_test.js
@@ -24,7 +24,7 @@ describe('Test smith plot defaults:', function() {
         _supply({
             smith: {}
         });
-        
+
         var smith = layoutOut.smith;
 
         expect(smith.domain.x).toEqual([0, 1]);
