From 8473667eb747d6018b9c917c886d8727d37a3f84 Mon Sep 17 00:00:00 2001
From: Rafael Reggiani Manzo <rr.manzo@gmail.com>
Date: Fri, 19 Jun 2015 13:10:50 -0300
Subject: [PATCH] Module.Graphic JS covered with unit tests

---
 spec/javascripts/module/graphic_spec.js.coffee | 105 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------
 1 file changed, 89 insertions(+), 16 deletions(-)

diff --git a/spec/javascripts/module/graphic_spec.js.coffee b/spec/javascripts/module/graphic_spec.js.coffee
index f1b675a..5a3e79e 100644
--- a/spec/javascripts/module/graphic_spec.js.coffee
+++ b/spec/javascripts/module/graphic_spec.js.coffee
@@ -1,38 +1,111 @@
 #= require spec_helper
+#= require Chart
 #= require module/graphic
 
 describe "Module.Graphic", ->
-  describe "constructor", ->
-    before ->
-      @container = 'container404829'
-      @metric_name = 'Pain'
-      @module_id = '54405'
+  before ->
+    @container = 'container404829'
+    @metric_name = 'Pain'
+    @module_id = '54405'
 
-      @drawer = sinon.stub()
-      sinon.stub(window, "$")
-      $.withArgs("tr#"+@container).returns(@drawer)
+    sinon.stub(window, "$")
 
+    @drawer = sinon.stub()
+    @drawer.is = sinon.stub().withArgs(':hidden').returns(false)
+    @drawer.slideUp = sinon.stub()
+    $.withArgs("tr#"+@container).returns(@drawer)
+
+  describe "constructor", ->
     context 'when the drawer is hidden', ->
       before ->
         @drawer.is = sinon.stub().withArgs(':hidden').returns(true)
+        @drawer.slideDown = sinon.stub()
+        sinon.stub(Module.Graphic.prototype, 'load')
 
       it "should show the drawer and start to load a graphic", ->
-        @drawer.slideDown = sinon.spy()
-
-        Module.Graphic.prototype.load = sinon.spy()
-
         @graphic = new Module.Graphic(@container, @metric_name, @module_id)
 
         assert.isTrue(@drawer.slideDown.calledOnce)
         assert.isTrue(@graphic.load.calledOnce)
 
-    context 'when the drawer is visible', ->
-      before ->
+      after ->
         @drawer.is = sinon.stub().withArgs(':hidden').returns(false)
+        @drawer.slideDown = undefined
+        Module.Graphic.prototype.load.restore()
 
+    context 'when the drawer is visible', ->
       it 'should hide the drawer', ->
-        @drawer.slideUp = sinon.spy()
-
         @graphic = new Module.Graphic(@container, @metric_name, @module_id)
 
         assert.isTrue(@drawer.slideUp.calledOnce)
+
+      after ->
+
+  describe 'load', ->
+    before ->
+      @graphic = new Module.Graphic(@container, @metric_name, @module_id)
+
+    it 'should make a POST request', ->
+      $.post = sinon.stub().withArgs('/modules/' + @module_id + '/metric_history', {metric_name: @metric_name, container: @container})
+
+      @graphic.load()
+
+      sinon.assert.calledOnce($.post)
+
+  describe 'display', ->
+    beforeEach ->
+      canvas_context = sinon.stub()
+      @canvas = sinon.stub()
+      @canvas.getContext = sinon.stub().withArgs('2d').returns(canvas_context)
+
+      elements = sinon.stub()
+      elements.get = sinon.stub().withArgs(0).returns(@canvas)
+      $.withArgs('canvas#'+@container).returns(elements)
+
+      @dates = ["2015-06-17", "2015-06-18", "2015-06-19"]
+      @values = [1.2, 2.3, 3.4]
+
+      opts = {
+        bezierCurve: false,
+        responsive: true,
+        maintainAspectRatio: false
+      }
+
+      data = {
+        labels : @dates,
+        datasets : [
+          {
+            fillColor : "rgba(220,220,220,0.5)",
+            strokeColor : "rgba(220,220,220,1)",
+            pointColor : "rgba(220,220,220,1)",
+            pointStrokeColor : "#000",
+            data : @values
+          }
+        ]
+      }
+
+      @line = sinon.stub().withArgs(data, opts)
+      sinon.stub(window, 'Chart').withArgs(canvas_context).returns({Line: @line})
+
+    context 'when the graphic exists', ->
+      it 'should render the chart', ->
+        Module.Graphic.display(@dates, @values, @container)
+        sinon.assert.calledOnce(@line)
+
+    context 'when the graphic does not exists', ->
+      beforeEach ->
+        @chart = sinon.stub()
+        @chart.destroy = sinon.stub()
+        @canvas.chart = @chart
+        @canvas.hasOwnProperty = sinon.stub().withArgs("chart").returns(true)
+
+      it 'should destroy the previous chart and render a new one', ->
+        Module.Graphic.display(@dates, @values, @container)
+        sinon.assert.calledOnce(@chart.destroy)
+        sinon.assert.calledOnce(@line)
+
+    afterEach ->
+      Chart.restore()
+
+  after ->
+    $.restore()
--
libgit2 0.21.2