summaryrefslogtreecommitdiff
path: root/spec/unit/puppet/parser/functions/validate_slength_spec.rb
diff options
context:
space:
mode:
authorJeff McCune <jeff@puppetlabs.com>2012-07-19 16:14:37 -0700
committerJeff McCune <jeff@puppetlabs.com>2012-07-23 09:13:08 -0700
commit98ff3abd09ed447da37f09dce1ed2d95d0d05397 (patch)
tree4b3db6ddfcdc204f68aeda875201be38a436dd1c /spec/unit/puppet/parser/functions/validate_slength_spec.rb
parentf3caf5f49383344228482b93ad533d06244a3cac (diff)
(Maint) use PuppetlabsSpec::PuppetSeams.parser_scope (2.3.x)
This patch is the same approach as the one that want into 2.2.x. It covers the functions in 2.3.x that do not exist in 2.2.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.
Diffstat (limited to 'spec/unit/puppet/parser/functions/validate_slength_spec.rb')
-rwxr-xr-xspec/unit/puppet/parser/functions/validate_slength_spec.rb8
1 files changed, 2 insertions, 6 deletions
diff --git a/spec/unit/puppet/parser/functions/validate_slength_spec.rb b/spec/unit/puppet/parser/functions/validate_slength_spec.rb
index d2d4ca0..eccf908 100755
--- a/spec/unit/puppet/parser/functions/validate_slength_spec.rb
+++ b/spec/unit/puppet/parser/functions/validate_slength_spec.rb
@@ -1,13 +1,9 @@
-#!/usr/bin/env rspec
+#! /usr/bin/env ruby -S rspec
require 'spec_helper'
describe "the validate_slength function" do
- before :all do
- Puppet::Parser::Functions.autoloader.loadall
- end
-
- let(:scope) { Puppet::Parser::Scope.new }
+ let(:scope) { PuppetlabsSpec::PuppetInternals.scope }
it "should exist" do
Puppet::Parser::Functions.function("validate_slength").should == "function_validate_slength"