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

[ckaushik]

Adding the instance variable @template_finder to the new TemplateContext class for rendering partials

[chef-supermarket]

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.

GitHub Users Who Are Not Authorized To Contribute

The following GitHub users do not appear to have signed a CLA:

• @ckaushik

Please sign the CLA here.

[chef-supermarket]

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

[samdunne]

Can someone look at this?

[thommay]

This looks reasonable; 👍
@chef/client-core

[tyler-ball]

Can https://github.com/chef/chef/blob/master/spec/unit/mixin/template_spec.rb be updated with a test for this?

[danielsdeleo]

I'm okay to merge if we need it now to fix the broken, but this really deserves a regression test to ensure it works in the future.

[ckaushik]

Can we merge it now to fix the broken template rendering?
On Jul 23, 2015 12:05 PM, "Dan DeLeo" [email protected] wrote:

I'm okay to merge if we need it now to fix the broken, but this really
deserves a regression test to ensure it works in the future.

—
Reply to this email directly or view it on GitHub
#3510 (comment).

[samdunne]

I added regression testing here: ckaushik#1

@ckaushik will you merge this into your PR?

cc/ @tyler-ball, @danielsdeleo

[ckaushik]

@danielsdeleo does this look good for merging?

[danielsdeleo]

Nice! 👍

[tyler-ball]

👍