Merge pull request #80 from Prakriti-nith/coverage2

Shekharrajak · web-flow · commit 9990f7d68614 · 2018-03-01T19:49:07.000+05:30
Improve coverage for googlecharts.rb file
diff --git a/spec/adapters/googlecharts_spec.rb b/spec/adapters/googlecharts_spec.rb
@@ -3,25 +3,55 @@
 describe Daru::View::Plot, 'plotting with googlecharts' do
   before { Daru::View.plotting_library = :googlecharts }
   before(:each) do
-    @data_array = [[1, 15], [2, 30], [4, 40]]
+    @data_array1 = [[1, 15], [2, 30], [4, 40]]
     @data_vec1 = Daru::Vector.new([1 ,2, 4])
     @data_vec2 = Daru::Vector.new([15 ,30, 40])
     @data_df = Daru::DataFrame.new(arr1: @data_vec1, arr2: @data_vec2)
-
     @options = {width: 800, height: 720}
-
     @plot = Daru::View::Plot.new(@data_df, @options)
+    @data_hash = {
+      cols: [
+              {id: 'Name', label: 'Name', type: 'string'},
+              {id: 'Salary', label: 'Salary', type: 'number'},
+              {type: 'boolean', label: 'Full Time Employee' },
+            ],
+      rows: [
+              {c:[{v: 'Mike'}, {v: 10000, f: '$10,000'}, {v: true}]},
+              {c:[{v: 'Jim'}, {v:8000,   f: '$8,000'}, {v: false}]},
+              {c:[{v: 'Alice'}, {v: 12500, f: '$12,500'}, {v: true}]},
+              {c:[{v: 'Bob'}, {v: 7000,  f: '$7,000'}, {v: true}]},
+            ]
+    }
+    @data_array2 = [
+      ['Galaxy', 'Distance', 'Brightness'],
+      ['Canis Major Dwarf', 8000, 230.3],
+      ['Sagittarius Dwarf', 24000, 4000.5],
+      ['Ursa Major II Dwarf', 30000, 1412.3],
+      ['Lg. Magellanic Cloud', 50000, 120.9],
+      ['Bootes I', 60000, 1223.1]
+    ]
+    @table_array = Daru::View::Table.new(@data_array2, @options)
+    @table_dv = Daru::View::Table.new(@data_vec1, @options)
+    @table_df = Daru::View::Table.new(@data_df, @options)
+    @table_hash = Daru::View::Table.new(@data_hash, @options)
   end
 
   describe "initialization Charts" do
     it "Default chart GoogleVisualr::Interactive::LineChart " do
-      expect(Daru::View::Plot.new.chart).to be_a GoogleVisualr::Interactive::LineChart
+      expect(Daru::View::Plot.new.chart)
+      .to be_a GoogleVisualr::Interactive::LineChart
     end
     it "Bar chart GoogleVisualr::Interactive::BarChart " do
-      expect(Daru::View::Plot.new([], type: :bar).chart).to be_a GoogleVisualr::Interactive::BarChart
+      expect(Daru::View::Plot.new(
+        [],
+        type: :bar).chart
+      ).to be_a GoogleVisualr::Interactive::BarChart
     end
     it "Column chart GoogleVisualr::Interactive::ColumnChart " do
-      expect(Daru::View::Plot.new([], type: :column).chart).to be_a GoogleVisualr::Interactive::ColumnChart
+      expect(Daru::View::Plot.new(
+        [],
+        type: :column).chart
+      ).to be_a GoogleVisualr::Interactive::ColumnChart
     end
     # TODO: all other kinds of charts
   end
@@ -30,5 +60,32 @@
     it "Table class must be GoogleVisualr::DataTable " do
       expect(Daru::View::Table.new.table).to be_a GoogleVisualr::DataTable
     end
+    it "Table class must be GoogleVisualr::DataTable when data objects are" \
+       " of class Daru::Vector" do
+      expect(@table_dv.table).to be_a GoogleVisualr::DataTable
+      expect(@table_dv.options).to eq @options
+      expect(@table_dv.data).to eq @data_vec1
+    end
+    it "Table class must be GoogleVisualr::DataTable when data objects are" \
+       " of class Daru::DataFrame" do
+      expect(@table_df.table).to be_a GoogleVisualr::DataTable
+      expect(@table_df.options).to eq @options
+      expect(@table_df.data).to eq @data_df
+    end
+    it "Table class must be GoogleVisualr::DataTable when data objects are" \
+       " of class Array" do
+      expect(@table_array.table).to be_a GoogleVisualr::DataTable
+      expect(@table_array.options).to eq @options
+      expect(@table_array.data).to eq @data_array2
+    end
+    it "Table class must be GoogleVisualr::DataTable when data objects are" \
+       " of class Hash" do
+      expect(@table_hash.table).to be_a GoogleVisualr::DataTable
+      expect(@table_hash.options).to eq @options
+      expect(@table_hash.data).to eq @data_hash
+    end
+    it "Raise error when data objects are none of the above" do
+      expect{Daru::View::Table.new("daru")}.to raise_error(ArgumentError) 
+    end
   end
 end
