puppet 5.5.22 → 6.0.0

data/spec/unit/parser/ast/block_expression_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 require 'puppet/parser/ast/block_expression'
 
@@ -15,9 +16,9 @@ describe 'Puppet::Parser::AST::BlockExpression' do
     StackDepthAST.new({})
   end
 
-  def sequence_probe(name)
-    probe = double("Sequence Probe #{name}")
-    expect(probe).to receive(:safeevaluate).ordered
+  def sequence_probe(name, sequence)
+    probe = mock("Sequence Probe #{name}")
+    probe.expects(:safeevaluate).in_sequence(sequence)
     probe
   end
 
@@ -56,9 +57,10 @@ describe 'Puppet::Parser::AST::BlockExpression' do
   end
 
   it "evaluates sequenced children in order" do
-    expr1 = block_of([sequence_probe("Step 1")])
-    expr2 = block_of([sequence_probe("Step 2")])
-    expr3 = block_of([sequence_probe("Step 3")])
+    evaluation_order = sequence("Child evaluation order")
+    expr1 = block_of([sequence_probe("Step 1", evaluation_order)])
+    expr2 = block_of([sequence_probe("Step 2", evaluation_order)])
+    expr3 = block_of([sequence_probe("Step 3", evaluation_order)])
 
     expr1.sequence_with(expr2).sequence_with(expr3).evaluate(NO_SCOPE)
   end

data/spec/unit/parser/ast/leaf_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 describe Puppet::Parser::AST::Leaf do
@@ -5,14 +6,14 @@ describe Puppet::Parser::AST::Leaf do
     node     = Puppet::Node.new('localhost')
     compiler = Puppet::Parser::Compiler.new(node)
     @scope   = Puppet::Parser::Scope.new(compiler)
-    @value = double('value')
+    @value = stub 'value'
     @leaf = Puppet::Parser::AST::Leaf.new(:value => @value)
   end
 
   describe "when converting to string" do
     it "should transform its value to string" do
-      value = double('value', :is_a? => true)
-      expect(value).to receive(:to_s)
+      value = stub 'value', :is_a? => true
+      value.expects(:to_s)
       Puppet::Parser::AST::Leaf.new( :value => value ).to_s
     end
   end
@@ -22,7 +23,7 @@ describe Puppet::Parser::AST::Leaf do
   end
 
   it "should delegate match to ==" do
-    expect(@value).to receive(:==).with("value")
+    @value.expects(:==).with("value")
 
     @leaf.match("value")
   end
@@ -38,14 +39,14 @@ describe Puppet::Parser::AST::Regex do
 
   describe "when initializing" do
     it "should create a Regexp with its content when value is not a Regexp" do
-      expect(Regexp).to receive(:new).with("/ab/")
+      Regexp.expects(:new).with("/ab/")
 
       Puppet::Parser::AST::Regex.new :value => "/ab/"
     end
 
     it "should not create a Regexp with its content when value is a Regexp" do
       value = Regexp.new("/ab/")
-      expect(Regexp).not_to receive(:new).with("/ab/")
+      Regexp.expects(:new).with("/ab/").never
 
       Puppet::Parser::AST::Regex.new :value => value
     end
@@ -60,11 +61,11 @@ describe Puppet::Parser::AST::Regex do
   end
 
   it 'should return the PRegexpType#regexp_to_s_with_delimiters with to_s' do
-    regex = double('regex')
-    allow(Regexp).to receive(:new).and_return(regex)
+    regex = stub 'regex'
+    Regexp.stubs(:new).returns(regex)
 
     val = Puppet::Parser::AST::Regex.new :value => '/ab/'
-    expect(Puppet::Pops::Types::PRegexpType).to receive(:regexp_to_s_with_delimiters)
+    Puppet::Pops::Types::PRegexpType.expects(:regexp_to_s_with_delimiters)
 
     val.to_s
   end
@@ -73,7 +74,7 @@ describe Puppet::Parser::AST::Regex do
     regex = Regexp.new("/ab/")
     val = Puppet::Parser::AST::Regex.new :value => regex
 
-    expect(regex).to receive(:match).with("value")
+    regex.expects(:match).with("value")
 
     val.match("value")
   end
@@ -85,8 +86,8 @@ describe Puppet::Parser::AST::HostName do
     compiler = Puppet::Parser::Compiler.new(node)
     @scope   = Puppet::Parser::Scope.new(compiler)
     @value   = 'value'
-    allow(@value).to receive(:to_s).and_return(@value)
-    allow(@value).to receive(:downcase).and_return(@value)
+    @value.stubs(:to_s).returns(@value)
+    @value.stubs(:downcase).returns(@value)
     @host = Puppet::Parser::AST::HostName.new(:value => @value)
   end
 
@@ -99,16 +100,16 @@ describe Puppet::Parser::AST::HostName do
   end
 
   it "should stringify the value" do
-    value = double('value', :=~ => false)
+    value = stub 'value', :=~ => false
 
-    expect(value).to receive(:to_s).and_return("test")
+    value.expects(:to_s).returns("test")
 
     Puppet::Parser::AST::HostName.new(:value => value)
   end
 
   it "should downcase the value" do
-    value = double('value', :=~ => false)
-    allow(value).to receive(:to_s).and_return("UPCASED")
+    value = stub 'value', :=~ => false
+    value.stubs(:to_s).returns("UPCASED")
     host = Puppet::Parser::AST::HostName.new(:value => value)
 
     host.value == "upcased"
@@ -119,18 +120,18 @@ describe Puppet::Parser::AST::HostName do
   end
 
   it "should delegate eql? to the underlying value if it is an HostName" do
-    expect(@value).to receive(:eql?).with("value")
+    @value.expects(:eql?).with("value")
     @host.eql?("value")
   end
 
   it "should delegate eql? to the underlying value if it is not an HostName" do
-    value = double('compared', :is_a? => true, :value => "value")
-    expect(@value).to receive(:eql?).with("value")
+    value = stub 'compared', :is_a? => true, :value => "value"
+    @value.expects(:eql?).with("value")
     @host.eql?(value)
   end
 
   it "should delegate hash to the underlying value" do
-    expect(@value).to receive(:hash)
+    @value.expects(:hash)
     @host.hash
   end
 end

data/spec/unit/parser/compiler_spec.rb

@@ -71,20 +71,20 @@ describe Puppet::Parser::Compiler do
     # hidden away in the implementation and we keep losing the race.)
     # --daniel 2011-04-21
     now = Time.now
-    allow(Time).to receive(:now).and_return(now)
+    Time.stubs(:now).returns(now)
 
     @node = Puppet::Node.new("testnode",
                              :facts => Puppet::Node::Facts.new("facts", {}),
                              :environment => environment)
     @known_resource_types = environment.known_resource_types
     @compiler = Puppet::Parser::Compiler.new(@node)
-    @scope = Puppet::Parser::Scope.new(@compiler, :source => double('source'))
+    @scope = Puppet::Parser::Scope.new(@compiler, :source => stub('source'))
     @scope_resource = Puppet::Parser::Resource.new(:file, "/my/file", :scope => @scope)
     @scope.resource = @scope_resource
   end
 
   it "should fail intelligently when a class-level compile fails" do
-    expect(Puppet::Parser::Compiler).to receive(:new).and_raise(ArgumentError)
+    Puppet::Parser::Compiler.expects(:new).raises ArgumentError
     expect { Puppet::Parser::Compiler.compile(@node) }.to raise_error(Puppet::Error)
   end
 
@@ -94,7 +94,7 @@ describe Puppet::Parser::Compiler do
 
   it "fails if the node's environment has validation errors" do
     conflicted_environment = Puppet::Node::Environment.create(:testing, [], '/some/environment.conf/manifest.pp')
-    allow(conflicted_environment).to receive(:validation_errors).and_return(['bad environment'])
+    conflicted_environment.stubs(:validation_errors).returns(['bad environment'])
     @node.environment = conflicted_environment
     expect { Puppet::Parser::Compiler.compile(@node) }.to raise_error(Puppet::Error, /Compilation has been halted because.*bad environment/)
   end
@@ -108,6 +108,7 @@ describe Puppet::Parser::Compiler do
   end
 
   describe "when initializing" do
+
     it 'should not create the settings class more than once' do
       logs = []
       Puppet::Util::Log.with_destination(Puppet::Test::LogCollector.new(logs)) do
@@ -217,6 +218,7 @@ describe Puppet::Parser::Compiler do
   end
 
   describe "when managing scopes" do
+
     it "should create a top scope" do
       expect(@compiler.topscope).to be_instance_of(Puppet::Parser::Scope)
     end
@@ -226,7 +228,7 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should set the parent scope of the new scope to be the passed-in parent" do
-      scope = double('scope')
+      scope = mock 'scope'
       newscope = @compiler.newscope(scope)
 
       expect(newscope.parent).to equal(scope)
@@ -240,6 +242,7 @@ describe Puppet::Parser::Compiler do
   end
 
   describe "when compiling" do
+
     def compile_methods
       [:set_node_parameters, :evaluate_main, :evaluate_ast_node, :evaluate_node_classes, :evaluate_generators, :fail_on_unevaluated,
         :finish, :store, :extract, :evaluate_relationships]
@@ -247,12 +250,12 @@ describe Puppet::Parser::Compiler do
 
     # Stub all of the main compile methods except the ones we're specifically interested in.
     def compile_stub(*except)
-      (compile_methods - except).each { |m| allow(@compiler).to receive(m) }
+      (compile_methods - except).each { |m| @compiler.stubs(m) }
     end
 
     it "should set node parameters as variables in the top scope" do
       params = {"a" => "b", "c" => "d"}
-      allow(@node).to receive(:parameters).and_return(params)
+      @node.stubs(:parameters).returns(params)
       compile_stub(:set_node_parameters)
       @compiler.compile
       expect(@compiler.topscope['a']).to eq("b")
@@ -261,7 +264,7 @@ describe Puppet::Parser::Compiler do
 
     it "should set node parameters that are of Symbol type as String variables in the top scope" do
       params = {"a" => :b}
-      allow(@node).to receive(:parameters).and_return(params)
+      @node.stubs(:parameters).returns(params)
       compile_stub(:set_node_parameters)
       @compiler.compile
       expect(@compiler.topscope['a']).to eq("b")
@@ -285,7 +288,7 @@ describe Puppet::Parser::Compiler do
 
     it "should set the client and server versions on the catalog" do
       params = {"clientversion" => "2", "serverversion" => "3"}
-      allow(@node).to receive(:parameters).and_return(params)
+      @node.stubs(:parameters).returns(params)
       compile_stub(:set_node_parameters)
       @compiler.compile
       expect(@compiler.catalog.client_version).to eq("2")
@@ -295,8 +298,8 @@ describe Puppet::Parser::Compiler do
     it "should evaluate the main class if it exists" do
       compile_stub(:evaluate_main)
       main_class = @known_resource_types.add Puppet::Resource::Type.new(:hostclass, "")
-      expect(main_class).to receive(:evaluate_code).with(be_a(Puppet::Parser::Resource))
-      expect(@compiler.topscope).to receive(:source=).with(main_class)
+      main_class.expects(:evaluate_code).with { |r| r.is_a?(Puppet::Parser::Resource) }
+      @compiler.topscope.expects(:source=).with(main_class)
 
       @compiler.compile
     end
@@ -318,9 +321,9 @@ describe Puppet::Parser::Compiler do
     it "should evaluate all added collections" do
       colls = []
       # And when the collections fail to evaluate.
-      colls << double("coll1-false")
-      colls << double("coll2-false")
-      colls.each { |c| expect(c).to receive(:evaluate).and_return(false) }
+      colls << mock("coll1-false")
+      colls << mock("coll2-false")
+      colls.each { |c| c.expects(:evaluate).returns(false) }
 
       @compiler.add_collection(colls[0])
       @compiler.add_collection(colls[1])
@@ -333,7 +336,7 @@ describe Puppet::Parser::Compiler do
       resource = resource(:file, "testing")
 
       @compiler.add_resource(@scope, resource)
-      expect(resource).not_to receive(:evaluate)
+      resource.expects(:evaluate).never
 
       @compiler.compile
     end
@@ -344,17 +347,17 @@ describe Puppet::Parser::Compiler do
       @compiler.add_resource(@scope, resource)
 
       # We have to now mark the resource as evaluated
-      expect(resource).to receive(:evaluate) { resource.evaluated = true }
+      resource.expects(:evaluate).with { |*whatever| resource.evaluated = true }
 
       @compiler.compile
     end
 
     it "should not evaluate already-evaluated resources" do
       resource = resource(:file, "testing")
-      allow(resource).to receive(:evaluated?).and_return(true)
+      resource.stubs(:evaluated?).returns true
 
       @compiler.add_resource(@scope, resource)
-      expect(resource).not_to receive(:evaluate)
+      resource.expects(:evaluate).never
 
       @compiler.compile
     end
@@ -366,8 +369,9 @@ describe Puppet::Parser::Compiler do
       resource2 = CompilerTestResource.new(:file, "other")
 
       # We have to now mark the resource as evaluated
-      expect(resource).to receive(:evaluate) { resource.evaluated = true; @compiler.add_resource(@scope, resource2) }
-      expect(resource2).to receive(:evaluate) { resource2.evaluated = true }
+      resource.expects(:evaluate).with { |*whatever| resource.evaluated = true; @compiler.add_resource(@scope, resource2) }
+      resource2.expects(:evaluate).with { |*whatever| resource2.evaluated = true }
+
 
       @compiler.compile
     end
@@ -388,12 +392,12 @@ describe Puppet::Parser::Compiler do
       it "should call finish() on all resources" do
         # Add a resource that does respond to :finish
         resource = Puppet::Parser::Resource.new "file", "finish", :scope => @scope
-        expect(resource).to receive(:finish)
+        resource.expects(:finish)
 
         @compiler.add_resource(@scope, resource)
 
         # And one that does not
-        dnf_resource = double("dnf", :ref => "File[dnf]", :type => "file", :resource_type => nil, :[] => nil, :class? => nil, :stage? => nil)
+        dnf_resource = stub_everything "dnf", :ref => "File[dnf]", :type => "file"
 
         @compiler.add_resource(@scope, dnf_resource)
 
@@ -401,11 +405,13 @@ describe Puppet::Parser::Compiler do
       end
 
       it "should call finish() in add_resource order" do
+        resources = sequence('resources')
+
         resource1 = add_resource("finish1")
-        expect(resource1).to receive(:finish).ordered
+        resource1.expects(:finish).in_sequence(resources)
 
         resource2 = add_resource("finish2")
-        expect(resource2).to receive(:finish).ordered
+        resource2.expects(:finish).in_sequence(resources)
 
         @compiler.send(:finish)
       end
@@ -546,21 +552,22 @@ describe Puppet::Parser::Compiler do
       resource.virtual = true
       @compiler.add_resource(@scope, resource)
 
-      expect(resource).not_to receive(:evaluate)
+      resource.expects(:evaluate).never
 
       @compiler.compile
     end
   end
 
   describe "when evaluating collections" do
+
     it "should evaluate each collection" do
       2.times { |i|
-        coll = double('coll%s' % i)
+        coll = mock 'coll%s' % i
         @compiler.add_collection(coll)
 
         # This is the hard part -- we have to emulate the fact that
         # collections delete themselves if they are done evaluating.
-        expect(coll).to receive(:evaluate) do
+        coll.expects(:evaluate).with do
           @compiler.delete_collection(coll)
         end
       }
@@ -569,8 +576,8 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should not fail when there are unevaluated resource collections that do not refer to specific resources" do
-      coll = double('coll', :evaluate => false)
-      expect(coll).to receive(:unresolved_resources).and_return(nil)
+      coll = stub 'coll', :evaluate => false
+      coll.expects(:unresolved_resources).returns(nil)
 
       @compiler.add_collection(coll)
 
@@ -578,8 +585,8 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should fail when there are unevaluated resource collections that refer to a specific resource" do
-      coll = double('coll', :evaluate => false)
-      expect(coll).to receive(:unresolved_resources).and_return(:something)
+      coll = stub 'coll', :evaluate => false
+      coll.expects(:unresolved_resources).returns(:something)
 
       @compiler.add_collection(coll)
 
@@ -587,8 +594,8 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should fail when there are unevaluated resource collections that refer to multiple specific resources" do
-      coll = double('coll', :evaluate => false)
-      expect(coll).to receive(:unresolved_resources).and_return([:one, :two])
+      coll = stub 'coll', :evaluate => false
+      coll.expects(:unresolved_resources).returns([:one, :two])
 
       @compiler.add_collection(coll)
 
@@ -598,21 +605,22 @@ describe Puppet::Parser::Compiler do
 
   describe "when evaluating relationships" do
     it "should evaluate each relationship with its catalog" do
-      dep = double('dep')
-      expect(dep).to receive(:evaluate).with(@compiler.catalog)
+      dep = stub 'dep'
+      dep.expects(:evaluate).with(@compiler.catalog)
       @compiler.add_relationship dep
       @compiler.evaluate_relationships
     end
   end
 
   describe "when told to evaluate missing classes" do
+
     it "should fail if there's no source listed for the scope" do
-      scope = double('scope', :source => nil)
+      scope = stub 'scope', :source => nil
       expect { @compiler.evaluate_classes(%w{one two}, scope) }.to raise_error(Puppet::DevError)
     end
 
     it "should raise an error if a class is not found" do
-      expect(@scope.environment.known_resource_types).to receive(:find_hostclass).with("notfound").and_return(nil)
+      @scope.environment.known_resource_types.expects(:find_hostclass).with("notfound").returns(nil)
       expect{ @compiler.evaluate_classes(%w{notfound}, @scope) }.to raise_error(Puppet::Error, /Could not find class/)
     end
 
@@ -624,10 +632,11 @@ describe Puppet::Parser::Compiler do
   end
 
   describe "when evaluating found classes" do
+
     before do
       Puppet.settings[:data_binding_terminus] = "none"
       @class = @known_resource_types.add Puppet::Resource::Type.new(:hostclass, "myclass")
-      @resource = double('resource', :ref => "Class[myclass]", :type => "file")
+      @resource = stub 'resource', :ref => "Class[myclass]", :type => "file"
     end
 
     around do |example|
@@ -640,10 +649,10 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should evaluate each class" do
-      allow(@compiler.catalog).to receive(:tag)
+      @compiler.catalog.stubs(:tag)
 
-      expect(@class).to receive(:ensure_in_catalog).with(@scope)
-      allow(@scope).to receive(:class_scope).with(@class)
+      @class.expects(:ensure_in_catalog).with(@scope)
+      @scope.stubs(:class_scope).with(@class)
 
       @compiler.evaluate_classes(%w{myclass}, @scope)
     end
@@ -731,71 +740,73 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should not evaluate the resources created for found classes unless asked" do
-      allow(@compiler.catalog).to receive(:tag)
+      @compiler.catalog.stubs(:tag)
 
-      expect(@resource).not_to receive(:evaluate)
+      @resource.expects(:evaluate).never
 
-      expect(@class).to receive(:ensure_in_catalog).and_return(@resource)
-      allow(@scope).to receive(:class_scope).with(@class)
+      @class.expects(:ensure_in_catalog).returns(@resource)
+      @scope.stubs(:class_scope).with(@class)
 
       @compiler.evaluate_classes(%w{myclass}, @scope)
     end
 
     it "should immediately evaluate the resources created for found classes when asked" do
-      allow(@compiler.catalog).to receive(:tag)
+      @compiler.catalog.stubs(:tag)
 
-      expect(@resource).to receive(:evaluate)
-      expect(@class).to receive(:ensure_in_catalog).and_return(@resource)
-      allow(@scope).to receive(:class_scope).with(@class)
+      @resource.expects(:evaluate)
+      @class.expects(:ensure_in_catalog).returns(@resource)
+      @scope.stubs(:class_scope).with(@class)
 
       @compiler.evaluate_classes(%w{myclass}, @scope, false)
     end
 
     it "should skip classes that have already been evaluated" do
-      allow(@compiler.catalog).to receive(:tag)
+      @compiler.catalog.stubs(:tag)
 
-      allow(@scope).to receive(:class_scope).with(@class).and_return(@scope)
+      @scope.stubs(:class_scope).with(@class).returns(@scope)
 
-      expect(@compiler).not_to receive(:add_resource)
+      @compiler.expects(:add_resource).never
 
-      expect(@resource).not_to receive(:evaluate)
+      @resource.expects(:evaluate).never
 
-      expect(Puppet::Parser::Resource).not_to receive(:new)
+      Puppet::Parser::Resource.expects(:new).never
       @compiler.evaluate_classes(%w{myclass}, @scope, false)
     end
 
     it "should skip classes previously evaluated with different capitalization" do
-      allow(@compiler.catalog).to receive(:tag)
-      allow(@scope.environment.known_resource_types).to receive(:find_hostclass).with("MyClass").and_return(@class)
-      allow(@scope).to receive(:class_scope).with(@class).and_return(@scope)
-      expect(@compiler).not_to receive(:add_resource)
-      expect(@resource).not_to receive(:evaluate)
-      expect(Puppet::Parser::Resource).not_to receive(:new)
+      @compiler.catalog.stubs(:tag)
+      @scope.environment.known_resource_types.stubs(:find_hostclass).with("MyClass").returns(@class)
+      @scope.stubs(:class_scope).with(@class).returns(@scope)
+      @compiler.expects(:add_resource).never
+      @resource.expects(:evaluate).never
+      Puppet::Parser::Resource.expects(:new).never
       @compiler.evaluate_classes(%w{MyClass}, @scope, false)
     end
   end
 
   describe "when evaluating AST nodes with no AST nodes present" do
+
     it "should do nothing" do
-      allow(@compiler.environment.known_resource_types).to receive(:nodes).and_return(false)
-      expect(Puppet::Parser::Resource).not_to receive(:new)
+      @compiler.environment.known_resource_types.stubs(:nodes).returns(false)
+      Puppet::Parser::Resource.expects(:new).never
 
       @compiler.send(:evaluate_ast_node)
     end
   end
 
   describe "when evaluating AST nodes with AST nodes present" do
+
     before do
-      allow(@compiler.environment.known_resource_types).to receive(:nodes?).and_return(true)
+      @compiler.environment.known_resource_types.stubs(:nodes?).returns true
 
       # Set some names for our test
-      allow(@node).to receive(:names).and_return(%w{a b c})
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("a").and_return(nil)
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("b").and_return(nil)
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("c").and_return(nil)
+      @node.stubs(:names).returns(%w{a b c})
+      @compiler.environment.known_resource_types.stubs(:node).with("a").returns(nil)
+      @compiler.environment.known_resource_types.stubs(:node).with("b").returns(nil)
+      @compiler.environment.known_resource_types.stubs(:node).with("c").returns(nil)
 
       # It should check this last, of course.
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("default").and_return(nil)
+      @compiler.environment.known_resource_types.stubs(:node).with("default").returns(nil)
     end
 
     it "should fail if the named node cannot be found" do
@@ -803,33 +814,33 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should evaluate the first node class matching the node name" do
-      node_class = double('node', :name => "c", :evaluate_code => nil)
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("c").and_return(node_class)
+      node_class = stub 'node', :name => "c", :evaluate_code => nil
+      @compiler.environment.known_resource_types.stubs(:node).with("c").returns(node_class)
 
-      node_resource = double('node resource', :ref => "Node[c]", :evaluate => nil, :type => "node")
-      expect(node_class).to receive(:ensure_in_catalog).and_return(node_resource)
+      node_resource = stub 'node resource', :ref => "Node[c]", :evaluate => nil, :type => "node"
+      node_class.expects(:ensure_in_catalog).returns(node_resource)
 
       @compiler.compile
     end
 
     it "should match the default node if no matching node can be found" do
-      node_class = double('node', :name => "default", :evaluate_code => nil)
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("default").and_return(node_class)
+      node_class = stub 'node', :name => "default", :evaluate_code => nil
+      @compiler.environment.known_resource_types.stubs(:node).with("default").returns(node_class)
 
-      node_resource = double('node resource', :ref => "Node[default]", :evaluate => nil, :type => "node")
-      expect(node_class).to receive(:ensure_in_catalog).and_return(node_resource)
+      node_resource = stub 'node resource', :ref => "Node[default]", :evaluate => nil, :type => "node"
+      node_class.expects(:ensure_in_catalog).returns(node_resource)
 
       @compiler.compile
     end
 
     it "should evaluate the node resource immediately rather than using lazy evaluation" do
-      node_class = double('node', :name => "c")
-      allow(@compiler.environment.known_resource_types).to receive(:node).with("c").and_return(node_class)
+      node_class = stub 'node', :name => "c"
+      @compiler.environment.known_resource_types.stubs(:node).with("c").returns(node_class)
 
-      node_resource = double('node resource', :ref => "Node[c]", :type => "node")
-      expect(node_class).to receive(:ensure_in_catalog).and_return(node_resource)
+      node_resource = stub 'node resource', :ref => "Node[c]", :type => "node"
+      node_class.expects(:ensure_in_catalog).returns(node_resource)
 
-      expect(node_resource).to receive(:evaluate)
+      node_resource.expects(:evaluate)
 
       @compiler.send(:evaluate_ast_node)
     end
@@ -967,8 +978,9 @@ describe Puppet::Parser::Compiler do
   end
 
   describe "when managing resource overrides" do
+
     before do
-      @override = double('override', :ref => "File[/foo]", :type => "my")
+      @override = stub 'override', :ref => "File[/foo]", :type => "my"
       @resource = resource(:file, "/foo")
     end
 
@@ -978,7 +990,7 @@ describe Puppet::Parser::Compiler do
 
     it "should apply overrides to the appropriate resources" do
       @compiler.add_resource(@scope, @resource)
-      expect(@resource).to receive(:merge).with(@override)
+      @resource.expects(:merge).with(@override)
 
       @compiler.add_override(@override)
 
@@ -986,7 +998,7 @@ describe Puppet::Parser::Compiler do
     end
 
     it "should accept overrides before the related resource has been created" do
-      expect(@resource).to receive(:merge).with(@override)
+      @resource.expects(:merge).with(@override)
 
       # First store the override
       @compiler.add_override(@override)