Fixing Issue #2513 - the broken render of nested partial templates

lib/chef/mixin/template.rb

@@ -89,6 +89,7 @@ def render(partial_name, options = {})
           raise "You cannot render partials in this context" unless @template_finder
 
           partial_variables = options.delete(:variables) || _public_instance_variables
+          partial_variables[:template_finder] = @template_finder
           partial_context = self.class.new(partial_variables)
           partial_context._extend_modules(@_extension_modules)
 

spec/data/cookbooks/openldap/templates/default/nested_openldap_partials.erb

@@ -0,0 +1 @@
+before <%= render 'nested_partial.erb', :variables => { :hello => @test } %> after

spec/data/cookbooks/openldap/templates/default/nested_partial.erb

@@ -0,0 +1 @@
+{<%= @hello %>}

spec/unit/cookbook/syntax_check_spec.rb

@@ -54,6 +54,8 @@
     basenames = %w{ helpers_via_partial_test.erb
                     helper_test.erb
                     openldap_stuff.conf.erb
+                    nested_openldap_partials.erb
+                    nested_partial.erb
                     openldap_variable_stuff.conf.erb
                     test.erb
                     some_windows_line_endings.erb

spec/unit/mixin/template_spec.rb

@@ -150,6 +150,11 @@
       output == "before {super secret is candy} after"
     end
 
+    it "should pass the template finder to the partials" do
+      output = @template_context.render_template_from_string("before {<%= render 'nested_openldap_partials.erb', :variables => {:hello => 'Hello World!' } %>} after")
+      output == "before {Hello World!} after"
+    end
+
     it "should pass variables to partials" do
       output = @template_context.render_template_from_string("before {<%= render 'openldap_variable_stuff.conf.erb', :variables => {:secret => 'whatever' } %>} after")
       expect(output).to eq("before {super secret is whatever} after")
@@ -266,4 +271,3 @@ def do_raise
     end
   end
 end
-