Test non-existent divs
authorDan Vanderkam <danvdk@gmail.com>
Sat, 28 Mar 2015 18:31:57 +0000 (14:31 -0400)
committerDan Vanderkam <danvdk@gmail.com>
Sat, 28 Mar 2015 18:32:44 +0000 (14:32 -0400)
auto_tests/tests/pathological_cases.js
src/dygraph.js

index ade714c..40e877f 100644 (file)
@@ -160,4 +160,18 @@ it('testConstantSeriesNegativeIncludeZero', function() {
   assert.deepEqual([-1.1, 0], g.yAxisRange());
 });
 
+it('should throw with non-existent divs', function() {
+  var data = "X,Y\n" +
+             "1,-1\n" +
+             "2,1\n";
+
+  assert.throws(function() {
+    new Dygraph(null, data);
+  }, /non-existent div/);
+
+  assert.throws(function() {
+    new Dygraph('non-existent-div-id', data);
+  }, /non-existent div/);
+});
+
 });
index 7121458..735ddca 100644 (file)
@@ -403,7 +403,7 @@ Dygraph.prototype.__init__ = function(div, file, attrs) {
   }
 
   if (!div) {
-    throw 'Constructing dygraph with a non-existent div!';
+    throw new Error('Constructing dygraph with a non-existent div!');
   }
 
   // Copy the important bits into the object