Pulling out logic into the Table class.

lib/autograph/autoperf.rb

@@ -28,8 +28,7 @@ def generate_graphs(reports, configuration)
     end
 
     reports.keys.each_with_index do |key, index|
-      max = reports[key].column('conn/s').map{|x| x.to_i}.max.to_i
-      graphs[:max_request_rate].series << GraphSeries.new([index], [max], "Max Request Rate for '#{key}'")
+      graphs[:max_request_rate].series << GraphSeries.new([index], [reports[key].max('conn/s')], "Max Request Rate for '#{key}'")
     end
 
     graphs

lib/autograph/table.rb

@@ -10,6 +10,10 @@ def column(name)
     @rows.map{|r| r[name.to_s.to_sym]}
   end
 
+  def max(name)
+    @rows.map{|r| r[name.to_s.to_sym].to_i}.max
+  end
+
   def <<(row_hash)
     # Symbolize keys on the way in
     @rows << row_hash.inject({}){|x,y| x[y[0].to_s.to_sym] = y[1]; x}

test/test_table.rb

@@ -43,6 +43,16 @@ def test_column_with_string_key
     assert_equal t.column("column_b"), [2,222]
   end
 
+  def test_max
+    zero  = {:column_b => 1}
+    one   = {:column_b => 2}
+    two   = {'column_b' => 222}
+    t = Table.new(COLUMN_NAMES)
+    t << one
+    t << two
+    assert_equal t.max("column_b"), 222
+  end
+
   def test_to_html_returns_string
     assert_equal Table.new(COLUMN_NAMES).to_html.class, String
   end