seer 0.4.0 → 0.5.0

data/CONTRIBUTORS

@@ -0,0 +1,4 @@
+Thanks go to the following individuals for their contributions:
+
+  Alexey Kuleshov (http://github.com/kulesa)
+  Harold Giménez (http://github.com/hgimenez)

data/README.rdoc

@@ -20,7 +20,7 @@ In your controller:
   
 In your view:
 
-  <div id="chart" class="chart"></div>
+  <div id="chart"></div>
 
   <%= Seer::visualize(
         @widgets, 
@@ -44,6 +44,8 @@ In your view:
 
 For examples of additional chart types, refer to the documentation for each of the individual chart objects, or see the blog post announcing Seer: {Simple, Semantic Graphing for Ruby on Rails with Seer}[http://www.idolhands.com/ruby-on-rails/gems-plugins-and-engines/graphing-for-ruby-on-rails-with-seer]
 
+A {sample project}[http://github.com/Bantik/seer_sample] that demonstrates each of the chart types is available on GitHub.
+
 Seer is developed and maintained by {Corey Ehmke}[http://www.idolhands.com/] at {SEO Logic}[http://www.seologic.com/].
 
 Copyright (c) 2010 Corey Ehmke / SEO Logic, released under the MIT license

data/Rakefile

@@ -12,6 +12,7 @@ begin
     gem.authors = ["Corey Ehmke / SEO Logic"]
     gem.add_development_dependency "rspec", ">= 1.2.9"
     gem.files = [
+       "CONTRIBUTORS",
        "init.rb",
        "lib/seer.rb",
        "lib/seer/area_chart.rb",

data/lib/seer/area_chart.rb

@@ -11,7 +11,7 @@ module Seer
   #
   # In your view:
   #
-  #   <div id="chart" class="chart"></div>
+  #   <div id="chart"></div>
   #
   #   <%= Seer::visualize(
   #         @data, 
@@ -66,7 +66,7 @@ module Seer
     end
   
     def data_columns #:nodoc:
-      _data_columns =  "            data.addRows(#{data_series.first.map{|d| d.send(data_label)}.uniq.size});\r"
+      _data_columns =  "            data.addRows(#{data_rows.size});\r"
       _data_columns << "            data.addColumn('string', 'Date');\r"
       data.each do |datum|
         _data_columns << "            data.addColumn('number', '#{datum.send(series_label)}');\r"
@@ -75,20 +75,26 @@ module Seer
     end
     
     def data_table #:nodoc:
-      _rows = data_series.first.map{|d| d.send(data_label)}.uniq
+      _rows = data_rows
       _rows.each_with_index do |r,i|
         @data_table << "            data.setCell(#{i}, 0,'#{r}');\r"
       end
       data_series.each_with_index do |column,i|
         column.each_with_index do |c,j|
-          @data_table << "data.setCell(#{j},#{i+1},#{c.send(data_method)});\r"
+          @data_table << "            data.setCell(#{j},#{i+1},#{c.send(data_method)});\r"
         end
       end
       @data_table
     end
+    
+    def data_rows
+      data_series.inject([]) do |rows, element|
+        rows |= element.map { |e| e.send(data_label) }
+      end
+    end
 
     def nonstring_options #:nodoc:
-      [ :axis_font_size, :colors, :enable_tooltip, :height, :legend_font_size, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width]
+      [ :axis_font_size, :colors, :enable_tooltip, :height, :is_stacked, :legend_font_size, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width]
     end
     
     def string_options #:nodoc:
@@ -107,7 +113,7 @@ module Seer
 #{data_table.to_s}
             var options = {};
 #{options}
-            var container = document.getElementById('chart');
+            var container = document.getElementById('#{self.chart_element}');
             var chart = new google.visualization.AreaChart(container);
             chart.draw(data, options);
           }

data/lib/seer/bar_chart.rb

@@ -9,7 +9,7 @@ module Seer
   #
   # In your view:
   #
-  #   <div id="chart" class="chart"></div>
+  #   <div id="chart"></div>
   #
   #   <%= Seer::visualize(
   #         @widgets, 
@@ -100,7 +100,7 @@ module Seer
 #{data_table.to_s}
             var options = {};
 #{options}
-            var container = document.getElementById('chart');
+            var container = document.getElementById('#{self.chart_element}');
             var chart = new google.visualization.BarChart(container);
             chart.draw(data, options);
           }

data/lib/seer/chart.rb

@@ -41,14 +41,16 @@ module Seer
     def options
       _options = ""
       nonstring_options.each do |opt|
+        next unless self.send(opt)
         if opt == :colors
-          _options << "            options['#{opt.to_s.camelize(:lower)}'] = #{self.send(:formatted_colors)};\r" if self.send(opt)
+          _options << "            options['#{opt.to_s.camelize(:lower)}'] = #{self.send(:formatted_colors)};\r"
         else
-          _options << "            options['#{opt.to_s.camelize(:lower)}'] = #{self.send(opt)};\r" if self.send(opt)
+          _options << "            options['#{opt.to_s.camelize(:lower)}'] = #{self.send(opt)};\r"
         end
       end
       string_options.each do |opt|
-        _options << "            options['#{opt.to_s.camelize(:lower)}'] = '#{self.send(opt)}';\r" if self.send(opt)
+        next unless self.send(opt)
+        _options << "            options['#{opt.to_s.camelize(:lower)}'] = '#{self.send(opt)}';\r"
       end
       _options
     end

data/lib/seer/column_chart.rb

@@ -9,7 +9,7 @@ module Seer
   #
   # In your view:
   #
-  #   <div id="chart" class="chart"></div>
+  #   <div id="chart"></div>
   #
   #   <%= Seer::visualize(
   #         @widgets, 
@@ -100,7 +100,7 @@ module Seer
 #{data_table.to_s}
             var options = {};
 #{options}
-            var container = document.getElementById('chart');
+            var container = document.getElementById('#{self.chart_element}');
             var chart = new google.visualization.ColumnChart(container);
             chart.draw(data, options);
           }

data/lib/seer/gauge.rb

@@ -9,7 +9,7 @@ module Seer
   #
   # In your view:
   #
-  #   <div id="chart" class="chart"></div>
+  #   <div id="chart"></div>
   #   
   #   <%= Seer::visualize(
   #         @data, 
@@ -92,7 +92,7 @@ module Seer
           function drawChart() {
             var data = new google.visualization.DataTable();
 #{data_columns}
-#{data_table.to_s}
+#{data_table.join}
             var options = {};
 #{options}
             var container = document.getElementById('#{self.chart_element}');

data/lib/seer/line_chart.rb

@@ -11,7 +11,7 @@ module Seer
   #
   # In your view:
   #
-  #   <div id="chart" class="chart"></div>
+  #   <div id="chart"></div>
   #
   #   <%= Seer::visualize(
   #         @data, 
@@ -66,7 +66,7 @@ module Seer
     end
   
     def data_columns  #:nodoc:
-      _data_columns =  "            data.addRows(#{data_series.first.map{|d| d.send(data_label)}.uniq.size});\r"
+      _data_columns =  "            data.addRows(#{data_rows.size});\r"
       _data_columns << "            data.addColumn('string', 'Date');\r"
       data.each do |datum|
         _data_columns << "            data.addColumn('number', '#{datum.send(series_label)}');\r"
@@ -75,7 +75,7 @@ module Seer
     end
     
     def data_table  #:nodoc:
-      _rows = data_series.first.map{|d| d.send(data_label)}.uniq
+      _rows = data_rows
       _rows.each_with_index do |r,i|
         @data_table << "            data.setCell(#{i}, 0,'#{r}');\r"
       end
@@ -86,6 +86,12 @@ module Seer
       end
       @data_table
     end
+    
+    def data_rows
+      data_series.inject([]) do |rows, element|
+        rows |= element.map { |e| e.send(data_label) }
+      end
+    end
 
     def nonstring_options  #:nodoc:
       [ :axis_font_size, :colors, :enable_tooltip, :height, :legend_font_size, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :smooth_line, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width]
@@ -107,7 +113,7 @@ module Seer
 #{data_table.to_s}
             var options = {};
 #{options}
-            var container = document.getElementById('chart');
+            var container = document.getElementById('#{self.chart_element}');
             var chart = new google.visualization.LineChart(container);
             chart.draw(data, options);
           }

data/lib/seer/pie_chart.rb

@@ -9,7 +9,7 @@ module Seer
   #
   # In your view:
   #
-  #   <div id="chart" class="chart"></div>
+  #   <div id="chart"></div>
   #
   #   <%= Seer::visualize(
   #         @data, 
@@ -92,7 +92,7 @@ module Seer
 #{data_table.to_s}
             var options = {};
 #{options}
-            var container = document.getElementById('chart');
+            var container = document.getElementById('#{self.chart_element}');
             var chart = new google.visualization.PieChart(container);
             chart.draw(data, options);
           }

data/spec/area_chart_spec.rb

@@ -14,34 +14,20 @@ describe "Seer::AreaChart" do
      )
   end
   
-  describe 'defaults' do
-  
-    it 'colors' do
-      @chart.colors.should == Seer::Chart::DEFAULT_COLORS
-    end
-
-    it 'legend' do
-      @chart.legend.should == Seer::Chart::DEFAULT_LEGEND_LOCATION
-    end
-    
-    it 'height' do
-      @chart.height.should == Seer::Chart::DEFAULT_HEIGHT
-    end
-    
-    it 'width' do
-      @chart.width.should == Seer::Chart::DEFAULT_WIDTH
-    end
-    
+  describe 'defaults' do  
+    it_should_behave_like 'it sets default values'
   end
 
   describe 'graph options' do
   
-    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :colors, :enable_tooltip, :focus_border_color, :height, :is_stacked, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :number, :tooltip_width, :width].each do |accessor|
+    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :enable_tooltip, :focus_border_color, :height, :is_stacked, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :number, :tooltip_width, :width].each do |accessor|
       it "sets its #{accessor} value" do
         @chart.send("#{accessor}=", 'foo')
         @chart.send(accessor).should == 'foo'
       end
     end
+    
+    it_should_behave_like 'it has colors attribute'   
   end
   
   it 'renders as JavaScript' do
@@ -50,24 +36,73 @@ describe "Seer::AreaChart" do
   end
   
   it 'sets its data columns' do
-    @chart.data_columns.should =~ /addRows\(3\)/
-    @chart.data_columns.should =~ /addColumn\('string', 'Date'\)/
-    @chart.data_columns.should =~ /addColumn\('string', 'Date'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '0'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '1'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '2'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '3'\)/
+    @chart.data_columns.should =~ /addRows\(5\)/
+    @chart.data_columns.should add_column('string', 'Date')
+    @chart.data_columns.should add_column('number', '0')
+    @chart.data_columns.should add_column('number', '1')
+    @chart.data_columns.should add_column('number', '2')
+    @chart.data_columns.should add_column('number', '3')
   end
   
   it 'sets its data table' do
-    @chart.data_table.to_s.should =~ /data\.setCell\(0, 0,'1'\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(1, 0,'2'\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(2, 0,'3'\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(0,1,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(2,1,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(0,2,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(1,2,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(2,2,8\)/
+    @chart.data_table.to_s.should set_cell(0, 0,'1')
+    @chart.data_table.to_s.should set_cell(1, 0,'2')
+    @chart.data_table.to_s.should set_cell(2, 0,'3')
+    @chart.data_table.to_s.should set_cell(3, 0,'4')
+    @chart.data_table.to_s.should set_cell(4, 0,'5')
+    @chart.data_table.to_s.should set_cell(0,1,8)
+    @chart.data_table.to_s.should set_cell(2,1,8)
+    @chart.data_table.to_s.should set_cell(0,2,8)
+    @chart.data_table.to_s.should set_cell(1,2,8)
+    @chart.data_table.to_s.should set_cell(2,2,8)
+  end
+  
+  describe 'when data_series is an array of arrays of arrays/hashes' do
+    before(:each) do 
+      data_series = Array.new(3) {|i| [[i, i+1], [i+1, i+2]]}
+      @chart = Seer::AreaChart.new(
+          :data => [0,1,2,3],
+          :series_label   => 'to_s',
+          :data_series => data_series,
+          :data_label => 'first',
+          :data_method => 'size',
+          :chart_options  => {},
+          :chart_element  => 'chart'
+       )
+     end
+
+    it 'calculates number of rows' do
+      @chart.data_columns.should =~ /addRows\(4\)/     
+    end
+    
+    it 'sets its data table' do 
+      @chart.data_table.to_s.should set_cell(0, 0,'0')
+      @chart.data_table.to_s.should set_cell(1, 0,'1')
+      @chart.data_table.to_s.should set_cell(2, 0,'2')
+      @chart.data_table.to_s.should set_cell(3, 0,'3')
+    end
   end
   
+  describe 'should receive options' do 
+    before(:each) do 
+      data_series = Array.new(3) {|i| [[i, i+1], [i+1, i+2]]}
+      @options = {
+          :data => [0,1,2,3],
+          :series_label   => 'to_s',
+          :data_series => data_series,
+          :data_label => 'first',
+          :data_method => 'size',
+          :chart_options  => {},
+          :chart_element  => 'chart'
+      }
+    end
+    
+    it 'should receive :is_stacked option' do 
+      create_chart_with_option(:is_stacked => true).to_js.should =~ /options\['isStacked'\] = true/
+      end
+  end
+end
+
+def create_chart_with_option(option)
+      Seer::AreaChart.new(@options.merge(option))
 end

data/spec/bar_chart_spec.rb

@@ -13,33 +13,19 @@ describe "Seer::BarChart" do
   end
   
   describe 'defaults' do
-  
-    it 'colors' do
-      @chart.colors.should == Seer::Chart::DEFAULT_COLORS
-    end
-
-    it 'legend' do
-      @chart.legend.should == Seer::Chart::DEFAULT_LEGEND_LOCATION
-    end
-    
-    it 'height' do
-      @chart.height.should == Seer::Chart::DEFAULT_HEIGHT
-    end
-    
-    it 'width' do
-      @chart.width.should == Seer::Chart::DEFAULT_WIDTH
-    end
-    
+    it_should_behave_like 'it sets default values'
   end
 
   describe 'graph options' do
   
-    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :colors, :enable_tooltip, :focus_border_color, :height, :is_3_d, :is_stacked, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :log_scale, :max, :min, :reverse_axis, :show_categories, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width].each do |accessor|
+    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :enable_tooltip, :focus_border_color, :height, :is_3_d, :is_stacked, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :log_scale, :max, :min, :reverse_axis, :show_categories, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width].each do |accessor|
       it "sets its #{accessor} value" do
         @chart.send("#{accessor}=", 'foo')
         @chart.send(accessor).should == 'foo'
       end
     end
+    
+    it_should_behave_like 'it has colors attribute'
   end
   
   it 'renders as JavaScript' do
@@ -52,14 +38,14 @@ describe "Seer::BarChart" do
   end
   
   it 'sets its data table' do
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 0,'0'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 0,'1'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 0,'2'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 0,'3'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 1, 8\)/
+    @chart.data_table.to_s.should set_value(0, 0,'0')
+    @chart.data_table.to_s.should set_value(0, 1, 8)
+    @chart.data_table.to_s.should set_value(1, 0,'1')
+    @chart.data_table.to_s.should set_value(1, 1, 8)
+    @chart.data_table.to_s.should set_value(2, 0,'2')
+    @chart.data_table.to_s.should set_value(2, 1, 8)
+    @chart.data_table.to_s.should set_value(3, 0,'3')
+    @chart.data_table.to_s.should set_value(3, 1, 8)
   end
   
 end

data/spec/chart_spec.rb

@@ -46,8 +46,7 @@ describe "Seer::Chart" do
   end
   
   it 'sets its data columns' do
-    @chart.data_columns.should =~ /addRows\(3\)/
-    @chart.data_columns.should =~ /addColumn\('string', 'Date'\)/
+    @chart.data_columns.should =~ /addRows\(5\)/
     @chart.data_columns.should =~ /addColumn\('string', 'Date'\)/
     @chart.data_columns.should =~ /addColumn\('number', '0'\)/
     @chart.data_columns.should =~ /addColumn\('number', '1'\)/
@@ -56,7 +55,7 @@ describe "Seer::Chart" do
   end
   
   it 'sets its options' do
-    puts @chart.options.should =~ /options\['titleX'\] = 'Something'/
+    @chart.options.should =~ /options\['titleX'\] = 'Something'/
   end
   
   

data/spec/column_chart_spec.rb

@@ -12,34 +12,20 @@ describe "Seer::ColumnChart" do
      )
   end
   
-  describe 'defaults' do
-  
-    it 'colors' do
-      @chart.colors.should == Seer::Chart::DEFAULT_COLORS
-    end
-
-    it 'legend' do
-      @chart.legend.should == Seer::Chart::DEFAULT_LEGEND_LOCATION
-    end
-    
-    it 'height' do
-      @chart.height.should == Seer::Chart::DEFAULT_HEIGHT
-    end
-    
-    it 'width' do
-      @chart.width.should == Seer::Chart::DEFAULT_WIDTH
-    end
-    
+  describe 'defaults' do  
+    it_should_behave_like 'it sets default values'
   end
 
   describe 'graph options' do
   
-    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :colors, :enable_tooltip, :focus_border_color, :height, :is_3_d, :is_stacked, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :log_scale, :max, :min, :reverse_axis, :show_categories, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width].each do |accessor|
+    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :enable_tooltip, :focus_border_color, :height, :is_3_d, :is_stacked, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :log_scale, :max, :min, :reverse_axis, :show_categories, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width].each do |accessor|
       it "sets its #{accessor} value" do
         @chart.send("#{accessor}=", 'foo')
         @chart.send(accessor).should == 'foo'
       end
     end
+    
+    it_should_behave_like 'it has colors attribute'   
   end
   
   it 'renders as JavaScript' do
@@ -52,14 +38,14 @@ describe "Seer::ColumnChart" do
   end
   
   it 'sets its data table' do
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 0,'0'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 0,'1'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 0,'2'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 0,'3'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 1, 8\)/
+    @chart.data_table.to_s.should set_value(0, 0,'0')
+    @chart.data_table.to_s.should set_value(0, 1, 8)
+    @chart.data_table.to_s.should set_value(1, 0,'1')
+    @chart.data_table.to_s.should set_value(1, 1, 8)
+    @chart.data_table.to_s.should set_value(2, 0,'2')
+    @chart.data_table.to_s.should set_value(2, 1, 8)
+    @chart.data_table.to_s.should set_value(3, 0,'3')
+    @chart.data_table.to_s.should set_value(3, 1, 8)
   end
   
 end

data/spec/custom_matchers.rb

@@ -0,0 +1,23 @@
+module CustomMatcher  
+  def set_cell(row, column, value)
+    value = "'#{value}'" if value.is_a?(String)
+    
+    simple_matcher("setCell(#{row}, #{column}, #{value})") do |actual|
+      actual =~ /data\.setCell\(#{row},\s*#{column},\s*#{value}\)/
+    end
+  end  
+  
+  def set_value(row, column, value)
+    value = "'#{value}'" if value.is_a?(String)
+    
+    simple_matcher("setValue(#{row}, #{column}, #{value})") do |actual|
+      actual =~ /data\.setValue\(#{row},\s*#{column},\s*#{value}\)/
+    end
+  end  
+  
+  def add_column(column_type, value)
+    simple_matcher("addColumn('#{column_type}', '#{value}')") do |actual|
+      actual =~ /data\.addColumn\('#{column_type}',\s*'#{value}'\)/
+    end
+  end
+end

data/spec/gauge_spec.rb

@@ -32,6 +32,8 @@ describe "Seer::Gauge" do
         @chart.send(accessor).should == 'foo'
       end
     end
+    
+      it_should_behave_like 'it has colors attribute'
   end
   
   it 'renders as JavaScript' do
@@ -44,14 +46,14 @@ describe "Seer::Gauge" do
   end
   
   it 'sets its data table' do
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 0,'0'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 0,'1'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 0,'2'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 0,'3'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 1, 8\)/
+    @chart.data_table.to_s.should set_value(0, 0,'0')
+    @chart.data_table.to_s.should set_value(0, 1, 8)
+    @chart.data_table.to_s.should set_value(1, 0,'1')
+    @chart.data_table.to_s.should set_value(1, 1, 8)
+    @chart.data_table.to_s.should set_value(2, 0,'2')
+    @chart.data_table.to_s.should set_value(2, 1, 8)
+    @chart.data_table.to_s.should set_value(3, 0,'3')
+    @chart.data_table.to_s.should set_value(3, 1, 8)
   end
   
 end

data/spec/helpers.rb

@@ -0,0 +1,37 @@
+describe "it has colors attribute", :shared => true do   
+  it 'sets its colors value' do
+    color_list =  ['#7e7587','#990000','#009900', '#3e5643', '#660000', '#003300']  
+    @chart.colors = color_list
+    @chart.colors.should == color_list
+  end
+  
+  it 'colors should be an array' do 
+    lambda {
+      @chart.colors = '#000000'
+    }.should raise_error(ArgumentError)
+  end
+
+  it 'colors should be valid hex values' do 
+    lambda {
+      @chart.colors = ['#000000', 'NOTHEX']
+    }.should raise_error(ArgumentError)
+  end
+end
+
+describe "it sets default values", :shared => true do
+  it 'colors' do
+    @chart.colors.should == Seer::Chart::DEFAULT_COLORS
+  end
+
+  it 'legend' do
+    @chart.legend.should == Seer::Chart::DEFAULT_LEGEND_LOCATION
+  end
+
+  it 'height' do
+    @chart.height.should == Seer::Chart::DEFAULT_HEIGHT
+  end
+
+  it 'width' do
+    @chart.width.should == Seer::Chart::DEFAULT_WIDTH
+  end
+end

data/spec/line_chart_spec.rb

@@ -14,34 +14,20 @@ describe "Seer::LineChart" do
      )
   end
   
-  describe 'defaults' do
-  
-    it 'colors' do
-      @chart.colors.should == Seer::Chart::DEFAULT_COLORS
-    end
-
-    it 'legend' do
-      @chart.legend.should == Seer::Chart::DEFAULT_LEGEND_LOCATION
-    end
-    
-    it 'height' do
-      @chart.height.should == Seer::Chart::DEFAULT_HEIGHT
-    end
-    
-    it 'width' do
-      @chart.width.should == Seer::Chart::DEFAULT_WIDTH
-    end
-    
+  describe 'defaults' do  
+    it_should_behave_like 'it sets default values'
   end
 
   describe 'graph options' do
   
-    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :colors, :enable_tooltip, :focus_border_color, :height, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :smooth_line, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :number, :tooltip_width, :width].each do |accessor|
+    [:axis_color, :axis_background_color, :axis_font_size, :background_color, :border_color, :enable_tooltip, :focus_border_color, :height, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :line_size, :log_scale, :max, :min, :point_size, :reverse_axis, :show_categories, :smooth_line, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :number, :tooltip_width, :width].each do |accessor|
       it "sets its #{accessor} value" do
         @chart.send("#{accessor}=", 'foo')
         @chart.send(accessor).should == 'foo'
       end
     end
+    
+    it_should_behave_like 'it has colors attribute'
   end
   
   it 'renders as JavaScript' do
@@ -50,24 +36,51 @@ describe "Seer::LineChart" do
   end
   
   it 'sets its data columns' do
-    @chart.data_columns.should =~ /addRows\(3\)/
-    @chart.data_columns.should =~ /addColumn\('string', 'Date'\)/
-    @chart.data_columns.should =~ /addColumn\('string', 'Date'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '0'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '1'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '2'\)/
-    @chart.data_columns.should =~ /addColumn\('number', '3'\)/
+    @chart.data_columns.should =~ /addRows\(5\)/
+    @chart.data_columns.should add_column('string', 'Date')
+    @chart.data_columns.should add_column('number', '0')
+    @chart.data_columns.should add_column('number', '1')
+    @chart.data_columns.should add_column('number', '2')
+    @chart.data_columns.should add_column('number', '3')
   end
   
   it 'sets its data table' do
-    @chart.data_table.to_s.should =~ /data\.setCell\(0, 0,'1'\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(1, 0,'2'\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(2, 0,'3'\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(0,1,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(2,1,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(0,2,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(1,2,8\)/
-    @chart.data_table.to_s.should =~ /data\.setCell\(2,2,8\)/
+    @chart.data_table.to_s.should set_cell(0, 0,'1')
+    @chart.data_table.to_s.should set_cell(1, 0,'2')
+    @chart.data_table.to_s.should set_cell(2, 0,'3')
+    @chart.data_table.to_s.should set_cell(3, 0,'4')
+    @chart.data_table.to_s.should set_cell(4, 0,'5')
+    @chart.data_table.to_s.should set_cell(0,1,8)
+    @chart.data_table.to_s.should set_cell(2,1,8)
+    @chart.data_table.to_s.should set_cell(0,2,8)
+    @chart.data_table.to_s.should set_cell(1,2,8)
+    @chart.data_table.to_s.should set_cell(2,2,8)
+  end
+  
+  describe 'when data_series is an array of arrays of arrays/hashes' do
+    before(:each) do 
+      data_series = Array.new(3) {|i| [[i, i+1], [i+1, i+2]]}
+      @chart = Seer::LineChart.new(
+          :data => [0,1,2,3],
+          :series_label   => 'to_s',
+          :data_series => data_series,
+          :data_label => 'first',
+          :data_method => 'size',
+          :chart_options  => {},
+          :chart_element  => 'chart'
+       )
+     end
+
+    it 'calculates number of rows' do
+      @chart.data_columns.should =~ /addRows\(4\)/     
+    end
+    
+    it 'sets its data table' do 
+      @chart.data_table.to_s.should set_cell(0, 0,'0')
+      @chart.data_table.to_s.should set_cell(1, 0,'1')
+      @chart.data_table.to_s.should set_cell(2, 0,'2')
+      @chart.data_table.to_s.should set_cell(3, 0,'3')
+    end
   end
   
 end

data/spec/pie_chart_spec.rb

@@ -12,34 +12,20 @@ describe "Seer::PieChart" do
      )
   end
   
-  describe 'defaults' do
-  
-    it 'colors' do
-      @chart.colors.should == Seer::Chart::DEFAULT_COLORS
-    end
-
-    it 'legend' do
-      @chart.legend.should == Seer::Chart::DEFAULT_LEGEND_LOCATION
-    end
-    
-    it 'height' do
-      @chart.height.should == Seer::Chart::DEFAULT_HEIGHT
-    end
-    
-    it 'width' do
-      @chart.width.should == Seer::Chart::DEFAULT_WIDTH
-    end
-    
+  describe 'defaults' do  
+    it_should_behave_like 'it sets default values'
   end
 
   describe 'graph options' do
   
-    [:background_color, :border_color, :colors, :enable_tooltip, :focus_border_color, :height, :is_3_d, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :pie_join_angle, :pie_minimal_angle, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width].each do |accessor|
+    [:background_color, :border_color, :enable_tooltip, :focus_border_color, :height, :is_3_d, :legend, :legend_background_color, :legend_font_size, :legend_text_color, :pie_join_angle, :pie_minimal_angle, :title, :title_x, :title_y, :title_color, :title_font_size, :tooltip_font_size, :tooltip_height, :tooltip_width, :width].each do |accessor|
       it "sets its #{accessor} value" do
         @chart.send("#{accessor}=", 'foo')
         @chart.send(accessor).should == 'foo'
       end
     end
+    
+    it_should_behave_like 'it has colors attribute'   
   end
   
   it 'renders as JavaScript' do
@@ -52,14 +38,14 @@ describe "Seer::PieChart" do
   end
   
   it 'sets its data table' do
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 0,'0'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(0, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 0,'1'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(1, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 0,'2'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(2, 1, 8\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 0,'3'\)/
-    @chart.data_table.to_s.should =~ /data\.setValue\(3, 1, 8\)/
+    @chart.data_table.to_s.should set_value(0, 0,'0')
+    @chart.data_table.to_s.should set_value(0, 1, 8)
+    @chart.data_table.to_s.should set_value(1, 0,'1')
+    @chart.data_table.to_s.should set_value(1, 1, 8)
+    @chart.data_table.to_s.should set_value(2, 0,'2')
+    @chart.data_table.to_s.should set_value(2, 1, 8)
+    @chart.data_table.to_s.should set_value(3, 0,'3')
+    @chart.data_table.to_s.should set_value(3, 1, 8)
   end
   
 end

data/spec/seer_spec.rb

@@ -68,8 +68,8 @@ describe "Seer" do
         :as => :bar_chart,
         :in_element => 'chart',
         :series => {
-          :label => 'to_s',
-          :data => 'size'
+          :series_label => 'to_s',
+          :data_method => 'size'
         },
         :chart_options => {}
       ) =~ /barchart/).should be_true
@@ -81,8 +81,8 @@ describe "Seer" do
         :as => :column_chart,
         :in_element => 'chart',
         :series => {
-          :label => 'to_s',
-          :data => 'size'
+          :series_label => 'to_s',
+          :data_method => 'size'
         },
         :chart_options => {}
       ) =~ /columnchart/).should be_true
@@ -94,8 +94,8 @@ describe "Seer" do
         :as => :gauge,
         :in_element => 'chart',
         :series => {
-          :label => 'to_s',
-          :data => 'size'
+          :series_label => 'to_s',
+          :data_method => 'size'
         },
         :chart_options => {}
       ) =~ /gauge/).should be_true

data/spec/spec_helper.rb

@@ -1,11 +1,13 @@
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+require 'rubygems'
 require 'actionpack'
-require 'activesupport'
+require 'active_support'
 require 'spec'
 require 'spec/autorun'
 require 'seer'
-
+require File.dirname(__FILE__) + "/custom_matchers"  
+require File.dirname(__FILE__) + '/helpers'
 Spec::Runner.configure do |config|
-
+  config.include(CustomMatcher)  
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: seer
 version: !ruby/object:Gem::Version 
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors: 
 - Corey Ehmke / SEO Logic
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-02-17 00:00:00 -06:00
+date: 2010-03-03 00:00:00 -06:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -32,6 +32,7 @@ extra_rdoc_files:
 - LICENSE
 - README.rdoc
 files: 
+- CONTRIBUTORS
 - LICENSE
 - README.rdoc
 - Rakefile
@@ -80,7 +81,9 @@ test_files:
 - spec/bar_chart_spec.rb
 - spec/chart_spec.rb
 - spec/column_chart_spec.rb
+- spec/custom_matchers.rb
 - spec/gauge_spec.rb
+- spec/helpers.rb
 - spec/line_chart_spec.rb
 - spec/pie_chart_spec.rb
 - spec/seer_spec.rb