Changeset 1291

Timestamp:

03/23/08 19:39:32 (8 months ago)

Author:

sandal

Message:

Fix for #387

Files:

• ruport/trunk/lib/ruport/controller.rb (modified) (1 diff)
• ruport/trunk/lib/ruport/formatter/pdf.rb (modified) (1 diff)
• ruport/trunk/test/pdf_formatter_test.rb (modified) (2 diffs)
• ruport/trunk/test/renderer_test.rb (modified) (2 diffs)

ruport/trunk/lib/ruport/controller.rb

@@ -480 +480 @@
-
-    finalize self.class.final_stage if self.class.final_stage
+    maybe :finalize
-  end  
-  

ruport/trunk/lib/ruport/formatter/pdf.rb

@@ -383 +383 @@
-          text_opts[:angle] || 0)
-        move_cursor_to(ypos)
+      end   
+      
+      def finalize
+        render_pdf
-      end
-    end   

ruport/trunk/test/pdf_formatter_test.rb

@@ -200 +200 @@
-     assert_equal({ :justification => :center }, 
-                  f.options[:table_format][:column_options][:heading])      
-
+        
-    
-end    
@@ -327 +327 @@
-    assert_equal 100, a.cursor
-  end
+end  
+  
+class SimpleController < Ruport::Controller
+  stage :foo
+  
+  class PDF < Ruport::Formatter::PDF
+    renders :pdf, :for => SimpleController
+    
+    build :foo do
+      add_text "Blah"
+    end
+  end
+end                                
+
+class TestPDFFinalize < Test::Unit::TestCase
+
+  context "When rendering a PDF" do    
+    def specify_finalize_should_be_called
+      SimpleController.render_pdf do |r|
+        r.formatter.expects(:render_pdf)
+      end 
+    end
+  end
+  
-end
+    

ruport/trunk/test/renderer_test.rb

@@ -85 +85 @@
-  renders :bin, :for => VanillaController
-  save_as_binary_file
-
-
+
+
+
+
+
+
+
+
+
-
-class TestController < Test::Unit::TestCase
@@ -97 +104 @@
-    actual = OldSchoolController.render(:text)
-    assert_equal "header\nbody\nfooter\n", actual
-
+
+
+
+
+
+
+
+
-  
-  context "when using templates" do