Skip to content

(Maint) Rename PuppetlabsSpec::Puppet{Seams,Internals} #18

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

Conversation

jeffmccune
Copy link
Contributor

We've added more methods to obtain instances of internal Puppet objects.
These additions make the name PuppetSeams more awkward than it already was.

This patch fixes the problem of the awkward name by renaming the module to
PuppetlabsSpec::PuppetInternals This module is to be used in tests to
provide easier (Puppet version agnostic) access to the scope, resource, node,
and compiler method.

Paired-with: Andy Parker [email protected]

We've added more methods to obtain instances of internal Puppet objects.
These additions make the name PuppetSeams more awkward than it already
was.

This patch fixes the problem of the awkward name by renaming the module
to `PuppetlabsSpec::PuppetInternals`  This module is to be used in
tests to provide easier (Puppet version agnostic) access to the
scope, resource, node, and compiler method.

Paired-with: Andy Parker <[email protected]>
@jeffmccune
Copy link
Contributor Author

Merging this needs to be coordinated with hiera-puppet and stdlib (all branches). Please see the card in trello for details.

@jeffmccune jeffmccune closed this Jul 20, 2012
@jeffmccune jeffmccune reopened this Jul 23, 2012
jeffmccune pushed a commit that referenced this pull request Jul 23, 2012
…als_module

(Maint) Rename PuppetlabsSpec::Puppet{Seams,Internals}
@jeffmccune jeffmccune merged commit da76515 into puppetlabs:master Jul 23, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants