abstract load script for mathjax tests

Author: archmoj

test/jasmine/bundle_tests/mathjax_v2_test.js

@@ -4,28 +4,27 @@ var d3Select = require('../../strict-d3').select;
 var createGraphDiv = require('../assets/create_graph_div');
 var destroyGraphDiv = require('../assets/destroy_graph_div');
 
+// N.B. we have to load MathJax "dynamically" as Karam
+// does not undefined the MathJax's `?config=` parameter.
+//
+// Now with the mathjax_config no longer needed,
+// it might be nice to move these tests in the "regular" test
+// suites, but to do that we'll need to find a way to remove MathJax from
+// page without breaking things downstream.
+
+function loadScript(src, done) {
+    var newScript;
+    newScript = document.createElement('script');
+    newScript.src = src;
+    newScript.type = 'text/javascript';
+    newScript.onload = done;
+    newScript.onerror = done.fail;
+    document.body.appendChild(newScript);
+}
 
 describe('Test MathJax v2:', function() {
-    var mathJaxScriptTag;
-
-    // N.B. we have to load MathJax "dynamically" as Karam
-    // does not undefined the MathJax's `?config=` parameter.
-    //
-    // Now with the mathjax_config no longer needed,
-    // it might be nice to move these tests in the "regular" test
-    // suites, but to do that we'll need to find a way to remove MathJax from
-    // page without breaking things downstream.
     beforeAll(function(done) {
-        mathJaxScriptTag = document.createElement('script');
-        mathJaxScriptTag.type = 'text/javascript';
-        mathJaxScriptTag.onload = function() {
-            done();
-        };
-        mathJaxScriptTag.onerror = function() {
-            fail('MathJax v2 failed to load');
-        };
-        mathJaxScriptTag.src = '/base/node_modules/mathjax-v2/MathJax.js?config=TeX-AMS-MML_SVG';
-        document.body.appendChild(mathJaxScriptTag);
+        loadScript('/base/node_modules/mathjax-v2/MathJax.js?config=TeX-AMS-MML_SVG', done);
     });
 
     describe('Test axis title scoot:', function() {

test/jasmine/bundle_tests/mathjax_v3_test.js

@@ -4,27 +4,27 @@ var d3Select = require('../../strict-d3').select;
 var createGraphDiv = require('../assets/create_graph_div');
 var destroyGraphDiv = require('../assets/destroy_graph_div');
 
+// TODO: find a better way reuse the code from mathjav v2 tests
+//
+//
+// Now with the mathjax_config no longer needed,
+// it might be nice to move these tests in the "regular" test
+// suites, but to do that we'll need to find a way to remove MathJax from
+// page without breaking things downstream.
+
+function loadScript(src, done) {
+    var newScript;
+    newScript = document.createElement('script');
+    newScript.src = src;
+    newScript.type = 'text/javascript';
+    newScript.onload = done;
+    newScript.onerror = done.fail;
+    document.body.appendChild(newScript);
+}
 
 describe('Test MathJax v3:', function() {
-    var mathJaxScriptTag;
-
-    // TODO: find a better way reuse the code from mathjav v2 tests
-    //
-    // Now with the mathjax_config no longer needed,
-    // it might be nice to move these tests in the "regular" test
-    // suites, but to do that we'll need to find a way to remove MathJax from
-    // page without breaking things downstream.
     beforeAll(function(done) {
-        mathJaxScriptTag = document.createElement('script');
-        mathJaxScriptTag.type = 'text/javascript';
-        mathJaxScriptTag.onload = function() {
-            done();
-        };
-        mathJaxScriptTag.onerror = function() {
-            fail('MathJax v3 failed to load');
-        };
-        mathJaxScriptTag.src = '/base/node_modules/mathjax-v3/es5/tex-svg.js';
-        document.body.appendChild(mathJaxScriptTag);
+        loadScript('/base/node_modules/mathjax-v3/es5/tex-svg.js', done);
     });
 
     describe('Test axis title scoot:', function() {