puppet 7.28.0-universal-darwin → 7.29.1-universal-darwin

data/spec/unit/application/apply_spec.rb

@@ -1,559 +0,0 @@
-require 'spec_helper'
-
-require 'puppet/application/apply'
-require 'puppet/file_bucket/dipper'
-require 'puppet/configurer'
-require 'fileutils'
-
-describe Puppet::Application::Apply do
-  include PuppetSpec::Files
-
-  before :each do
-    @apply = Puppet::Application[:apply]
-    Puppet[:reports] = "none"
-  end
-
-  [:debug,:loadclasses,:test,:verbose,:use_nodes,:detailed_exitcodes,:catalog].each do |option|
-    it "should store argument value when calling handle_#{option}" do
-      expect(@apply.options).to receive(:[]=).with(option, 'arg')
-      @apply.send("handle_#{option}".to_sym, 'arg')
-    end
-  end
-
-  it "should handle write_catalog_summary" do
-    @apply.send(:handle_write_catalog_summary, true)
-
-    expect(Puppet[:write_catalog_summary]).to eq(true)
-  end
-
-  it "should set the code to the provided code when :execute is used" do
-    expect(@apply.options).to receive(:[]=).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")
-
-      @apply.handle_logdest("console")
-    end
-
-    it "should set the setdest options to true" do
-      expect(@apply.options).to receive(:[]=).with(:setdest,true)
-
-      @apply.handle_logdest("console")
-    end
-  end
-
-  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=)
-    end
-
-    describe "with --test" do
-      it "should set options[:verbose] to true" do
-        @apply.setup_test
-
-        expect(@apply.options[:verbose]).to eq(true)
-      end
-
-      it "should set options[:show_diff] to true" do
-        Puppet.settings.override_default(:show_diff, false)
-        @apply.setup_test
-        expect(Puppet[:show_diff]).to eq(true)
-      end
-
-      it "should set options[:detailed_exitcodes] to true" do
-        @apply.setup_test
-
-        expect(@apply.options[:detailed_exitcodes]).to eq(true)
-      end
-    end
-
-    it "should set console as the log destination if logdest option wasn't provided" do
-      expect(Puppet::Log).to receive(: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[:logdest] = "set_via_config"
-
-      @apply.setup
-    end
-
-    it "should set INT trap" do
-      expect(Signal).to receive(:trap).with(:INT)
-
-      @apply.setup
-    end
-
-    it "should set log level to debug if --debug was passed" do
-      @apply.options[:debug] = true
-      @apply.setup
-      expect(Puppet::Log.level).to eq(:debug)
-    end
-
-    it "should set log level to info if --verbose was passed" do
-      @apply.options[:verbose] = true
-      @apply.setup
-      expect(Puppet::Log.level).to eq(:info)
-    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)
-      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)
-      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)
-
-      @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)
-
-      @apply.setup
-    end
-
-    it "configures a profiler when profiling is enabled" do
-      Puppet[:profile] = true
-
-      @apply.setup
-
-      expect(Puppet::Util::Profiler.current).to satisfy do |ps|
-        ps.any? {|p| p.is_a? Puppet::Util::Profiler::WallClock }
-      end
-    end
-
-    it "does not have a profiler if profiling is disabled" do
-      Puppet[:profile] = false
-
-      @apply.setup
-
-      expect(Puppet::Util::Profiler.current.length).to be 0
-    end
-
-    it "should set default_file_terminus to `file_server` to be local" do
-      expect(@apply.app_defaults[:default_file_terminus]).to eq(:file_server)
-    end
-  end
-
-  describe "when executing" do
-    it "should dispatch to 'apply' if it was called with a catalog" do
-      @apply.options[:catalog] = "foo"
-
-      expect(@apply).to receive(:apply)
-      @apply.run_command
-    end
-
-    it "should dispatch to main otherwise" do
-      allow(@apply).to receive(:options).and_return({})
-
-      expect(@apply).to receive(:main)
-      @apply.run_command
-    end
-
-    describe "the main command" do
-      before :each do
-        Puppet[:prerun_command] = ''
-        Puppet[:postrun_command] = ''
-
-        Puppet::Node.indirection.terminus_class = :memory
-        Puppet::Node.indirection.cache_class = :memory
-
-        facts = Puppet::Node::Facts.new(Puppet[:node_name_value])
-        Puppet::Node::Facts.indirection.save(facts)
-
-        @node = Puppet::Node.new(Puppet[:node_name_value])
-        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)
-
-        allow(Puppet::Resource::Catalog.indirection).to receive(:find).and_return(@catalog)
-
-        allow(STDIN).to receive(:read)
-
-        @transaction = double('transaction')
-        allow(@catalog).to receive(:apply).and_return(@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
-      end
-
-      after :each do
-        Puppet::Node::Facts.indirection.reset_terminus_class
-        Puppet::Node::Facts.indirection.cache_class = nil
-      end
-
-      around :each do |example|
-        Puppet.override(:current_environment =>
-                        Puppet::Node::Environment.create(:production, [])) do
-          example.run
-        end
-      end
-
-      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")
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should set the code to run from STDIN if no arguments" do
-        @apply.command_line.args = []
-        allow(STDIN).to receive(:read).and_return("code to run")
-
-        expect(Puppet).to receive(:[]=).with(:code,"code to run")
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      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')
-        @apply.command_line.args << noexist
-        expect {
-          @apply.run
-        }.to exit_with(1)
-         .and output(anything).to_stdout
-         .and output(/Could not find file #{noexist}/).to_stderr
-      end
-
-      it "should set the manifest to the first file and warn other files will be skipped" do
-        manifest = tmpfile('starwarsIV')
-        FileUtils.touch(manifest)
-
-        @apply.command_line.args << manifest << 'starwarsI' << 'starwarsII'
-        expect {
-          @apply.run
-        }.to exit_with(0)
-         .and output(anything).to_stdout
-         .and output(/Warning: Only one file can be applied per run.  Skipping starwarsI, starwarsII/).to_stderr
-      end
-
-      it "should splay" do
-        expect(@apply).to receive(:splay)
-
-        expect {
-          @apply.run
-        }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should exit with 1 if we can't find the node" do
-        expect(Puppet::Node.indirection).to receive(:find).and_return(nil)
-
-        expect { @apply.run }.to exit_with(1).and output(/Could not find node/).to_stderr
-      end
-
-      it "should load custom classes if loadclasses" do
-        @apply.options[:loadclasses] = true
-        classfile = tmpfile('classfile')
-        File.open(classfile, 'w') { |c| c.puts 'class' }
-        Puppet[:classfile] = classfile
-
-        expect(@node).to receive(:classes=).with(['class'])
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should compile the catalog" do
-        expect(Puppet::Resource::Catalog.indirection).to receive(:find).and_return(@catalog)
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      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)
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      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)
-          fail('Loaders not found') unless Puppet.lookup(:loaders) { nil }.is_a?(Puppet::Pops::Loaders)
-          true
-        end.and_return(0)
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should transform the catalog to ral" do
-        expect(@catalog).to receive(:to_ral).and_return(@catalog)
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should finalize the catalog" do
-        expect(@catalog).to receive(:finalize)
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      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)
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should save the classes and resources files when requested on the command line using dashes" do
-        expect(@catalog).to receive(:write_class_file).once
-        expect(@catalog).to receive(:write_resource_file).once
-
-        # dashes are parsed by the application's OptionParser
-        @apply.command_line.args = ['--write-catalog-summary']
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should save the classes and resources files when requested on the command line using underscores" do
-        expect(@catalog).to receive(:write_class_file).once
-        expect(@catalog).to receive(:write_resource_file).once
-
-        # underscores are parsed by the settings PuppetOptionParser
-        @apply.command_line.args = ['--write_catalog_summary']
-        Puppet.initialize_settings(['--write_catalog_summary'])
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should save the classes and resources files when specified as a setting" do
-        Puppet[:write_catalog_summary] = true
-
-        expect(@catalog).to receive(:write_class_file).once
-        expect(@catalog).to receive(:write_resource_file).once
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      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)
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      it "should apply the catalog" do
-        expect(@catalog).to receive(:apply).and_return(double('transaction'))
-
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      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)
-
-        expect_any_instance_of(Puppet::Configurer).to receive(:save_last_run_summary).with(report)
-        expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-      end
-
-      describe "when using node_name_fact" do
-        before :each do
-          @facts = Puppet::Node::Facts.new(Puppet[:node_name_value], 'my_name_fact' => 'other_node_name')
-          Puppet::Node::Facts.indirection.save(@facts)
-          @node = Puppet::Node.new('other_node_name')
-          Puppet::Node.indirection.save(@node)
-          Puppet[:node_name_fact] = 'my_name_fact'
-        end
-
-        it "should set the facts name based on the node_name_fact" do
-          expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-          expect(@facts.name).to eq('other_node_name')
-        end
-
-        it "should set the node_name_value based on the node_name_fact" do
-          expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-          expect(Puppet[:node_name_value]).to eq('other_node_name')
-        end
-
-        it "should merge in our node the loaded facts" do
-          @facts.values.merge!('key' => 'value')
-
-          expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-
-          expect(@node.parameters['key']).to eq('value')
-        end
-
-        it "should exit if we can't find the facts" do
-          expect(Puppet::Node::Facts.indirection).to receive(:find).and_return(nil)
-
-          expect { @apply.run }.to exit_with(1).and output(/Could not find facts/).to_stderr
-        end
-      end
-
-      describe "with detailed_exitcodes" do
-        before :each do
-          @apply.options[:detailed_exitcodes] = true
-        end
-
-        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)
-
-          expect { @apply.run }.to exit_with(666).and output(anything).to_stdout
-        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)
-
-          expect { @apply.run }.to exit_with(666).and output(anything).to_stdout
-        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)
-
-          expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-        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)
-
-          expect { @apply.run }.to exit_with(0).and output(anything).to_stdout
-        end
-      end
-    end
-
-    describe "the 'apply' command" do
-      # We want this memoized, and to be able to adjust the content, so we
-      # have to do it ourselves.
-      def temporary_catalog(content = '"something"')
-        @tempfile = Tempfile.new('catalog.json')
-        @tempfile.write(content)
-        @tempfile.close
-        @tempfile.path
-      end
-
-      let(:default_format) { Puppet::Resource::Catalog.default_format }
-      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)
-        @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"')
-        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)
-        @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)
-        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)
-        @apply.apply
-      end
-
-      it "should fail helpfully if deserializing fails" do
-        @apply.options[:catalog] = temporary_catalog('something syntactically invalid')
-        expect { @apply.apply }.to raise_error(Puppet::Error)
-      end
-
-      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")
-
-        configurer = double('configurer')
-        expect(Puppet::Configurer).to receive(:new).and_return(configurer)
-        expect(configurer).to receive(:run).
-          with(:catalog => "mycatalog", :pluginsync => false)
-
-        @apply.apply
-      end
-
-      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"')
-          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)
-          fail('Loaders not found') unless Puppet.lookup(:loaders) { nil }.is_a?(Puppet::Pops::Loaders)
-          true
-        end
-        expect { @apply.apply }.not_to raise_error
-      end
-
-      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)
-        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)
-        @apply.apply
-      end
-    end
-  end
-
-  describe "when really executing" do
-    let(:testfile) { tmpfile('secret_file_name') }
-    let(:resourcefile) { tmpfile('resourcefile') }
-    let(:classfile) { tmpfile('classfile') }
-
-    it "should not expose sensitive data in the relationship file" do
-      @apply.options[:code] = <<-CODE
-        $secret = Sensitive('cat #{testfile}')
-
-        exec { 'do it':
-          command => $secret,
-          path    => '/bin/'
-        }
-      CODE
-
-      Puppet.settings[:write_catalog_summary] = true
-      Puppet.settings[:resourcefile] = resourcefile
-      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_any_instance_of(Puppet::Type.type(:exec).defaultprovider).to receive(:which).and_return('cat')
-      allow(Puppet::Util::Execution).to receive(:execute).and_return(double(exitstatus: 0, output: ''))
-
-      expect { @apply.run }.to exit_with(0).and output(%r{Exec\[do it\]/returns: executed successfully}).to_stdout
-      result = File.read(resourcefile)
-
-      expect(result).not_to match(/secret_file_name/)
-      expect(result).to match(/do it/)
-    end
-  end
-
-  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).
-        with(:catalog => catalog, :pluginsync => false)
-      @apply.send(:apply_catalog, catalog)
-    end
-  end
-
-  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)
-
-    @apply.initialize_app_defaults
-    @apply.setup
-  end
-end