puppet 6.0.10-x86-mingw32 → 6.1.0-x86-mingw32

data/spec/unit/environments_spec.rb

@@ -266,7 +266,7 @@ config_version=relative/script
       end
 
       it "interprets glob modulepaths from the environment's directory" do
-        allow(Dir).to receive(:glob).with(File.join(envdir, 'env1', 'other', '*', 'modules')).and_return([
+        Dir.stubs(:glob).with(File.join(envdir, 'env1', 'other', '*', 'modules')).returns([
           File.join(envdir, 'env1', 'other', 'foo', 'modules'),
           File.join(envdir, 'env1', 'other', 'bar', 'modules')
         ])
@@ -563,9 +563,9 @@ config_version=$vardir/random/scripts
 
       it "does not reload the environment if it isn't expired" do
         env = Puppet::Node::Environment.create(:cached, [])
-        mocked_loader = double('loader')
-        expect(mocked_loader).to receive(:get).with(:cached).and_return(env).once
-        expect(mocked_loader).to receive(:get_conf).with(:cached).and_return(Puppet::Settings::EnvironmentConf.static_for(env, 20)).once
+        mocked_loader = mock('loader')
+        mocked_loader.expects(:get).with(:cached).returns(env).once
+        mocked_loader.expects(:get_conf).with(:cached).returns(Puppet::Settings::EnvironmentConf.static_for(env, 20)).once
 
         cached = Puppet::Environments::Cached.new(mocked_loader)
 
@@ -589,9 +589,9 @@ config_version=$vardir/random/scripts
 
       it "does not reload the environment if it isn't expired" do
         env = Puppet::Node::Environment.create(:cached, [])
-        mocked_loader = double('loader')
-        expect(mocked_loader).to receive(:get).with(:cached).and_return(env).once
-        expect(mocked_loader).to receive(:get_conf).with(:cached).and_return(Puppet::Settings::EnvironmentConf.static_for(env, 20)).once
+        mocked_loader = mock('loader')
+        mocked_loader.expects(:get).with(:cached).returns(env).once
+        mocked_loader.expects(:get_conf).with(:cached).returns(Puppet::Settings::EnvironmentConf.static_for(env, 20)).once
 
         cached = Puppet::Environments::Cached.new(mocked_loader)
 

data/spec/unit/etc_spec.rb

@@ -102,11 +102,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
     user
   end
 
-  shared_examples "methods that return an overridden group struct from Etc" do |param|
+  shared_examples "methods that return an overridden group struct from Etc" do |params|
+
     it "should return a new Struct object with corresponding canonical_ members" do
       group = Etc::Group.new
-      expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(group)
-      puppet_group = Puppet::Etc.send(subject, *param)
+      Etc.expects(subject).with(*params).returns(group)
+      puppet_group = Puppet::Etc.send(subject, *params)
 
       expect(puppet_group.members).to include(*group.members)
       expect(puppet_group.members).to include(*group.members.map { |mem| "canonical_#{mem}".to_sym })
@@ -117,12 +118,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
     context "when Encoding.default_external is UTF-8" do
       before do
-        expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(utf_8_group_struct)
+        Etc.expects(subject).with(*params).returns(utf_8_group_struct)
       end
 
       let(:overridden) {
         PuppetSpec::CharacterEncoding.with_external_encoding(Encoding::UTF_8) do
-          Puppet::Etc.send(subject, *param)
+          Puppet::Etc.send(subject, *params)
         end
       }
 
@@ -167,12 +168,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
     context "when Encoding.default_external is EUC_KR (i.e., neither UTF-8 nor POSIX)" do
       before do
-        expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(euc_kr_group_struct)
+        Etc.expects(subject).with(*params).returns(euc_kr_group_struct)
       end
 
       let(:overridden) {
         PuppetSpec::CharacterEncoding.with_external_encoding(Encoding::EUC_KR) do
-          Puppet::Etc.send(subject, *param)
+          Puppet::Etc.send(subject, *params)
         end
       }
 
@@ -209,12 +210,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
     context "when Encoding.default_external is POSIX (ASCII-7bit)" do
       before do
-        expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(ascii_group_struct)
+        Etc.expects(subject).with(*params).returns(ascii_group_struct)
       end
 
       let(:overridden) {
         PuppetSpec::CharacterEncoding.with_external_encoding(Encoding::ASCII) do
-          Puppet::Etc.send(subject, *param)
+          Puppet::Etc.send(subject, *params)
         end
       }
 
@@ -250,11 +251,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
     end
   end
 
-  shared_examples "methods that return an overridden user struct from Etc" do |param|
+  shared_examples "methods that return an overridden user struct from Etc" do |params|
+
     it "should return a new Struct object with corresponding canonical_ members" do
       user = Etc::Passwd.new
-      expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(user)
-      puppet_user = Puppet::Etc.send(subject, *param)
+      Etc.expects(subject).with(*params).returns(user)
+      puppet_user = Puppet::Etc.send(subject, *params)
 
       expect(puppet_user.members).to include(*user.members)
       expect(puppet_user.members).to include(*user.members.map { |mem| "canonical_#{mem}".to_sym })
@@ -264,12 +266,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
     context "when Encoding.default_external is UTF-8" do
       before do
-        expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(utf_8_user_struct)
+        Etc.expects(subject).with(*params).returns(utf_8_user_struct)
       end
 
       let(:overridden) {
         PuppetSpec::CharacterEncoding.with_external_encoding(Encoding::UTF_8) do
-          Puppet::Etc.send(subject, *param)
+          Puppet::Etc.send(subject, *params)
         end
       }
 
@@ -301,12 +303,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
     context "when Encoding.default_external is EUC_KR (i.e., neither UTF-8 nor POSIX)" do
       before do
-        expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(euc_kr_user_struct)
+        Etc.expects(subject).with(*params).returns(euc_kr_user_struct)
       end
 
       let(:overridden) {
         PuppetSpec::CharacterEncoding.with_external_encoding(Encoding::EUC_KR) do
-          Puppet::Etc.send(subject, *param)
+          Puppet::Etc.send(subject, *params)
         end
       }
 
@@ -334,12 +336,12 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
     context "when Encoding.default_external is POSIX (ASCII-7bit)" do
       before do
-        expect(Etc).to receive(subject).with(param.nil? ? no_args : param).and_return(ascii_user_struct)
+        Etc.expects(subject).with(*params).returns(ascii_user_struct)
       end
 
       let(:overridden) {
         PuppetSpec::CharacterEncoding.with_external_encoding(Encoding::ASCII) do
-          Puppet::Etc.send(subject, *param)
+          Puppet::Etc.send(subject, *params)
         end
       }
 
@@ -374,7 +376,7 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
     it_should_behave_like "methods that return an overridden group struct from Etc", 'foo'
 
     it "should call Etc.getgrnam with the supplied group name" do
-      expect(Etc).to receive(:getgrnam).with('foo')
+      Etc.expects(:getgrnam).with('foo')
       Puppet::Etc.getgrnam('foo')
     end
   end
@@ -383,7 +385,7 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
     it_should_behave_like "methods that return an overridden group struct from Etc", 0
 
     it "should call Etc.getgrgid with supplied group id" do
-      expect(Etc).to receive(:getgrgid).with(0)
+      Etc.expects(:getgrgid).with(0)
       Puppet::Etc.getgrgid(0)
     end
   end
@@ -396,7 +398,7 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
     it_should_behave_like "methods that return an overridden user struct from Etc", 'foo'
 
     it "should call Etc.getpwnam with that username" do
-      expect(Etc).to receive(:getpwnam).with('foo')
+      Etc.expects(:getpwnam).with('foo')
       Puppet::Etc.getpwnam('foo')
     end
   end
@@ -405,14 +407,14 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
     it_should_behave_like "methods that return an overridden user struct from Etc", 2
 
     it "should call Etc.getpwuid with the id" do
-      expect(Etc).to receive(:getpwuid).with(2)
+      Etc.expects(:getpwuid).with(2)
       Puppet::Etc.getpwuid(2)
     end
   end
 
   describe :group do
     it 'should return the next group struct if a block is not provided' do
-      expect(Puppet::Etc).to receive(:getgrent).and_return(ascii_group_struct)
+      Puppet::Etc.expects(:getgrent).returns(ascii_group_struct)
 
       expect(Puppet::Etc.group).to eql(ascii_group_struct)
     end
@@ -423,10 +425,10 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
         euc_kr_group_struct,
         ascii_group_struct
       ]
-      allow(Puppet::Etc).to receive(:getgrent).and_return(*(expected_groups + [nil]))
+      Puppet::Etc.stubs(:getgrent).returns(*(expected_groups + [nil]))
 
-      expect(Puppet::Etc).to receive(:setgrent)
-      expect(Puppet::Etc).to receive(:endgrent)
+      Puppet::Etc.expects(:setgrent)
+      Puppet::Etc.expects(:endgrent)
 
       actual_groups = []
       Puppet::Etc.group { |group| actual_groups << group }
@@ -437,28 +439,28 @@ describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
 
   describe "endgrent" do
     it "should call Etc.getgrent" do
-      expect(Etc).to receive(:getgrent)
+      Etc.expects(:getgrent)
       Puppet::Etc.getgrent
     end
   end
 
   describe "setgrent" do
     it "should call Etc.setgrent" do
-      expect(Etc).to receive(:setgrent)
+      Etc.expects(:setgrent)
       Puppet::Etc.setgrent
     end
   end
 
   describe "endpwent" do
     it "should call Etc.endpwent" do
-      expect(Etc).to receive(:endpwent)
+      Etc.expects(:endpwent)
       Puppet::Etc.endpwent
     end
   end
 
   describe "setpwent" do
     it "should call Etc.setpwent" do
-      expect(Etc).to receive(:setpwent)
+      Etc.expects(:setpwent)
       Puppet::Etc.setpwent
     end
   end

data/spec/unit/external/pson_spec.rb

@@ -1,3 +1,4 @@
+#! /usr/bin/env ruby
 # Encoding: UTF-8
 require 'spec_helper'
 

data/spec/unit/face/catalog_spec.rb

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

data/spec/unit/face/config_spec.rb

@@ -1,6 +1,8 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 require 'puppet/face'
 
+module PuppetFaceSpecs
 describe Puppet::Face[:config, '0.0.1'] do
 
   let(:config) { described_class }
@@ -63,7 +65,7 @@ trace = true
     #This has to be after the settings above, which resets the value
     Puppet[:log_level] = 'info'
 
-    expect(Puppet).not_to receive(:warning)
+    Puppet.expects(:warning).never
     expect {
       result = subject.print("syslogfacility", :section => "user")
       expect(render(:print, result)).to eq("file\n")
@@ -74,7 +76,7 @@ trace = true
     Puppet[:log_level] = 'info'
     Puppet[:trace] = true
 
-    expect(Puppet).to receive(:warning).with("No section specified; defaulting to 'main'.\nSet the config section " +
+    Puppet.expects(:warning).with("No section specified; defaulting to 'main'.\nSet the config section " +
       "by using the `--section` flag.\nFor example, `puppet config --section user print foo`.\nFor more " +
       "information, see https://puppet.com/docs/puppet/latest/configuration.html")
     expect {
@@ -87,7 +89,7 @@ trace = true
     Puppet[:log_level] = 'notice'
     Puppet[:trace] = true
 
-    expect(Puppet).not_to receive(:warning)
+    Puppet.expects(:warning).never
     expect {
       result = subject.print("trace")
       expect(render(:print, result)).to eq("true\n")
@@ -130,13 +132,13 @@ trace = true
     let(:path) { Pathname.new(config_file).expand_path }
     before(:each) do
       Puppet[:config] = config_file
-      allow(Puppet::FileSystem).to receive(:pathname).with(path.to_s).and_return(path)
-      allow(Puppet::FileSystem).to receive(:touch)
+      Puppet::FileSystem.stubs(:pathname).with(path.to_s).returns(path)
+      Puppet::FileSystem.stubs(:touch)
     end
 
     it "prints the section and environment when no section is given and verbose is set" do
       Puppet[:log_level] = 'info'
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
       expect {
         subject.set('foo', 'bar')
       }.to output("\e[1;33mResolving settings from section 'main' in environment 'production'\e[0m\n").to_stderr
@@ -144,40 +146,40 @@ trace = true
 
     it "prints the section and environment when a section is given and verbose is set" do
       Puppet[:log_level] = 'info'
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
       expect {
         subject.set('foo', 'bar', {:section => "baz"})
       }.to output("\e[1;33mResolving settings from section 'baz' in environment 'production'\e[0m\n").to_stderr
     end
 
     it "writes to the correct puppet config file" do
-      expect(Puppet::FileSystem).to receive(:open).with(path, anything, anything)
+      Puppet::FileSystem.expects(:open).with(path, anything, anything)
       subject.set('foo', 'bar')
     end
 
     it "creates a config file if one does not exist" do
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
-      expect(Puppet::FileSystem).to receive(:touch).with(path)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
+      Puppet::FileSystem.expects(:touch).with(path)
       subject.set('foo', 'bar')
     end
 
     it "sets the supplied config/value in the default section (main)" do
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
       config = Puppet::Settings::IniFile.new([Puppet::Settings::IniFile::DefaultSection.new])
       manipulator = Puppet::Settings::IniFile::Manipulator.new(config)
-      allow(Puppet::Settings::IniFile::Manipulator).to receive(:new).and_return(manipulator)
+      Puppet::Settings::IniFile::Manipulator.stubs(:new).returns(manipulator)
 
-      expect(manipulator).to receive(:set).with("main", "foo", "bar")
+      manipulator.expects(:set).with("main", "foo", "bar")
       subject.set('foo', 'bar')
     end
 
     it "sets the value in the supplied section" do
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
       config = Puppet::Settings::IniFile.new([Puppet::Settings::IniFile::DefaultSection.new])
       manipulator = Puppet::Settings::IniFile::Manipulator.new(config)
-      allow(Puppet::Settings::IniFile::Manipulator).to receive(:new).and_return(manipulator)
+      Puppet::Settings::IniFile::Manipulator.stubs(:new).returns(manipulator)
 
-      expect(manipulator).to receive(:set).with("baz", "foo", "bar")
+      manipulator.expects(:set).with("baz", "foo", "bar")
       subject.set('foo', 'bar', {:section => "baz"})
     end
 
@@ -188,7 +190,7 @@ trace = true
       CONF
 
       myfile = StringIO.new(contents)
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(myfile)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(myfile)
 
       subject.set('foo', 'bar')
 
@@ -197,7 +199,7 @@ trace = true
     end
 
     it "opens the file with UTF-8 encoding" do
-      expect(Puppet::FileSystem).to receive(:open).with(path, nil, 'r+:UTF-8')
+      Puppet::FileSystem.expects(:open).with(path, nil, 'r+:UTF-8')
       subject.set('foo', 'bar')
     end
   end
@@ -205,15 +207,14 @@ trace = true
   context 'when the puppet.conf file does not exist' do
     let(:config_file) { '/foo/puppet.conf' }
     let(:path) { Pathname.new(config_file).expand_path }
-
     before(:each) do
       Puppet[:config] = config_file
-      allow(Puppet::FileSystem).to receive(:pathname).with(path.to_s).and_return(path)
+      Puppet::FileSystem.stubs(:pathname).with(path.to_s).returns(path)
     end
 
     it 'prints a message when the puppet.conf file does not exist' do
-      allow(Puppet::FileSystem).to receive(:exist?).with(path).and_return(false)
-      expect(Puppet).to receive(:warning).with("The puppet.conf file does not exist #{path.to_s}")
+      Puppet::FileSystem.stubs(:exist?).with(path).returns(false)
+      Puppet.expects(:warning).with("The puppet.conf file does not exist #{path.to_s}")
       subject.delete('setting', {:section => 'main'})
     end
   end
@@ -223,33 +224,34 @@ trace = true
     let(:path) { Pathname.new(config_file).expand_path }
     before(:each) do
       Puppet[:config] = config_file
-      allow(Puppet::FileSystem).to receive(:pathname).with(path.to_s).and_return(path)
-      allow(Puppet::FileSystem).to receive(:exist?).with(path).and_return(true)
+      Puppet::FileSystem.stubs(:pathname).with(path.to_s).returns(path)
+      Puppet::FileSystem.stubs(:exist?).with(path).returns(true)
     end
 
     it 'prints a message about what was deleted' do
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
       config = Puppet::Settings::IniFile.new([Puppet::Settings::IniFile::DefaultSection.new])
       manipulator = Puppet::Settings::IniFile::Manipulator.new(config)
-      allow(Puppet::Settings::IniFile::Manipulator).to receive(:new).and_return(manipulator)
+      Puppet::Settings::IniFile::Manipulator.stubs(:new).returns(manipulator)
 
-      expect(manipulator).to receive(:delete).with('main', 'setting').and_return('    setting=value')
+      manipulator.expects(:delete).with('main', 'setting').returns('    setting=value')
       expect { subject.delete('setting', {:section => 'main'}) }.to have_printed("Deleted setting from 'main': 'setting=value'")
     end
 
     it 'prints a warning when a setting is not found to delete' do
-      allow(Puppet::FileSystem).to receive(:open).with(path, anything, anything).and_yield(StringIO.new)
+      Puppet::FileSystem.stubs(:open).with(path, anything, anything).yields(StringIO.new)
       config = Puppet::Settings::IniFile.new([Puppet::Settings::IniFile::DefaultSection.new])
       manipulator = Puppet::Settings::IniFile::Manipulator.new(config)
-      allow(Puppet::Settings::IniFile::Manipulator).to receive(:new).and_return(manipulator)
+      Puppet::Settings::IniFile::Manipulator.stubs(:new).returns(manipulator)
 
-      expect(manipulator).to receive(:delete).with('main', 'setting').and_return(nil)
-      expect(Puppet).to receive(:warning).with("No setting found in configuration file for section 'main' setting name 'setting'")
+      manipulator.expects(:delete).with('main', 'setting').returns(nil)
+      Puppet.expects(:warning).with("No setting found in configuration file for section 'main' setting name 'setting'")
       subject.delete('setting', {:section => 'main'})
     end
   end
 
   shared_examples_for :config_printing_a_section do |section|
+
     def add_section_option(args, section)
       args << { :section => section } if section
       args
@@ -335,6 +337,7 @@ modulepath =
     end
 
     context "from master section" do
+
       before(:each) do
         Puppet.settings.parse_config(<<-CONF)
         [master]
@@ -347,3 +350,4 @@ modulepath =
     end
   end
 end
+end

data/spec/unit/face/epp_face_spec.rb

@@ -102,6 +102,7 @@ describe Puppet::Face[:epp, :current] do
     end
   end
 
+
   context "dump" do
     it "prints the AST of a template given with the -e option" do
       expect(eppface.dump({ :e => 'hello world' })).to eq("(lambda (epp (block\n  (render-s 'hello world')\n)))\n")
@@ -375,11 +376,11 @@ goodbye world
   end
 
   def from_an_interactive_terminal
-    allow(STDIN).to receive(:tty?).and_return(true)
+    STDIN.stubs(:tty?).returns(true)
   end
 
   def from_a_piped_input_of(contents)
-    allow(STDIN).to receive(:tty?).and_return(false)
-    allow(STDIN).to receive(:read).and_return(contents)
+    STDIN.stubs(:tty?).returns(false)
+    STDIN.stubs(:read).returns(contents)
   end
 end