standardize fail logic in bar test

Author: alexcjohnson

test/jasmine/tests/bar_test.js

@@ -3,15 +3,14 @@ var Plotly = require('@lib/index');
 var Bar = require('@src/traces/bar');
 var Lib = require('@src/lib');
 var Plots = require('@src/plots/plots');
+var Drawing = require('@src/components/drawing');
 
-var PlotlyInternal = require('@src/plotly');
-var Axes = PlotlyInternal.Axes;
+var Axes = require('@src/plots/cartesian/axes');
 
 var createGraphDiv = require('../assets/create_graph_div');
 var destroyGraphDiv = require('../assets/destroy_graph_div');
 var fail = require('../assets/fail_test');
 var customMatchers = require('../assets/custom_matchers');
-var failTest = require('../assets/fail_test');
 
 describe('Bar.supplyDefaults', function() {
     'use strict';
@@ -855,9 +854,9 @@ describe('A bar plot', function() {
             }
 
             expect(foundTextNodes).toBe(true);
-
-            done();
-        });
+        })
+        .catch(fail)
+        .then(done);
     });
 
     it('should show bar texts (outside case)', function(done) {
@@ -889,9 +888,9 @@ describe('A bar plot', function() {
             }
 
             expect(foundTextNodes).toBe(true);
-
-            done();
-        });
+        })
+        .catch(fail)
+        .then(done);
     });
 
     it('should show bar texts (horizontal case)', function(done) {
@@ -922,9 +921,9 @@ describe('A bar plot', function() {
             }
 
             expect(foundTextNodes).toBe(true);
-
-            done();
-        });
+        })
+        .catch(fail)
+        .then(done);
     });
 
     it('should show bar texts (barnorm case)', function(done) {
@@ -957,9 +956,9 @@ describe('A bar plot', function() {
             }
 
             expect(foundTextNodes).toBe(true);
-
-            done();
-        });
+        })
+        .catch(fail)
+        .then(done);
     });
 
     it('should be able to restyle', function(done) {
@@ -1107,9 +1106,9 @@ describe('A bar plot', function() {
             assertTextIsInsidePath(text12, path12); // inside
             assertTextIsInsidePath(text20, path20); // inside
             assertTextIsInsidePath(text30, path30); // inside
-
-            done();
-        });
+        })
+        .catch(fail)
+        .then(done);
     });
 
     it('should coerce text-related attributes', function(done) {
@@ -1189,9 +1188,9 @@ describe('A bar plot', function() {
             assertTextFont(textNodes[0], expected.insidetextfont, 0);
             assertTextFont(textNodes[1], expected.outsidetextfont, 1);
             assertTextFont(textNodes[2], expected.insidetextfont, 2);
-
-            done();
-        });
+        })
+        .catch(fail)
+        .then(done);
     });
 });
 
@@ -1248,7 +1247,9 @@ describe('bar hover', function() {
 
             var mock = Lib.extendDeep({}, require('@mocks/11.json'));
 
-            Plotly.plot(gd, mock.data, mock.layout).then(done);
+            Plotly.plot(gd, mock.data, mock.layout)
+            .catch(fail)
+            .then(done);
         });
 
         it('should return the correct hover point data (case x)', function() {
@@ -1272,7 +1273,9 @@ describe('bar hover', function() {
 
             var mock = Lib.extendDeep({}, require('@mocks/bar_attrs_group_norm.json'));
 
-            Plotly.plot(gd, mock.data, mock.layout).then(done);
+            Plotly.plot(gd, mock.data, mock.layout)
+            .catch(fail)
+            .then(done);
         });
 
         it('should return the correct hover point data (case y)', function() {
@@ -1387,7 +1390,7 @@ describe('bar hover', function() {
                     expect(out).toBe(false, hoverSpec);
                 });
             })
-            .catch(failTest)
+            .catch(fail)
             .then(done);
         });
 
@@ -1425,7 +1428,7 @@ describe('bar hover', function() {
                 expect(out.style).toEqual([1, 'red', 200, 1]);
                 assertPos(out.pos, [203, 304, 168, 168]);
             })
-            .catch(failTest)
+            .catch(fail)
             .then(done);
         });
     });