@@ -37,7 +37,14 @@ module Prawn # # end # + # Will raise <tt>Prawn::Errors::EmptyTable</tt> given + # a nil or empty <tt>data</tt> paramater. + # def table(data,options={}) + if data.nil? || data.empty? + raise Prawn::Errors::EmptyTable, + "data must be a non-empty, non-nil, two dimensional array of Prawn::Cells or strings" + end Prawn::Document::Table.new(data,self,options).draw end lib/prawn/document/table.rb @@ -39,6 +39,10 @@ module Prawn # This error is raised when table data is malformed # class InvalidTableData < StandardError; end + + # This error is raised when an empty or nil table is rendered + # + class EmptyTable < StandardError; end end end lib/prawn/errors.rb @@ -109,6 +109,22 @@ describe "A table's content" do @pdf.table(data) }.should.not.raise end + + it "should raise an EmptyTableError with an empty table" do + lambda { + data = [] + @pdf = Prawn::Document.new + @pdf.table(data) + }.should.raise( Prawn::Errors::EmptyTable ) + end + + it "should raise an EmptyTableError with a nil table" do + lambda { + data = nil + @pdf = Prawn::Document.new + @pdf.table(data) + }.should.raise( Prawn::Errors::EmptyTable ) + end it "should paginate for large tables" do # 30 rows fit on the table with default setting, 31 exceed. spec/table_spec.rb 286077da66c66c64e95380e5dce4f5cf5d6dea2f Gavin Stark g.stark@realdigitalmedia.com http://github.com/sandal/prawn/commit/1c554c949357c3bd35160b1816849f7789fef9a9 1c554c949357c3bd35160b1816849f7789fef9a9 2008-11-06T19:59:32-08:00 2008-11-06T19:59:32-08:00 [#108 state:resolved] Added specs for testing for empty and nil table data. Now raises Prawn::Errors::EmptyTable 0c457e7a2e36f03b67f9f130209441158f2f641f Gavin Stark g.stark@realdigitalmedia.com