puppet 6.0.10 → 6.1.0

data/spec/integration/application/doc_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 require 'puppet_spec/files'
 require 'puppet/application/doc'
@@ -7,7 +8,7 @@ describe Puppet::Application::Doc do
 
   it "should respect the -o option" do
     puppetdoc = Puppet::Application[:doc]
-    allow(puppetdoc.command_line).to receive(:args).and_return(['foo', '-o', 'bar'])
+    puppetdoc.command_line.stubs(:args).returns(['foo', '-o', 'bar'])
     puppetdoc.parse_options
     expect(puppetdoc.options[:outputdir]).to eq('bar')
   end

data/spec/integration/application/lookup_spec.rb

@@ -52,7 +52,7 @@ describe 'lookup' do
 
     def lookup(key, options = {}, explain = false)
       key = [key] unless key.is_a?(Array)
-      allow(app.command_line).to receive(:args).and_return(key)
+      app.command_line.stubs(:args).returns(key)
       if explain
         app.options[:explain] = true
         app.options[:render_as] = :s
@@ -98,15 +98,15 @@ describe 'lookup' do
 
       it ':plain without --compile' do
         Puppet.settings[:node_terminus] = 'exec'
-        expect_any_instance_of(Puppet::Node::Plain).to receive(:find).and_return(node)
-        expect_any_instance_of(Puppet::Node::Exec).not_to receive(:find)
+        Puppet::Node::Plain.any_instance.expects(:find).returns(node)
+        Puppet::Node::Exec.any_instance.expects(:find).never
         expect(lookup('a')).to eql('value a')
       end
 
       it 'configured in Puppet settings with --compile' do
         Puppet.settings[:node_terminus] = 'exec'
-        expect_any_instance_of(Puppet::Node::Plain).not_to receive(:find)
-        expect_any_instance_of(Puppet::Node::Exec).to receive(:find).and_return(node)
+        Puppet::Node::Plain.any_instance.expects(:find).never
+        Puppet::Node::Exec.any_instance.expects(:find).returns(node)
         expect(lookup('a', :compile => true)).to eql('value a')
       end
     end

data/spec/integration/configurer_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/configurer'
@@ -19,10 +20,10 @@ describe Puppet::Configurer do
     end
 
     it "should send a transaction report with valid data" do
-      allow(@configurer).to receive(:save_last_run_summary)
-      expect(Puppet::Transaction::Report.indirection).to receive(:save) do |report, x|
-        expect(report.time).to be_a(Time)
-        expect(report.logs.length).to be > 0
+
+      @configurer.stubs(:save_last_run_summary)
+      Puppet::Transaction::Report.indirection.expects(:save).with do |report, x|
+        report.time.class == Time and report.logs.length > 0
       end
 
       Puppet[:report] = true
@@ -32,7 +33,7 @@ describe Puppet::Configurer do
 
     it "should save a correct last run summary" do
       report = Puppet::Transaction::Report.new
-      allow(Puppet::Transaction::Report.indirection).to receive(:save)
+      Puppet::Transaction::Report.indirection.stubs(:save)
 
       Puppet[:lastrunfile] = tmpfile("lastrunfile")
       Puppet.settings.setting(:lastrunfile).mode = 0666

data/spec/integration/defaults_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/defaults'
@@ -25,7 +26,7 @@ describe "Puppet defaults" do
 
   describe "when setting the :factpath" do
     it "should add the :factpath to Facter's search paths" do
-      expect(Facter).to receive(:search).with("/my/fact/path")
+      Facter.expects(:search).with("/my/fact/path")
 
       Puppet.settings[:factpath] = "/my/fact/path"
     end
@@ -82,19 +83,19 @@ describe "Puppet defaults" do
   end
 
   it "should use the service user and group for the yamldir" do
-    allow(Puppet.settings).to receive(:service_user_available?).and_return(true)
-    allow(Puppet.settings).to receive(:service_group_available?).and_return(true)
+    Puppet.settings.stubs(:service_user_available?).returns true
+    Puppet.settings.stubs(:service_group_available?).returns true
     expect(Puppet.settings.setting(:yamldir).owner).to eq(Puppet.settings[:user])
     expect(Puppet.settings.setting(:yamldir).group).to eq(Puppet.settings[:group])
   end
 
   it "should specify that the host private key should be owned by the service user" do
-    allow(Puppet.settings).to receive(:service_user_available?).and_return(true)
+    Puppet.settings.stubs(:service_user_available?).returns true
     expect(Puppet.settings.setting(:hostprivkey).owner).to eq(Puppet.settings[:user])
   end
 
   it "should specify that the host certificate should be owned by the service user" do
-    allow(Puppet.settings).to receive(:service_user_available?).and_return(true)
+    Puppet.settings.stubs(:service_user_available?).returns true
     expect(Puppet.settings.setting(:hostcert).owner).to eq(Puppet.settings[:user])
   end
 

data/spec/integration/directory_environments_spec.rb

@@ -4,7 +4,7 @@ describe "directory environments" do
   let(:args) { ['--configprint', 'modulepath', '--environment', 'direnv'] }
   let(:puppet) do
     app = Puppet::Application[:apply]
-    allow(app).to receive(:command_line).and_return(double('command_line', :args => []))
+    app.stubs(:command_line).returns(stub('command_line', :args => []))
     app
   end
 

data/spec/integration/faces/config_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 require 'puppet/face'
 
@@ -64,7 +65,7 @@ rando_key=foobar
     end
 
     before :each do
-      allow(subject).to receive(:report_section_and_environment)
+      subject.stubs(:report_section_and_environment)
     end
 
     # key must be a defined setting
@@ -73,13 +74,13 @@ rando_key=foobar
         value = "value#{key.reverse}value"
 
         # needed for the subject.set to write to correct file
-        allow(Puppet.settings).to receive(:which_configuration_file).and_return(tmp_config)
+        Puppet.settings.stubs(:which_configuration_file).returns(tmp_config)
         subject.set(key, value)
 
         # make sure subject.print looks at the newly modified settings
         test_settings = load_settings(tmp_config)
         # instead of the default Puppet.settings (implementation detail)
-        allow(Puppet).to receive(:settings).and_return(test_settings)
+        Puppet.stubs(:settings).returns(test_settings)
 
         result = subject.print()
         expect(render(:print, result)).to match(/^#{key} = #{value}$/)

data/spec/integration/faces/documentation_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 require 'puppet/face'
 

data/spec/integration/faces/plugin_spec.rb

@@ -46,7 +46,7 @@ describe "Puppet plugin face" do
   end
 
   def init_cli_args_and_apply_app(args = ["download"])
-    Puppet::Application.find(:plugin).new(double('command_line', :subcommand_name => :plugin, :args => args))
+    Puppet::Application.find(:plugin).new(stub('command_line', :subcommand_name => :plugin, :args => args))
   end
 
   it "processes a download request" do

data/spec/integration/file_bucket/file_spec.rb

@@ -1,3 +1,5 @@
+#! /usr/bin/env ruby
+
 require 'spec_helper'
 
 require 'puppet/file_bucket/file'
@@ -6,7 +8,7 @@ describe Puppet::FileBucket::File do
   describe "#indirection" do
     before :each do
       # Never connect to the network, no matter what
-      allow_any_instance_of(described_class.indirection.terminus(:rest).class).to receive(:find)
+      described_class.indirection.terminus(:rest).class.any_instance.stubs(:find)
     end
 
     describe "when running another application" do
@@ -15,7 +17,7 @@ describe Puppet::FileBucket::File do
         "filebucket://puppetmaster:8140/md5/d41d8cd98f00b204e9800998ecf8427e" => :rest,
       }.each do |key, terminus|
         it "should use the #{terminus} terminus when requesting #{key.inspect}" do
-          expect_any_instance_of(described_class.indirection.terminus(terminus).class).to receive(:find)
+          described_class.indirection.terminus(terminus).class.any_instance.expects(:find)
 
           described_class.indirection.find(key)
         end

data/spec/integration/file_serving/content_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/file_serving/content'

data/spec/integration/file_serving/fileset_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/file_serving/fileset'

data/spec/integration/file_serving/metadata_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/file_serving/metadata'

data/spec/integration/file_serving/terminus_helper_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/file_serving/terminus_helper'

data/spec/integration/indirector/catalog/compiler_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/resource/catalog'
@@ -6,7 +7,7 @@ Puppet::Resource::Catalog.indirection.terminus(:compiler)
 
 describe Puppet::Resource::Catalog::Compiler do
   before do
-    allow(Facter).to receive(:value).and_return("something")
+    Facter.stubs(:value).returns "something"
     @catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
     @catalog.add_resource(@one = Puppet::Resource.new(:file, "/one"))
     @catalog.add_resource(@two = Puppet::Resource.new(:file, "/two"))
@@ -31,8 +32,8 @@ describe Puppet::Resource::Catalog::Compiler do
   it "should filter out virtual resources when finding a catalog" do
     Puppet[:node_terminus] = :memory
     Puppet::Node.indirection.save(Puppet::Node.new("mynode"))
-    allow(Puppet::Resource::Catalog.indirection.terminus).to receive(:extract_facts_from_request)
-    allow(Puppet::Resource::Catalog.indirection.terminus).to receive(:compile).and_return(@catalog)
+    Puppet::Resource::Catalog.indirection.terminus.stubs(:extract_facts_from_request)
+    Puppet::Resource::Catalog.indirection.terminus.stubs(:compile).returns(@catalog)
 
     @one.virtual = true
 
@@ -42,8 +43,8 @@ describe Puppet::Resource::Catalog::Compiler do
   it "should not filter out exported resources when finding a catalog" do
     Puppet[:node_terminus] = :memory
     Puppet::Node.indirection.save(Puppet::Node.new("mynode"))
-    allow(Puppet::Resource::Catalog.indirection.terminus).to receive(:extract_facts_from_request)
-    allow(Puppet::Resource::Catalog.indirection.terminus).to receive(:compile).and_return(@catalog)
+    Puppet::Resource::Catalog.indirection.terminus.stubs(:extract_facts_from_request)
+    Puppet::Resource::Catalog.indirection.terminus.stubs(:compile).returns(@catalog)
 
     @one.exported = true
 
@@ -53,8 +54,8 @@ describe Puppet::Resource::Catalog::Compiler do
   it "should filter out virtual exported resources when finding a catalog" do
     Puppet[:node_terminus] = :memory
     Puppet::Node.indirection.save(Puppet::Node.new("mynode"))
-    allow(Puppet::Resource::Catalog.indirection.terminus).to receive(:extract_facts_from_request)
-    allow(Puppet::Resource::Catalog.indirection.terminus).to receive(:compile).and_return(@catalog)
+    Puppet::Resource::Catalog.indirection.terminus.stubs(:extract_facts_from_request)
+    Puppet::Resource::Catalog.indirection.terminus.stubs(:compile).returns(@catalog)
 
     @one.exported = true
     @one.virtual = true
@@ -66,10 +67,10 @@ describe Puppet::Resource::Catalog::Compiler do
     Puppet[:node_terminus] = :memory
     Puppet::Node.indirection.save(Puppet::Node.new("mynode"))
 
-    catalog_environment = nil
-    expect_any_instance_of(Puppet::Parser::Resource::Catalog).to receive(:to_resource) {catalog_environment = Puppet.lookup(:current_environment).name}
+    Puppet::Parser::Resource::Catalog.any_instance.expects(:to_resource).with do |catalog|
+      expect(Puppet.lookup(:current_environment).name).to eq(:production)
+    end
 
     Puppet::Resource::Catalog.indirection.find("mynode")
-    expect(catalog_environment).to eq(:production)
   end
 end

data/spec/integration/indirector/direct_file_server_spec.rb

@@ -14,7 +14,7 @@ describe Puppet::Indirector::DirectFileServer, " when interacting with the files
 
   it "should return an instance of the model" do
     filepath = make_absolute("/path/to/my/file")
-    expect(Puppet::FileSystem).to receive(:exist?).with(filepath).and_return(true)
+    Puppet::FileSystem.expects(:exist?).with(filepath).returns(true)
 
     expect(@terminus.find(@terminus.indirection.request(:find, Puppet::Util.path_to_uri(filepath).to_s, nil))).to be_instance_of(Puppet::FileServing::Content)
   end

data/spec/integration/indirector/facts/facter_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 require 'puppet_spec/files'
 require 'puppet_spec/compiler'
@@ -14,7 +15,7 @@ describe Puppet::Node::Facts::Facter do
       end
     end
 
-    allow(Facter).to receive(:reset)
+    Facter.stubs(:reset)
 
     cat = compile_to_catalog('notify { $downcase_test: }',
                              Puppet::Node.indirection.find('foo'))
@@ -37,7 +38,7 @@ describe Puppet::Node::Facts::Facter do
       EOF
 
       Puppet.initialize_settings(['--modulepath', factdir])
-      apply = Puppet::Application.find(:apply).new(double('command_line', :subcommand_name => :apply, :args => ['--modulepath', factdir, '-e', 'notify { $custom: }']))
+      apply = Puppet::Application.find(:apply).new(stub('command_line', :subcommand_name => :apply, :args => ['--modulepath', factdir, '-e', 'notify { $custom: }']))
 
       expect do
         expect { apply.run }.to exit_with(0)
@@ -52,7 +53,7 @@ describe Puppet::Node::Facts::Facter do
         EOF
 
       Puppet.initialize_settings(['--pluginfactdest', factdir])
-      apply = Puppet::Application.find(:apply).new(double('command_line', :subcommand_name => :apply, :args => ['--pluginfactdest', factdir, '-e', 'notify { $foo: }']))
+      apply = Puppet::Application.find(:apply).new(stub('command_line', :subcommand_name => :apply, :args => ['--pluginfactdest', factdir, '-e', 'notify { $foo: }']))
 
       expect do
         expect { apply.run }.to exit_with(0)
@@ -61,7 +62,7 @@ describe Puppet::Node::Facts::Facter do
   end
 
   it "adds the puppetversion fact" do
-    allow(Facter).to receive(:reset)
+    Facter.stubs(:reset)
 
     cat = compile_to_catalog('notify { $::puppetversion: }',
                              Puppet::Node.indirection.find('foo'))

data/spec/integration/indirector/file_content/file_server_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/indirector/file_content/file_server'
@@ -74,8 +75,8 @@ describe Puppet::Indirector::FileContent::FileServer, " when finding files" do
   end
 
   it "should find file content in files when node name expansions are used" do
-    allow(Puppet::FileSystem).to receive(:exist?).and_return(true)
-    allow(Puppet::FileSystem).to receive(:exist?).with(Puppet[:fileserverconfig]).and_return(true)
+    Puppet::FileSystem.stubs(:exist?).returns true
+    Puppet::FileSystem.stubs(:exist?).with(Puppet[:fileserverconfig]).returns(true)
 
     path = tmpfile("file_server_testing")
 
@@ -86,14 +87,14 @@ describe Puppet::Indirector::FileContent::FileServer, " when finding files" do
 
     # Use a real mount, so the integration is a bit deeper.
     mount1 = Puppet::FileServing::Configuration::Mount::File.new("one")
-    allow(mount1).to receive(:globalallow?).and_return(true)
-    allow(mount1).to receive(:allowed?).and_return(true)
+    mount1.stubs(:globalallow?).returns true
+    mount1.stubs(:allowed?).returns true
     mount1.path = File.join(path, "%h")
 
-    parser = double('parser', :changed? => false)
-    allow(parser).to receive(:parse).and_return("one" => mount1)
+    parser = stub 'parser', :changed? => false
+    parser.stubs(:parse).returns("one" => mount1)
 
-    allow(Puppet::FileServing::Configuration::Parser).to receive(:new).and_return(parser)
+    Puppet::FileServing::Configuration::Parser.stubs(:new).returns(parser)
 
     path = File.join(path, "myfile")
 

data/spec/integration/indirector/file_metadata/file_server_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/indirector/file_metadata/file_server'
@@ -54,19 +55,19 @@ describe Puppet::Indirector::FileMetadata::FileServer, " when finding files" do
   describe "when node name expansions are used" do
     with_checksum_types("file_server_testing", "mynode/myfile") do
       it "should return the correct metadata" do
-        allow(Puppet::FileSystem).to receive(:exist?).with(checksum_file).and_return(true)
-        allow(Puppet::FileSystem).to receive(:exist?).with(Puppet[:fileserverconfig]).and_return(true)
+        Puppet::FileSystem.stubs(:exist?).with(checksum_file).returns true
+        Puppet::FileSystem.stubs(:exist?).with(Puppet[:fileserverconfig]).returns(true)
 
         # Use a real mount, so the integration is a bit deeper.
         mount1 = Puppet::FileServing::Configuration::Mount::File.new("one")
-        allow(mount1).to receive(:globalallow?).and_return(true)
-        allow(mount1).to receive(:allowed?).and_return(true)
+        mount1.stubs(:globalallow?).returns true
+        mount1.stubs(:allowed?).returns true
         mount1.path = File.join(env_path, "%h")
 
-        parser = double('parser', :changed? => false)
-        allow(parser).to receive(:parse).and_return("one" => mount1)
+        parser = stub 'parser', :changed? => false
+        parser.stubs(:parse).returns("one" => mount1)
 
-        allow(Puppet::FileServing::Configuration::Parser).to receive(:new).and_return(parser)
+        Puppet::FileServing::Configuration::Parser.stubs(:new).returns(parser)
         env = Puppet::Node::Environment.create(:foo, [])
 
         result = Puppet::FileServing::Metadata.indirection.find("one/myfile", :environment => env, :node => "mynode", :checksum_type => checksum_type)

data/spec/integration/network/authconfig_spec.rb

@@ -3,7 +3,8 @@ require 'spec_helper'
 require 'puppet/network/authconfig'
 require 'puppet/network/auth_config_parser'
 
-RSpec::Matchers.define :auth_allow do |params|
+RSpec::Matchers.define :allow do |params|
+
   match do |auth|
     begin
       auth.check_authorization(*params)
@@ -60,7 +61,7 @@ describe Puppet::Network::AuthConfig do
     it "should not match IP addresses" do
       add_rule("allow 10.1.1.1")
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
     end
 
     it "should not accept CIDR IPv4 address" do
@@ -84,13 +85,13 @@ describe Puppet::Network::AuthConfig do
     it "should support hostname" do
       add_rule("allow host.domain.com")
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should support wildcard host" do
       add_rule("allow *.domain.com")
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it 'should warn about missing path before allow_ip in stanza' do
@@ -108,51 +109,51 @@ describe Puppet::Network::AuthConfig do
     it "should support hostname backreferences" do
       add_regex_rule('^/test/([^/]+)$', "allow $1.domain.com")
 
-      expect(@auth).to auth_allow(request(:key => 'host'))
+      expect(@auth).to allow(request(:key => 'host'))
     end
 
     it "should support opaque strings" do
       add_rule("allow this-is-opaque@or-not")
 
-      expect(@auth).to auth_allow(request(:node => 'this-is-opaque@or-not'))
+      expect(@auth).to allow(request(:node => 'this-is-opaque@or-not'))
     end
 
     it "should support opaque strings and backreferences" do
       add_regex_rule('^/test/([^/]+)$', "allow $1")
 
-      expect(@auth).to auth_allow(request(:key => 'this-is-opaque@or-not', :node => 'this-is-opaque@or-not'))
+      expect(@auth).to allow(request(:key => 'this-is-opaque@or-not', :node => 'this-is-opaque@or-not'))
     end
 
     it "should support hostname ending with '.'" do
       pending('bug #7589')
       add_rule("allow host.domain.com.")
 
-      expect(@auth).to auth_allow(request(:node => 'host.domain.com.'))
+      expect(@auth).to allow(request(:node => 'host.domain.com.'))
     end
 
     it "should support hostname ending with '.' and backreferences" do
       pending('bug #7589')
       add_regex_rule('^/test/([^/]+)$',"allow $1")
 
-      expect(@auth).to auth_allow(request(:node => 'host.domain.com.'))
+      expect(@auth).to allow(request(:node => 'host.domain.com.'))
     end
 
     it "should support trailing whitespace" do
       add_rule('allow host.domain.com    ')
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should support inlined comments" do
       add_rule('allow host.domain.com # will it work?')
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should deny non-matching host" do
       add_rule("allow inexistent")
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
     end
   end
 
@@ -160,7 +161,7 @@ describe Puppet::Network::AuthConfig do
     it "should not warn when matches against IP addresses fail" do
       add_rule("allow_ip 10.1.1.2")
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
 
       expect(@logs).not_to be_any {|log| log.level == :warning and log.message =~ /Authentication based on IP address is deprecated/}
     end
@@ -168,25 +169,25 @@ describe Puppet::Network::AuthConfig do
     it "should support IPv4 address" do
       add_rule("allow_ip 10.1.1.1")
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should support CIDR IPv4 address" do
       add_rule("allow_ip 10.0.0.0/8")
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should support wildcard IPv4 address" do
       add_rule("allow_ip 10.1.1.*")
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should support IPv6 address" do
       add_rule("allow_ip 2001:DB8::8:800:200C:417A")
 
-      expect(@auth).to auth_allow(request(:ip => '2001:DB8::8:800:200C:417A'))
+      expect(@auth).to allow(request(:ip => '2001:DB8::8:800:200C:417A'))
     end
 
     it "should support hostname" do
@@ -203,7 +204,7 @@ describe Puppet::Network::AuthConfig do
         allow *.domain.com
       EOALLOWRULE
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
     end
 
     it "denies denied hosts after allowing them" do
@@ -212,7 +213,7 @@ describe Puppet::Network::AuthConfig do
         deny host.domain.com
       EOALLOWRULE
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
     end
 
     it "should not deny based on IP" do
@@ -221,7 +222,7 @@ describe Puppet::Network::AuthConfig do
         allow host.domain.com
       EOALLOWRULE
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
 
     it "should not deny based on IP (ordering #2)" do
@@ -230,7 +231,7 @@ describe Puppet::Network::AuthConfig do
         deny 10.1.1.1
       EOALLOWRULE
 
-      expect(@auth).to auth_allow(request)
+      expect(@auth).to allow(request)
     end
   end
 
@@ -241,7 +242,7 @@ describe Puppet::Network::AuthConfig do
         allow host.domain.com
       EOALLOWRULE
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
     end
 
     it "should deny based on IP (ordering #2)" do
@@ -250,7 +251,7 @@ describe Puppet::Network::AuthConfig do
         deny_ip 10.1.1.1
       EOALLOWRULE
 
-      expect(@auth).not_to auth_allow(request)
+      expect(@auth).not_to allow(request)
     end
   end
 end