Merge pull request #8 from chaslemley/master

Ignore trailing whitespace in templates

Author: bsiggelkow

lib/jsonify-rails/jsonify_builder.rb

@@ -10,7 +10,7 @@ class JsonifyBuilder < Template::Handler
         def compile(template)
           "json = ::Jsonify::Builder.new(:format => :#{jsonify_format});" +
             template.source +
-          ";json.compile!;"
+          ";\njson.compile!;"
         end
 
         private 
@@ -29,7 +29,7 @@ def default_format
         def self.call(template)
           "json = ::Jsonify::Builder.new(:format => :#{jsonify_format});" +
             template.source +
-          ";json.compile!;"
+          ";\njson.compile!;"
         end
 
         private 

spec/jsonify_builder_spec.rb

@@ -11,6 +11,10 @@
     ViewTemplate.new("json.hello 'world'")
   end
 
+  let( :template_with_trailing_comment ) do
+    ViewTemplate.new("json.hello 'world' #trailing comment")
+  end
+
   context "#{Rails.version}" do
     before do
       Rails.stub_chain(:application, :config, :jsonify_format).and_return( :plain )
@@ -25,6 +29,11 @@
       result.should == '{"hello":"world"}'
     end
 
+    it 'should ignore trailing comments' do
+      result = eval( Handler.call template_with_trailing_comment )
+      result.should == '{"hello":"world"}'
+    end
+
     describe 'when configured with ' do
       def expect_format(format)
         Jsonify::Builder.should_receive(:new).with({:format => format}).and_return( double("json").as_null_object )