From d413f7917332b8cdb23c2775dd25e928ceb47ed4 Mon Sep 17 00:00:00 2001
From: eichsjul <julian.eichstaedt@ch.sauter-bc.com>
Date: Thu, 11 Apr 2013 11:49:29 +0200
Subject: [PATCH] FEATURE: added test for issue #451

---
 auto_tests/misc/local.html                   |  1 +
 auto_tests/tests/connect_separated_points.js | 96 ++++++++++++++++++++++++++++
 2 files changed, 97 insertions(+)
 create mode 100644 auto_tests/tests/connect_separated_points.js

diff --git a/auto_tests/misc/local.html b/auto_tests/misc/local.html
index 3841372..fb99a26 100644
--- a/auto_tests/misc/local.html
+++ b/auto_tests/misc/local.html
@@ -23,6 +23,7 @@
   <script type="text/javascript" src="../tests/axis_labels.js"></script>
   <script type="text/javascript" src="../tests/axis_labels-deprecated.js"></script>
   <script type="text/javascript" src="../tests/callback.js"></script>
+  <script type="text/javascript" src="../tests/connect_separated_points.js"></script>
   <script type="text/javascript" src="../tests/css.js"></script>
   <script type="text/javascript" src="../tests/custom_bars.js"></script>
   <script type="text/javascript" src="../tests/date_formats.js"></script>
diff --git a/auto_tests/tests/connect_separated_points.js b/auto_tests/tests/connect_separated_points.js
new file mode 100644
index 0000000..65baf91
--- /dev/null
+++ b/auto_tests/tests/connect_separated_points.js
@@ -0,0 +1,96 @@
+/**
+ * @fileoverview Test cases for the option "connectSeparatedPoints" especially for the scenario where not every series has a value for each timestamp.
+ *
+ * @author julian.eichstaedt@ch.sauter-bc.com (Fr. Sauter AG)
+ */
+var StepTestCase = TestCase("connect_separated_points");
+
+StepTestCase.prototype.setUp = function() {
+  document.body.innerHTML = "<div id='graph'></div>";
+};
+
+StepTestCase.origFunc = Dygraph.getContext;
+
+StepTestCase.prototype.setUp = function() {
+  document.body.innerHTML = "<div id='graph'></div>";
+  Dygraph.getContext = function(canvas) {
+    return new Proxy(StepTestCase.origFunc(canvas));
+  };
+};
+
+StepTestCase.prototype.tearDown = function() {
+  Dygraph.getContext = StepTestCase.origFunc;
+};
+
+StepTestCase.prototype.testEdgePoints = function() {
+  var opts = {
+    width: 480,
+    height: 320,
+//    drawXGrid: false,
+//    drawYGrid: false,
+//    drawXAxis: false,
+//    drawYAxis: false,
+    errorBars: false,
+    labels: ["x", "series1", "series2"],
+    connectSeparatedPoints: true,
+    dateWindow: [1.5,6.5]
+  };
+
+  var data = [
+              [0,0,2],
+              [1,null,0.5],
+              [2,0.5,1],
+              [3,1,-1],
+              [4,2,-2],
+              [5,2.5,-2.5],
+              [6,3,-3],
+              [7,4,null],
+              [8,4,-4],
+             ];
+
+  var graph = document.getElementById("graph");
+  var g = new Dygraph(graph, data, opts);
+  
+  htx = g.hidden_ctx_;
+
+  var attrs = {};  
+
+  //Test if series1 is drawn correctly.
+  //------------------------------------
+  
+  // The first point of the first series
+  var x1 = data[0][0];
+  var y1 = data[0][1];
+  var xy1 = g.toDomCoords(x1, y1);
+  
+  // The third (the second valid) point of the first series
+  // This series has no value at the second position.
+  var x2 = data[2][0];
+  var y2 = data[2][1];
+  var xy2 = g.toDomCoords(x2, y2);
+  
+  // Check if both points are connected at the left edge of the canvas and if the option "connectSeparatedPoints" works properly
+  // even if the point is outside the visible range and only one series has a valid value for this point.
+  CanvasAssertions.assertLineDrawn(htx, xy1, xy2, attrs);
+
+  //Test if series2 is drawn correctly.
+  //------------------------------------
+  
+  // The sixth point of the second series
+  // Use the sixth and NOT the seventh point despite this series has eight points 
+  // since this series has no value at the seventh position.
+  var x1 = data[6][0];
+  var y1 = data[6][2];
+  var xy1 = g.toDomCoords(x1, y1);
+  
+  // The last point of the second series.
+  var x2 = data[8][0];
+  var y2 = data[8][2];
+  var xy2 = g.toDomCoords(x2, y2);
+  
+  // Check if both points are connected at the right edge of the canvas and if the option "connectSeparatedPoints" works properly
+  // even if the point is outside the visible range and only one series has a valid value for this point.
+  CanvasAssertions.assertLineDrawn(htx, xy1, xy2, attrs);
+
+    
+};
-- 
2.7.4