puppet 6.0.10 → 6.1.0

data/spec/unit/application/apply_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 require 'puppet/application/apply'
@@ -10,7 +11,7 @@ describe Puppet::Application::Apply do
 
   before :each do
     @apply = Puppet::Application[:apply]
-    allow(Puppet::Util::Log).to receive(:newdestination)
+    Puppet::Util::Log.stubs(:newdestination)
     Puppet[:reports] = "none"
   end
 
@@ -28,25 +29,26 @@ describe Puppet::Application::Apply do
     end
 
     it "should store argument value when calling handle_#{option}" do
-      expect(@apply.options).to receive(:[]=).with(option, 'arg')
+      @apply.options.expects(:[]=).with(option, 'arg')
       @apply.send("handle_#{option}".to_sym, 'arg')
     end
   end
 
   it "should set the code to the provided code when :execute is used" do
-    expect(@apply.options).to receive(:[]=).with(:code, 'arg')
+    @apply.options.expects(:[]=).with(:code, 'arg')
     @apply.send("handle_execute".to_sym, 'arg')
   end
 
   describe "when applying options" do
+
     it "should set the log destination with --logdest" do
-      expect(Puppet::Log).to receive(:newdestination).with("console")
+      Puppet::Log.expects(:newdestination).with("console")
 
       @apply.handle_logdest("console")
     end
 
     it "should set the setdest options to true" do
-      expect(@apply.options).to receive(:[]=).with(:setdest,true)
+      @apply.options.expects(:[]=).with(:setdest,true)
 
       @apply.handle_logdest("console")
     end
@@ -54,16 +56,16 @@ describe Puppet::Application::Apply do
 
   describe "during setup" do
     before :each do
-      allow(Puppet::Log).to receive(:newdestination)
-      allow(Puppet::FileBucket::Dipper).to receive(:new)
-      allow(STDIN).to receive(:read)
-      allow(Puppet::Transaction::Report.indirection).to receive(:cache_class=)
+      Puppet::Log.stubs(:newdestination)
+      Puppet::FileBucket::Dipper.stubs(:new)
+      STDIN.stubs(:read)
+      Puppet::Transaction::Report.indirection.stubs(:cache_class=)
     end
 
     describe "with --test" do
       it "should call setup_test" do
         @apply.options[:test] = true
-        expect(@apply).to receive(:setup_test)
+        @apply.expects(:setup_test)
 
         @apply.setup
       end
@@ -86,20 +88,20 @@ describe Puppet::Application::Apply do
     end
 
     it "should set console as the log destination if logdest option wasn't provided" do
-      expect(Puppet::Log).to receive(:newdestination).with(:console)
+      Puppet::Log.expects(:newdestination).with(:console)
 
       @apply.setup
     end
 
     it "sets the log destination if logdest is provided via settings" do
-      expect(Puppet::Log).to receive(:newdestination).with("set_via_config")
+      Puppet::Log.expects(:newdestination).with("set_via_config")
       Puppet[:logdest] = "set_via_config"
 
       @apply.setup
     end
 
     it "should set INT trap" do
-      expect(Signal).to receive(:trap).with(:INT)
+      Signal.expects(:trap).with(:INT)
 
       @apply.setup
     end
@@ -117,24 +119,25 @@ describe Puppet::Application::Apply do
     end
 
     it "should print puppet config if asked to in Puppet config" do
-      allow(Puppet.settings).to receive(:print_configs?).and_return(true)
-      expect(Puppet.settings).to receive(:print_configs).and_return(true)
+      Puppet.settings.stubs(:print_configs?).returns  true
+      Puppet.settings.expects(:print_configs).returns true
       expect { @apply.setup }.to exit_with 0
     end
 
     it "should exit after printing puppet config if asked to in Puppet config" do
-      allow(Puppet.settings).to receive(:print_configs?).and_return(true)
+      Puppet.settings.stubs(:print_configs?).returns(true)
       expect { @apply.setup }.to exit_with 1
     end
 
     it "should use :main, :puppetd, and :ssl" do
-      expect(Puppet.settings).to receive(:use).with(:main, :agent, :ssl)
+      Puppet.settings.unstub(:use)
+      Puppet.settings.expects(:use).with(:main, :agent, :ssl)
 
       @apply.setup
     end
 
     it "should tell the report handler to cache locally as yaml" do
-      expect(Puppet::Transaction::Report.indirection).to receive(:cache_class=).with(:yaml)
+      Puppet::Transaction::Report.indirection.expects(:cache_class=).with(:yaml)
 
       @apply.setup
     end
@@ -166,14 +169,14 @@ describe Puppet::Application::Apply do
     it "should dispatch to 'apply' if it was called with 'apply'" do
       @apply.options[:catalog] = "foo"
 
-      expect(@apply).to receive(:apply)
+      @apply.expects(:apply)
       @apply.run_command
     end
 
     it "should dispatch to main otherwise" do
-      allow(@apply).to receive(:options).and_return({})
+      @apply.stubs(:options).returns({})
 
-      expect(@apply).to receive(:main)
+      @apply.expects(:main)
       @apply.run_command
     end
 
@@ -194,17 +197,17 @@ describe Puppet::Application::Apply do
         Puppet::Node.indirection.save(@node)
 
         @catalog = Puppet::Resource::Catalog.new("testing", Puppet.lookup(:environments).get(Puppet[:environment]))
-        allow(@catalog).to receive(:to_ral).and_return(@catalog)
+        @catalog.stubs(:to_ral).returns(@catalog)
 
-        allow(Puppet::Resource::Catalog.indirection).to receive(:find).and_return(@catalog)
+        Puppet::Resource::Catalog.indirection.stubs(:find).returns(@catalog)
 
-        allow(STDIN).to receive(:read)
+        STDIN.stubs(:read)
 
-        @transaction = double('transaction')
-        allow(@catalog).to receive(:apply).and_return(@transaction)
+        @transaction = stub('transaction')
+        @catalog.stubs(:apply).returns(@transaction)
 
-        allow(Puppet::Util::Storage).to receive(:load)
-        allow_any_instance_of(Puppet::Configurer).to receive(:save_last_run_summary) # to prevent it from trying to write files
+        Puppet::Util::Storage.stubs(:load)
+        Puppet::Configurer.any_instance.stubs(:save_last_run_summary) # to prevent it from trying to write files
       end
 
       after :each do
@@ -221,23 +224,23 @@ describe Puppet::Application::Apply do
 
       it "should set the code to run from --code" do
         @apply.options[:code] = "code to run"
-        expect(Puppet).to receive(:[]=).with(:code,"code to run")
+        Puppet.expects(:[]=).with(:code,"code to run")
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should set the code to run from STDIN if no arguments" do
-        allow(@apply.command_line).to receive(:args).and_return([])
-        allow(STDIN).to receive(:read).and_return("code to run")
+        @apply.command_line.stubs(:args).returns([])
+        STDIN.stubs(:read).returns("code to run")
 
-        expect(Puppet).to receive(:[]=).with(:code,"code to run")
+        Puppet.expects(:[]=).with(:code,"code to run")
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should raise an error if a file is passed on command line and the file does not exist" do
         noexist = tmpfile('noexist.pp')
-        allow(@apply.command_line).to receive(:args).and_return([noexist])
+        @apply.command_line.stubs(:args).returns([noexist])
         expect { @apply.main }.to raise_error(RuntimeError, "Could not find file #{noexist}")
       end
 
@@ -245,7 +248,7 @@ describe Puppet::Application::Apply do
         manifest = tmpfile('starwarsIV')
         FileUtils.touch(manifest)
 
-        allow(@apply.command_line).to receive(:args).and_return([manifest, 'starwarsI', 'starwarsII'])
+        @apply.command_line.stubs(:args).returns([manifest, 'starwarsI', 'starwarsII'])
 
         expect { @apply.main }.to exit_with 0
 
@@ -255,13 +258,13 @@ describe Puppet::Application::Apply do
       end
 
       it "should splay" do
-        expect(@apply).to receive(:splay)
+        @apply.expects(:splay)
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should raise an error if we can't find the node" do
-        expect(Puppet::Node.indirection).to receive(:find).and_return(nil)
+        Puppet::Node.indirection.expects(:find).returns(nil)
 
         expect { @apply.main }.to raise_error(RuntimeError, /Could not find node/)
       end
@@ -272,68 +275,68 @@ describe Puppet::Application::Apply do
         File.open(classfile, 'w') { |c| c.puts 'class' }
         Puppet[:classfile] = classfile
 
-        expect(@node).to receive(:classes=).with(['class'])
+        @node.expects(:classes=).with(['class'])
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should compile the catalog" do
-        expect(Puppet::Resource::Catalog.indirection).to receive(:find).and_return(@catalog)
+        Puppet::Resource::Catalog.indirection.expects(:find).returns(@catalog)
 
         expect { @apply.main }.to exit_with 0
       end
 
       it 'should called the DeferredResolver to resolve any Deferred values' do
-        expect(Puppet::Pops::Evaluator::DeferredResolver).to receive(:resolve_and_replace).with(any_args)
+        Puppet::Pops::Evaluator::DeferredResolver.expects(:resolve_and_replace).with(any_parameters)
         expect { @apply.main }.to exit_with 0
       end
 
       it 'should make the Puppet::Pops::Loaders available when applying the compiled catalog' do
-        expect(Puppet::Resource::Catalog.indirection).to receive(:find).and_return(@catalog)
-        expect(@apply).to receive(:apply_catalog) do |catalog|
-          expect(@catalog).to eq(@catalog)
+        Puppet::Resource::Catalog.indirection.expects(:find).returns(@catalog)
+        @apply.expects(:apply_catalog).with(@catalog) do
           fail('Loaders not found') unless Puppet.lookup(:loaders) { nil }.is_a?(Puppet::Pops::Loaders)
           true
-        end.and_return(0)
+        end.returns(0)
         expect { @apply.main }.to exit_with 0
       end
 
       it "should transform the catalog to ral" do
-        expect(@catalog).to receive(:to_ral).and_return(@catalog)
+
+        @catalog.expects(:to_ral).returns(@catalog)
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should finalize the catalog" do
-        expect(@catalog).to receive(:finalize)
+        @catalog.expects(:finalize)
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should not save the classes or resource file by default" do
-        expect(@catalog).not_to receive(:write_class_file)
-        expect(@catalog).not_to receive(:write_resource_file)
+        @catalog.expects(:write_class_file).never
+        @catalog.expects(:write_resource_file).never
         expect { @apply.main }.to exit_with 0
       end
 
       it "should save the classes and resources files when requested" do
         @apply.options[:write_catalog_summary] = true
 
-        expect(@catalog).to receive(:write_class_file).once
-        expect(@catalog).to receive(:write_resource_file).once
+        @catalog.expects(:write_class_file).once
+        @catalog.expects(:write_resource_file).once
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should call the prerun and postrun commands on a Configurer instance" do
-        expect_any_instance_of(Puppet::Configurer).to receive(:execute_prerun_command).and_return(true)
-        expect_any_instance_of(Puppet::Configurer).to receive(:execute_postrun_command).and_return(true)
+        Puppet::Configurer.any_instance.expects(:execute_prerun_command).returns(true)
+        Puppet::Configurer.any_instance.expects(:execute_postrun_command).returns(true)
 
         expect { @apply.main }.to exit_with 0
       end
 
       it "should apply the catalog" do
-        expect(@catalog).to receive(:apply).and_return(double('transaction'))
+        @catalog.expects(:apply).returns(stub_everything('transaction'))
 
         expect { @apply.main }.to exit_with 0
       end
@@ -341,9 +344,9 @@ describe Puppet::Application::Apply do
       it "should save the last run summary" do
         Puppet[:noop] = false
         report = Puppet::Transaction::Report.new
-        allow(Puppet::Transaction::Report).to receive(:new).and_return(report)
+        Puppet::Transaction::Report.stubs(:new).returns(report)
 
-        expect_any_instance_of(Puppet::Configurer).to receive(:save_last_run_summary).with(report)
+        Puppet::Configurer.any_instance.expects(:save_last_run_summary).with(report)
         expect { @apply.main }.to exit_with 0
       end
 
@@ -375,7 +378,7 @@ describe Puppet::Application::Apply do
         end
 
         it "should raise an error if we can't find the facts" do
-          expect(Puppet::Node::Facts.indirection).to receive(:find).and_return(nil)
+          Puppet::Node::Facts.indirection.expects(:find).returns(nil)
 
           expect { @apply.main }.to raise_error(RuntimeError, /Could not find facts/)
         end
@@ -388,30 +391,30 @@ describe Puppet::Application::Apply do
 
         it "should exit with report's computed exit status" do
           Puppet[:noop] = false
-          allow_any_instance_of(Puppet::Transaction::Report).to receive(:exit_status).and_return(666)
+          Puppet::Transaction::Report.any_instance.stubs(:exit_status).returns(666)
 
           expect { @apply.main }.to exit_with 666
         end
 
         it "should exit with report's computed exit status, even if --noop is set" do
           Puppet[:noop] = true
-          allow_any_instance_of(Puppet::Transaction::Report).to receive(:exit_status).and_return(666)
+          Puppet::Transaction::Report.any_instance.stubs(:exit_status).returns(666)
 
           expect { @apply.main }.to exit_with 666
         end
 
         it "should always exit with 0 if option is disabled" do
           Puppet[:noop] = false
-          report = double('report', :exit_status => 666)
-          allow(@transaction).to receive(:report).and_return(report)
+          report = stub 'report', :exit_status => 666
+          @transaction.stubs(:report).returns(report)
 
           expect { @apply.main }.to exit_with 0
         end
 
         it "should always exit with 0 if --noop" do
           Puppet[:noop] = true
-          report = double('report', :exit_status => 666)
-          allow(@transaction).to receive(:report).and_return(report)
+          report = stub 'report', :exit_status => 666
+          @transaction.stubs(:report).returns(report)
 
           expect { @apply.main }.to exit_with 0
         end
@@ -432,23 +435,23 @@ describe Puppet::Application::Apply do
       it "should read the catalog in from disk if a file name is provided" do
         @apply.options[:catalog] = temporary_catalog
         catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
-        allow(Puppet::Resource::Catalog).to receive(:convert_from).with(default_format, '"something"').and_return(catalog)
+        Puppet::Resource::Catalog.stubs(:convert_from).with(default_format, '"something"').returns(catalog)
         @apply.apply
       end
 
       it "should read the catalog in from stdin if '-' is provided" do
         @apply.options[:catalog] = "-"
-        expect($stdin).to receive(:read).and_return('"something"')
+        $stdin.expects(:read).returns '"something"'
         catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
-        allow(Puppet::Resource::Catalog).to receive(:convert_from).with(default_format, '"something"').and_return(catalog)
+        Puppet::Resource::Catalog.stubs(:convert_from).with(default_format, '"something"').returns(catalog)
         @apply.apply
       end
 
       it "should deserialize the catalog from the default format" do
         @apply.options[:catalog] = temporary_catalog
-        allow(Puppet::Resource::Catalog).to receive(:default_format).and_return(:rot13_piglatin)
+        Puppet::Resource::Catalog.stubs(:default_format).returns :rot13_piglatin
         catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
-        allow(Puppet::Resource::Catalog).to receive(:convert_from).with(:rot13_piglatin,'"something"').and_return(catalog)
+        Puppet::Resource::Catalog.stubs(:convert_from).with(:rot13_piglatin,'"something"').returns(catalog)
         @apply.apply
       end
 
@@ -460,12 +463,12 @@ describe Puppet::Application::Apply do
       it "should convert the catalog to a RAL catalog and use a Configurer instance to apply it" do
         @apply.options[:catalog] = temporary_catalog
         catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
-        allow(Puppet::Resource::Catalog).to receive(:convert_from).with(default_format, '"something"').and_return(catalog)
-        expect(catalog).to receive(:to_ral).and_return("mycatalog")
+        Puppet::Resource::Catalog.stubs(:convert_from).with(default_format, '"something"').returns catalog
+        catalog.expects(:to_ral).returns "mycatalog"
 
-        configurer = double('configurer')
-        expect(Puppet::Configurer).to receive(:new).and_return(configurer)
-        expect(configurer).to receive(:run).
+        configurer = stub 'configurer'
+        Puppet::Configurer.expects(:new).returns configurer
+        configurer.expects(:run).
           with(:catalog => "mycatalog", :pluginsync => false)
 
         @apply.apply
@@ -474,13 +477,11 @@ describe Puppet::Application::Apply do
       it 'should make the Puppet::Pops::Loaders available when applying a catalog' do
         @apply.options[:catalog] = temporary_catalog
         catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
-        expect(@apply).to receive(:read_catalog) do |arg|
-          expect(arg).to eq('"something"')
+        @apply.expects(:read_catalog).with('something') do
           fail('Loaders not found') unless Puppet.lookup(:loaders) { nil }.is_a?(Puppet::Pops::Loaders)
           true
-        end.and_return(catalog)
-        expect(@apply).to receive(:apply_catalog) do |cat|
-          expect(cat).to eq(catalog)
+        end.returns(catalog)
+        @apply.expects(:apply_catalog).with(catalog) do
           fail('Loaders not found') unless Puppet.lookup(:loaders) { nil }.is_a?(Puppet::Pops::Loaders)
           true
         end
@@ -489,10 +490,10 @@ describe Puppet::Application::Apply do
 
       it "should call the DeferredResolver to resolve Deferred values" do
         @apply.options[:catalog] = temporary_catalog
-        allow(Puppet::Resource::Catalog).to receive(:default_format).and_return(:rot13_piglatin)
+        Puppet::Resource::Catalog.stubs(:default_format).returns :rot13_piglatin
         catalog = Puppet::Resource::Catalog.new("testing", Puppet::Node::Environment::NONE)
-        allow(Puppet::Resource::Catalog).to receive(:convert_from).with(:rot13_piglatin, '"something"').and_return(catalog)
-        expect(Puppet::Pops::Evaluator::DeferredResolver).to receive(:resolve_and_replace).with(any_args)
+        Puppet::Resource::Catalog.stubs(:convert_from).with(:rot13_piglatin,'"something"').returns(catalog)
+        Puppet::Pops::Evaluator::DeferredResolver.expects(:resolve_and_replace).with(any_parameters)
         @apply.apply
       end
     end
@@ -519,7 +520,7 @@ describe Puppet::Application::Apply do
       Puppet.settings[:classfile] = classfile
 
       #We don't actually need the resource to do anything, we are using it's properties in other parts of the workflow.
-      allow(Puppet::Util::Execution).to receive(:execute)
+      Puppet::Util::Execution.stubs(:execute)
 
       expect { @apply.main }.to exit_with 0
 
@@ -533,7 +534,7 @@ describe Puppet::Application::Apply do
   describe "apply_catalog" do
     it "should call the configurer with the catalog" do
       catalog = "I am a catalog"
-      expect_any_instance_of(Puppet::Configurer).to receive(:run).
+      Puppet::Configurer.any_instance.expects(:run).
         with(:catalog => catalog, :pluginsync => false)
       @apply.send(:apply_catalog, catalog)
     end
@@ -541,7 +542,7 @@ describe Puppet::Application::Apply do
 
   it "should honor the catalog_cache_terminus setting" do
     Puppet.settings[:catalog_cache_terminus] = "json"
-    expect(Puppet::Resource::Catalog.indirection).to receive(:cache_class=).with(:json)
+    Puppet::Resource::Catalog.indirection.expects(:cache_class=).with(:json)
 
     @apply.initialize_app_defaults
     @apply.setup