X-Git-Url: https://adrianiainlam.tk/git/?a=blobdiff_plain;f=auto_tests%2Ftests%2Fsimple_drawing.js;h=a4acd2830cf619ea452ebc220b10d40a922380a7;hb=45df6dd0902ef39691f061a70f4702e606fb747e;hp=6bafa0dff7129d4341f8041b0a2ed14c19c7b9ea;hpb=dc2e4ba6e9ba661563f1c42a64cc5be1db362e74;p=dygraphs.git

diff --git a/auto_tests/tests/simple_drawing.js b/auto_tests/tests/simple_drawing.js
index 6bafa0d..a4acd28 100644
--- a/auto_tests/tests/simple_drawing.js
+++ b/auto_tests/tests/simple_drawing.js
@@ -58,6 +58,27 @@ SimpleDrawingTestCase.prototype.testDrawSimpleRangePlusOne = function() {
   CanvasAssertions.assertBalancedSaveRestore(htx);
 };
 
+// See http://code.google.com/p/dygraphs/issues/detail?id=185
+SimpleDrawingTestCase.prototype.testDrawSimpleRangeZeroToFifty = function() {
+  var opts = {
+    drawXGrid: false,
+    drawYGrid: false,
+    drawXAxis: false,
+    drawYAxis: false,
+    valueRange: [0,50] }
+
+  var graph = document.getElementById("graph");
+  var g = new Dygraph(graph, ZERO_TO_FIFTY, opts);
+  var htx = g.hidden_ctx_;
+
+  var lines = CanvasAssertions.getLinesDrawn(htx, {
+    strokeStyle: "#008080",
+    lineWidth: 1
+  });
+  assertEquals(1, lines.length);
+  CanvasAssertions.assertBalancedSaveRestore(htx);
+};
+
 SimpleDrawingTestCase.prototype.testDrawWithAxis = function() {
   var graph = document.getElementById("graph");
   var g = new Dygraph(graph, ZERO_TO_FIFTY);
@@ -87,79 +108,43 @@ SimpleDrawingTestCase.prototype.testDrawSimpleDash = function() {
   var g = new Dygraph(graph, [[1, 4], [2, 5], [3, 3], [4, 7], [5, 9]], opts);
   htx = g.hidden_ctx_;
 
-  assertEquals(29, CanvasAssertions.numLinesDrawn(htx, "#ff0000"));
+  // TODO(danvk): figure out a good way to restore this test.
+  // assertEquals(29, CanvasAssertions.numLinesDrawn(htx, "#ff0000"));
   CanvasAssertions.assertBalancedSaveRestore(htx);
 };
 
-SimpleDrawingTestCase.prototype.testSeparatedPointsDontDraw = function() {
-  var graph = document.getElementById("graph");
-  var g = new Dygraph(
-      graph,
-      [[1, 10, 11],
-       [2, 11, null],
-       [3, 12, 13]],
-      { colors: ['red', 'blue']});
-  var htx = g.hidden_ctx_;
-  assertEquals(2, CanvasAssertions.numLinesDrawn(htx, '#ff0000'));
-  assertEquals(0, CanvasAssertions.numLinesDrawn(htx, '#0000ff'));
-}
+/**
+ * Tests that thick lines are drawn continuously.
+ * Regression test for http://code.google.com/p/dygraphs/issues/detail?id=328
+ */
+SimpleDrawingTestCase.prototype.testDrawThickLine = function() {
+  var opts = {
+    drawXGrid: false,
+    drawYGrid: false,
+    drawXAxis: false,
+    drawYAxis: false,
+    strokeWidth: 15,
+    colors: ['#ff0000']
+  };
 
-SimpleDrawingTestCase.prototype.testSeparatedPointsDontDraw_expanded = function() {
   var graph = document.getElementById("graph");
-  var g = new Dygraph(
-      graph,
-      [[0, 10],
-       [1, 11],
-       [2, null],
-       [3, 13],
-       [4, 14]],
-      { colors: ['blue']});
-  var htx = g.hidden_ctx_;
-  /*
-  var num_lines = 0;
-  var lines = CanvasAssertions.getLinesDrawn(htx);
-  for (var idx = 0; idx < lines.length; idx++) {
-    var line = lines[idx];
-    var color = line[1].properties.strokeStyle;
-    if (color === "#ff0000" || color === "#0000ff") {
-      console.log(line[0].args, line[1].args, color);
-    }
-  }
-  */
+  // Set the dims so we pass if default changes.
+  graph.style.width='480px';
+  graph.style.height='320px';
+  var g = new Dygraph(graph, [[1, 2], [2, 5], [3, 2], [4, 7], [5, 0]], opts);
+  htx = g.hidden_ctx_;
 
-  assertEquals(2, CanvasAssertions.numLinesDrawn(htx, '#0000ff'));
-  CanvasAssertions.assertLineDrawn(htx, [56, 275], [161, 212],
-      { strokeStyle: '#0000ff', });
-  CanvasAssertions.assertLineDrawn(htx, [370, 87], [475, 25],
-      { strokeStyle: '#0000ff', });
-}
+  // There's a big gap in the line at (2, 5)
+  // If the bug is fixed, then there should be some red going up from here.
+  var xy = g.toDomCoords(2, 5);
+  var x = Math.round(xy[0]), y = Math.round(xy[1]);
 
-SimpleDrawingTestCase.prototype.testSeparatedPointsDontDraw_expanded_connected = function() {
-  var graph = document.getElementById("graph");
-  var g = new Dygraph(
-      graph,
-      [[0, 10],
-       [1, 11],
-       [2, null],
-       [3, 13],
-       [4, 14]],
-      { colors: ['blue'], connectSeparatedPoints: true});
-  var htx = g.hidden_ctx_;
-  var num_lines = 0;
-  var lines = CanvasAssertions.getLinesDrawn(htx);
-  for (var idx = 0; idx < lines.length; idx++) {
-    var line = lines[idx];
-    var color = line[1].properties.strokeStyle;
-    if (color === "#ff0000" || color === "#0000ff") {
-      console.log(line[0].args, line[1].args, color);
-    }
-  }
+  var sampler = new PixelSampler(g);
+  assertEquals([255,0,0,255], sampler.colorAtPixel(x, y));
+  assertEquals([255,0,0,255], sampler.colorAtPixel(x, y - 1));
+  assertEquals([255,0,0,255], sampler.colorAtPixel(x, y - 2));
 
-  assertEquals(3, CanvasAssertions.numLinesDrawn(htx, '#0000ff'));
-  CanvasAssertions.assertLineDrawn(htx, [56, 275], [161, 212],
-      { strokeStyle: '#0000ff', });
-  CanvasAssertions.assertLineDrawn(htx, [161, 212], [370, 87],
-      { strokeStyle: '#0000ff', });
-  CanvasAssertions.assertLineDrawn(htx, [370, 87], [475, 25],
-      { strokeStyle: '#0000ff', });
-}
+  // TODO(danvk): figure out a good way to restore this test.
+  // assertEquals(29, CanvasAssertions.numLinesDrawn(htx, "#ff0000"));
+  CanvasAssertions.assertBalancedSaveRestore(htx);
+};