From 7f17aa48df26113f800110c3e263800931e8eff7 Mon Sep 17 00:00:00 2001 From: Stefan Rohde Date: Tue, 5 Jun 2018 15:11:14 +0200 Subject: [PATCH] nil-single-object should be returned as nil --- lib/jbuilder/jbuilder_template.rb | 2 +- test/jbuilder_template_test.rb | 8 -------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/lib/jbuilder/jbuilder_template.rb b/lib/jbuilder/jbuilder_template.rb index 1cf8cbed..36d525ce 100644 --- a/lib/jbuilder/jbuilder_template.rb +++ b/lib/jbuilder/jbuilder_template.rb @@ -183,7 +183,7 @@ def _is_active_model?(object) def _set_inline_partial(name, object, options) value = if object.nil? - [] + nil elsif _is_collection?(object) _scope{ _render_partial_with_options options.merge(collection: object) } else diff --git a/test/jbuilder_template_test.rb b/test/jbuilder_template_test.rb index 7ec99d6a..39566319 100644 --- a/test/jbuilder_template_test.rb +++ b/test/jbuilder_template_test.rb @@ -217,14 +217,6 @@ def assert_collection_rendered(result, context = nil) assert_collection_rendered result, "posts" end - test "render as empty array if partials as a nil value" do - result = jbuild <<-JBUILDER - json.posts nil, partial: "blog_post", as: :blog_post - JBUILDER - - assert_equal [], result["posts"] - end - test "cache an empty block" do undef_context_methods :fragment_name_with_digest, :cache_fragment_name