Merge pull request #614 from alexceder/make-method-missing-private

Author: rafaelfranca

lib/jbuilder.rb

@@ -272,9 +272,10 @@ def target!
     @attributes.to_json
   end
 
-  private
-
   alias_method :method_missing, :set!
+  private :method_missing
+
+  private
 
   def _extract(object, attributes)
     if ::Hash === object

lib/jbuilder/jbuilder_template.rb

@@ -141,9 +141,10 @@ def set!(name, object = BLANK, *args)
     end
   end
 
-  private
-
   alias_method :method_missing, :set!
+  private :method_missing
+
+  private
 
   def _render_partial_with_options(options)
     options[:locals] ||= options.except(:partial, :as, :collection, :cached)

test/jbuilder_template_test.rb

@@ -39,6 +39,11 @@ class JbuilderTemplateTest < ActiveSupport::TestCase
     assert_equal "hello", result["content"]
   end
 
+  test "method_missing can be used as a key" do
+    result = render('json.method_missing "hello"')
+    assert_equal({ "method_missing" => "hello" }, result)
+  end
+
   test "partial by name with top-level locals" do
     result = render('json.partial! "partial", content: "hello"')
     assert_equal "hello", result["content"]

test/jbuilder_test.rb

@@ -61,6 +61,14 @@ class JbuilderTest < ActiveSupport::TestCase
     assert_equal 'hello', result['content']
   end
 
+  test 'method_missing key' do
+    result = jbuild do |json|
+      json.method_missing 'hello'
+    end
+
+    assert_equal 'hello', result['method_missing']
+  end
+
   test 'single key with false value' do
     result = jbuild do |json|
       json.content false