Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explicitly load functions for scope instance #38

Closed
wants to merge 1 commit into from

Conversation

bodepd
Copy link
Collaborator

@bodepd bodepd commented Aug 14, 2012

Functions are loaded in Puppet as they are encountered.

If a user is just invoking functions from Ruby, then they
need to manually call the Puppet::Parser::Functions.function method
to ensure that functions are properly loaded.

This commit adds the load function call to ensure that the functions being
tested can be found by puppet.

Functions are loaded in Puppet as they are encountered.

If a user is just invoking functions from Ruby, then they
need to manually call the Puppet::Parser::Functions.function method
to ensure that functions are properly loaded.

This commit adds the load function call to ensure that the functions being
tested can be found by puppet.
@travisbot
Copy link

This pull request passes (merged 263998c into 8d4c964).

@jeffmccune
Copy link
Collaborator

@bodepd

I'm not sure this fixes the problem we're facing with the stdlib functions.

Could we pair briefly on this today, I have a couple of ideas that I think could make this process of improving function testing in rspec-puppet and stdlib much quicker. I think it would be quick to show them to you in person rather than go back and forth on the pull requests.

Thanks,
-Jeff

@jeffmccune
Copy link
Collaborator

@bodepd I'm going to close this pull request. Please see #39.

@jeffmccune jeffmccune closed this Aug 15, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants