puppet 5.5.22-x86-mingw32 → 6.0.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

@@ -9,7 +9,7 @@ require 'puppet_spec/character_encoding'
 # - We correctly set external encoding values IF they're valid UTF-8 bytes
 # - We do not modify non-UTF-8 values if they're NOT valid UTF-8 bytes
 
-describe Puppet::Etc, :if => !Puppet.features.microsoft_windows? do
+describe Puppet::Etc, :if => !Puppet::Util::Platform.windows? do
   # http://www.fileformat.info/info/unicode/char/5e0c/index.htm
   # 希 Han Character 'rare; hope, expect, strive for'
   # In EUC_KR: \xfd \xf1 - 253 241
@@ -102,11 +102,12 @@ describe Puppet::Etc, :if => !Puppet.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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.features.microsoft_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,60 +407,35 @@ describe Puppet::Etc, :if => !Puppet.features.microsoft_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)
-
-      expect(Puppet::Etc.group).to eql(ascii_group_struct)
-    end
-
-    it 'should iterate over the available groups if a block is provided' do
-      expected_groups = [
-        utf_8_group_struct,
-        euc_kr_group_struct,
-        ascii_group_struct
-      ]
-      allow(Puppet::Etc).to receive(:getgrent).and_return(*(expected_groups + [nil]))
-
-      expect(Puppet::Etc).to receive(:setgrent)
-      expect(Puppet::Etc).to receive(:endgrent)
-
-      actual_groups = []
-      Puppet::Etc.group { |group| actual_groups << group }
-
-      expect(actual_groups).to eql(expected_groups)
-    end
-  end
-
   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