From dca562c8fccdbaf6927b88d20cc4197ddd5853d8 Mon Sep 17 00:00:00 2001
From: madhuka <madhukaudantha@gmail.com>
Date: Fri, 26 Jun 2015 16:41:45 +0530
Subject: [PATCH] Adding Google chart Facotry View model Test

---
 app/scripts/chart-factory/google-chart-factory.js | 2 +-
 test/spec/chart-factory/google-chart-factory.js   | 9 ++++++++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/app/scripts/chart-factory/google-chart-factory.js b/app/scripts/chart-factory/google-chart-factory.js
index d3347a6..5a0dd36 100644
--- a/app/scripts/chart-factory/google-chart-factory.js
+++ b/app/scripts/chart-factory/google-chart-factory.js
@@ -38,7 +38,7 @@ angular.module('apacheZeppelinGsocApp').factory('GoogleChartFactory', function(
     options: {
       'isStacked': 'true',
       'fill': 20,
-      'height': 300,
+      'height': 400,
       'displayExactValues': true,
       'vAxis': {
         'gridlines': {
diff --git a/test/spec/chart-factory/google-chart-factory.js b/test/spec/chart-factory/google-chart-factory.js
index 97f42ec..bbb7e78 100644
--- a/test/spec/chart-factory/google-chart-factory.js
+++ b/test/spec/chart-factory/google-chart-factory.js
@@ -36,12 +36,19 @@ describe("apacheZeppelinGsocApp GoogleChartFactory", function() {
       //changing the mockchart type
       myChart.setChartType('Bar');
       expect(myChart.type).toBe('Bar');
+      expect(myChart.viewModel.type).toBe('BarChart');
     });
   });
   describe("view model", function() {
     it("testing for attributes", function() {
       expect(myChart.viewModel.options).toBeDefined();
-      //to-Do more Testing
+      expect(myChart.viewModel.type).toBeDefined();
+      expect(myChart.viewModel.cssStyle).toBeDefined();
+      expect(myChart.viewModel.data).toBeDefined();
+      expect(myChart.viewModel.formatters).toBeDefined();
+    });
+    it("testing for deafults vales of attributes", function() {
+      expect(myChart.viewModel.options.height).toBe(300);
     });
   });
 });
\ No newline at end of file