]> gitweb.fluxo.info Git - puppet-stdlib.git/commit
(Maint) use PuppetlabsSpec::PuppetInternals.scope (master)
authorJeff McCune <jeff@puppetlabs.com>
Thu, 19 Jul 2012 23:14:37 +0000 (16:14 -0700)
committerPatrick Carlisle <patrick@puppetlabs.com>
Fri, 10 Aug 2012 18:54:44 +0000 (11:54 -0700)
commitdeafe88e02499a7bdababf0b5dc264fcc3edecf1
tree155348624ada5fe2c875453f1141979d09f171ea
parentbb1afc311c7d372bb6972504c3d5faa1eaeee6f0
(Maint) use PuppetlabsSpec::PuppetInternals.scope (master)

This patch is the same approach as the one that want into 2.3.x.  It
covers the functions in master that do not exist in 2.3.x.

Without this patch all of the spec tests for parser functions in stdlib
would instantiate their own scope instances.  This is a problem because
the standard library is tightly coupled with the internal behavior of
Puppet.  Tight coupling like this creates failures when we change the
internal behavior of Puppet.  This is exactly what happened recently
when we changed the method signature for the initializer of
Puppet::Parser::Scope instances.

This patch fixes the problem by creating scope instances using the
puppet labs spec helper.  The specific method that provides scope
instances in Puppet-version-independent way is something like this:

    let(:scope) { PuppetlabsSpec::PuppetInternals.scope }

This patch simply implements this across the board.

Paired-with: Andrew Parker <andy@puppetlabs.com>
spec/unit/puppet/parser/functions/fqdn_rotate_spec.rb