Adding Google chart Facotry View model Test

Madhuka · Jun 26, 2015 · dca562c · dca562c
1 parent 1f30f80
commit dca562c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git 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
@@ -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);
     });
   });
 });