puppet 5.5.22-universal-darwin → 6.0.0-universal-darwin

data/spec/unit/provider/user/hpux_spec.rb

@@ -1,7 +1,10 @@
+#!/usr/bin/env ruby
 require 'spec_helper'
 require 'etc'
 
-describe Puppet::Type.type(:user).provider(:hpuxuseradd), :unless => Puppet.features.microsoft_windows? do
+provider_class = Puppet::Type.type(:user).provider(:hpuxuseradd)
+
+describe provider_class, :unless => Puppet::Util::Platform.windows? do
   let :resource do
     Puppet::Type.type(:user).new(
       :title => 'testuser',
@@ -12,14 +15,15 @@ describe Puppet::Type.type(:user).provider(:hpuxuseradd), :unless => Puppet.feat
   let(:provider) { resource.provider }
 
   it "should add -F when modifying a user" do
-    allow(resource).to receive(:allowdupe?).and_return(true)
-    expect(provider).to receive(:execute).with(include("-F"), anything)
+    resource.stubs(:allowdupe?).returns true
+    provider.stubs(:trusted).returns true
+    provider.expects(:execute).with { |args| args.include?("-F") }
     provider.uid = 1000
   end
 
   it "should add -F when deleting a user" do
-    allow(provider).to receive(:exists?).and_return(true)
-    expect(provider).to receive(:execute).with(include("-F"), anything)
+    provider.stubs(:exists?).returns(true)
+    provider.expects(:execute).with { |args| args.include?("-F") }
     provider.delete
   end
 
@@ -29,17 +33,17 @@ describe Puppet::Type.type(:user).provider(:hpuxuseradd), :unless => Puppet.feat
     end
 
     before :each do
-      allow(Etc).to receive(:getpwent).and_return(pwent)
-      allow(Etc).to receive(:getpwnam).and_return(pwent)
-      allow(resource).to receive(:command).with(:modify).and_return('/usr/sam/lbin/usermod.sam')
+      Etc.stubs(:getpwent).returns(pwent)
+      Etc.stubs(:getpwnam).returns(pwent)
+      resource.stubs(:command).with(:modify).returns '/usr/sam/lbin/usermod.sam'
     end
 
     it "should have feature manages_passwords" do
-      expect(described_class).to be_manages_passwords
+      expect(provider_class).to be_manages_passwords
     end
 
     it "should return nil if user does not exist" do
-      allow(Etc).to receive(:getpwent).and_return(nil)
+      Etc.stubs(:getpwent).returns(nil)
       expect(provider.password).to be_nil
     end
 
@@ -50,20 +54,20 @@ describe Puppet::Type.type(:user).provider(:hpuxuseradd), :unless => Puppet.feat
 
   context "check for trusted computing" do
     before :each do
-      allow(provider).to receive(:command).with(:modify).and_return('/usr/sam/lbin/usermod.sam')
+      provider.stubs(:command).with(:modify).returns '/usr/sam/lbin/usermod.sam'
     end
 
     it "should add modprpw to modifycmd if Trusted System" do
-      allow(resource).to receive(:allowdupe?).and_return(true)
-      expect(provider).to receive(:exec_getprpw).with('root','-m uid').and_return('uid=0')
-      expect(provider).to receive(:execute).with(['/usr/sam/lbin/usermod.sam', '-F', '-u', 1000, '-o', 'testuser', ';', '/usr/lbin/modprpw', '-v', '-l', 'testuser'], hash_including(custom_environment: {}))
+      resource.stubs(:allowdupe?).returns true
+      provider.expects(:exec_getprpw).with('root','-m uid').returns('uid=0')
+      provider.expects(:execute).with(['/usr/sam/lbin/usermod.sam', '-u', 1000, '-o', 'testuser', '-F', ';', '/usr/lbin/modprpw', '-v', '-l', 'testuser'], has_entry(:custom_environment, {}))
       provider.uid = 1000
     end
 
     it "should not add modprpw if not Trusted System" do
-      allow(resource).to receive(:allowdupe?).and_return(true)
-      expect(provider).to receive(:exec_getprpw).with('root','-m uid').and_return('System is not trusted')
-      expect(provider).to receive(:execute).with(['/usr/sam/lbin/usermod.sam', '-F', '-u', 1000, '-o', 'testuser'], hash_including(custom_environment: {}))
+      resource.stubs(:allowdupe?).returns true
+      provider.expects(:exec_getprpw).with('root','-m uid').returns('System is not trusted')
+      provider.expects(:execute).with(['/usr/sam/lbin/usermod.sam', '-u', 1000, '-o', 'testuser', '-F'], has_entry(:custom_environment, {}))
       provider.uid = 1000
     end
   end

data/spec/unit/provider/user/ldap_spec.rb

@@ -1,24 +1,27 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
-describe Puppet::Type.type(:user).provider(:ldap) do
+provider_class = Puppet::Type.type(:user).provider(:ldap)
+
+describe provider_class do
   it "should have the Ldap provider class as its baseclass" do
-    expect(described_class.superclass).to equal(Puppet::Provider::Ldap)
+    expect(provider_class.superclass).to equal(Puppet::Provider::Ldap)
   end
 
   it "should manage :posixAccount and :person objectclasses" do
-    expect(described_class.manager.objectclasses).to eq([:posixAccount, :person])
+    expect(provider_class.manager.objectclasses).to eq([:posixAccount, :person])
   end
 
   it "should use 'ou=People' as its relative base" do
-    expect(described_class.manager.location).to eq("ou=People")
+    expect(provider_class.manager.location).to eq("ou=People")
   end
 
   it "should use :uid as its rdn" do
-    expect(described_class.manager.rdn).to eq(:uid)
+    expect(provider_class.manager.rdn).to eq(:uid)
   end
 
   it "should be able to manage passwords" do
-    expect(described_class).to be_manages_passwords
+    expect(provider_class).to be_manages_passwords
   end
 
   {:name => "uid",
@@ -30,80 +33,80 @@ describe Puppet::Type.type(:user).provider(:ldap) do
     :shell => "loginShell"
   }.each do |puppet, ldap|
     it "should map :#{puppet.to_s} to '#{ldap}'" do
-      expect(described_class.manager.ldap_name(puppet)).to eq(ldap)
+      expect(provider_class.manager.ldap_name(puppet)).to eq(ldap)
     end
   end
 
-  context "when being created" do
+  describe "when being created" do
     before do
       # So we don't try to actually talk to ldap
-      @connection = double('connection')
-      allow(described_class.manager).to receive(:connect).and_yield(@connection)
+      @connection = mock 'connection'
+      provider_class.manager.stubs(:connect).yields @connection
     end
 
     it "should generate the sn as the last field of the cn" do
-      expect(Puppet::Type.type(:group).provider(:ldap)).to receive(:name2id).with(["whatever"]).and_return([123])
+      Puppet::Type.type(:group).provider(:ldap).expects(:name2id).with(["whatever"]).returns [123]
 
-      resource = double('resource', :should => %w{whatever})
-      allow(resource).to receive(:should).with(:comment).and_return(["Luke Kanies"])
-      allow(resource).to receive(:should).with(:ensure).and_return(:present)
-      instance = described_class.new(:name => "luke", :ensure => :absent)
+      resource = stub 'resource', :should => %w{whatever}
+      resource.stubs(:should).with(:comment).returns ["Luke Kanies"]
+      resource.stubs(:should).with(:ensure).returns :present
+      instance = provider_class.new(:name => "luke", :ensure => :absent)
 
-      allow(instance).to receive(:resource).and_return(resource)
+      instance.stubs(:resource).returns resource
 
-      expect(@connection).to receive(:add).with(anything, hash_including("sn" => ["Kanies"]))
+      @connection.expects(:add).with { |dn, attrs| attrs["sn"] == ["Kanies"] }
 
       instance.create
       instance.flush
     end
 
     it "should translate a group name to the numeric id" do
-      expect(Puppet::Type.type(:group).provider(:ldap)).to receive(:name2id).with("bar").and_return(101)
+      Puppet::Type.type(:group).provider(:ldap).expects(:name2id).with("bar").returns 101
 
-      resource = double('resource', :should => %w{whatever})
-      allow(resource).to receive(:should).with(:gid).and_return('bar')
-      allow(resource).to receive(:should).with(:ensure).and_return(:present)
-      instance = described_class.new(:name => "luke", :ensure => :absent)
-      allow(instance).to receive(:resource).and_return(resource)
+      resource = stub 'resource', :should => %w{whatever}
+      resource.stubs(:should).with(:gid).returns 'bar'
+      resource.stubs(:should).with(:ensure).returns :present
+      instance = provider_class.new(:name => "luke", :ensure => :absent)
+      instance.stubs(:resource).returns resource
 
-      expect(@connection).to receive(:add).with(anything, hash_including("gidNumber" => ["101"]))
+      @connection.expects(:add).with { |dn, attrs| attrs["gidNumber"] == ["101"] }
 
       instance.create
       instance.flush
     end
 
-    context "with no uid specified" do
+    describe "with no uid specified" do
       it "should pick the first available UID after the largest existing UID" do
-        expect(Puppet::Type.type(:group).provider(:ldap)).to receive(:name2id).with(["whatever"]).and_return([123])
+        Puppet::Type.type(:group).provider(:ldap).expects(:name2id).with(["whatever"]).returns [123]
 
         low = {:name=>["luke"], :shell=>:absent, :uid=>["600"], :home=>["/h"], :gid=>["1000"], :password=>["blah"], :comment=>["l k"]}
         high = {:name=>["testing"], :shell=>:absent, :uid=>["640"], :home=>["/h"], :gid=>["1000"], :password=>["blah"], :comment=>["t u"]}
-        expect(described_class.manager).to receive(:search).and_return([low, high])
+        provider_class.manager.expects(:search).returns([low, high])
 
-        resource = double('resource', :should => %w{whatever})
-        allow(resource).to receive(:should).with(:uid).and_return(nil)
-        allow(resource).to receive(:should).with(:ensure).and_return(:present)
-        instance = described_class.new(:name => "luke", :ensure => :absent)
-        allow(instance).to receive(:resource).and_return(resource)
+        resource = stub 'resource', :should => %w{whatever}
+        resource.stubs(:should).with(:uid).returns nil
+        resource.stubs(:should).with(:ensure).returns :present
+        instance = provider_class.new(:name => "luke", :ensure => :absent)
+        instance.stubs(:resource).returns resource
 
-        expect(@connection).to receive(:add).with(anything, hash_including("uidNumber" => ["641"]))
+        @connection.expects(:add).with { |dn, attrs| attrs["uidNumber"] == ["641"] }
 
         instance.create
         instance.flush
       end
 
       it "should pick 501 of no users exist" do
-        expect(Puppet::Type.type(:group).provider(:ldap)).to receive(:name2id).with(["whatever"]).and_return([123])
+        Puppet::Type.type(:group).provider(:ldap).expects(:name2id).with(["whatever"]).returns [123]
 
-        expect(described_class.manager).to receive(:search).and_return(nil)
+        provider_class.manager.expects(:search).returns nil
 
-        resource = double('resource', :should => %w{whatever})
-        allow(resource).to receive(:should).with(:uid).and_return(nil)
-        allow(resource).to receive(:should).with(:ensure).and_return(:present)
-        instance = described_class.new(:name => "luke", :ensure => :absent)
-        allow(instance).to receive(:resource).and_return(resource)
+        resource = stub 'resource', :should => %w{whatever}
+        resource.stubs(:should).with(:uid).returns nil
+        resource.stubs(:should).with(:ensure).returns :present
+        instance = provider_class.new(:name => "luke", :ensure => :absent)
+        instance.stubs(:resource).returns resource
 
-        expect(@connection).to receive(:add).with(anything, hash_including("uidNumber" => ["501"]))
+        @connection.expects(:add).with { |dn, attrs| attrs["uidNumber"] == ["501"] }
 
         instance.create
         instance.flush
@@ -111,21 +114,21 @@ describe Puppet::Type.type(:user).provider(:ldap) do
     end
   end
 
-  context "when flushing" do
+  describe "when flushing" do
     before do
-      allow(described_class).to receive(:suitable?).and_return(true)
+      provider_class.stubs(:suitable?).returns true
 
-      @instance = described_class.new(:name => "myname", :groups => %w{whatever}, :uid => "400")
+      @instance = provider_class.new(:name => "myname", :groups => %w{whatever}, :uid => "400")
     end
 
     it "should remove the :groups value before updating" do
-      expect(@instance.class.manager).to receive(:update).with(anything, anything, hash_excluding(:groups))
+      @instance.class.manager.expects(:update).with { |name, ldap, puppet| puppet[:groups].nil? }
 
       @instance.flush
     end
 
     it "should empty the property hash" do
-      allow(@instance.class.manager).to receive(:update)
+      @instance.class.manager.stubs(:update)
 
       @instance.flush
 
@@ -133,7 +136,7 @@ describe Puppet::Type.type(:user).provider(:ldap) do
     end
 
     it "should empty the ldap property hash" do
-      allow(@instance.class.manager).to receive(:update)
+      @instance.class.manager.stubs(:update)
 
       @instance.flush
 
@@ -141,109 +144,109 @@ describe Puppet::Type.type(:user).provider(:ldap) do
     end
   end
 
-  context "when checking group membership" do
+  describe "when checking group membership" do
     before do
       @groups = Puppet::Type.type(:group).provider(:ldap)
       @group_manager = @groups.manager
-      allow(described_class).to receive(:suitable?).and_return(true)
+      provider_class.stubs(:suitable?).returns true
 
-      @instance = described_class.new(:name => "myname")
+      @instance = provider_class.new(:name => "myname")
     end
 
     it "should show its group membership as the sorted list of all groups returned by an ldap query of group memberships" do
       one = {:name => "one"}
       two = {:name => "two"}
-      expect(@group_manager).to receive(:search).with("memberUid=myname").and_return([two, one])
+      @group_manager.expects(:search).with("memberUid=myname").returns([two, one])
 
       expect(@instance.groups).to eq("one,two")
     end
 
     it "should show its group membership as :absent if no matching groups are found in ldap" do
-      expect(@group_manager).to receive(:search).with("memberUid=myname").and_return(nil)
+      @group_manager.expects(:search).with("memberUid=myname").returns(nil)
 
       expect(@instance.groups).to eq(:absent)
     end
 
     it "should cache the group value" do
-      expect(@group_manager).to receive(:search).with("memberUid=myname").once.and_return(nil)
+      @group_manager.expects(:search).with("memberUid=myname").once.returns nil
 
       @instance.groups
       expect(@instance.groups).to eq(:absent)
     end
   end
 
-  context "when modifying group membership" do
+  describe "when modifying group membership" do
     before do
       @groups = Puppet::Type.type(:group).provider(:ldap)
       @group_manager = @groups.manager
-      allow(described_class).to receive(:suitable?).and_return(true)
+      provider_class.stubs(:suitable?).returns true
 
       @one = {:name => "one", :gid => "500"}
-      allow(@group_manager).to receive(:find).with("one").and_return(@one)
+      @group_manager.stubs(:find).with("one").returns(@one)
 
       @two = {:name => "one", :gid => "600"}
-      allow(@group_manager).to receive(:find).with("two").and_return(@two)
+      @group_manager.stubs(:find).with("two").returns(@two)
 
-      @instance = described_class.new(:name => "myname")
+      @instance = provider_class.new(:name => "myname")
 
-      allow(@instance).to receive(:groups).and_return(:absent)
+      @instance.stubs(:groups).returns :absent
     end
 
     it "should fail if the group does not exist" do
-      expect(@group_manager).to receive(:find).with("mygroup").and_return(nil)
+      @group_manager.expects(:find).with("mygroup").returns nil
 
       expect { @instance.groups = "mygroup" }.to raise_error(Puppet::Error)
     end
 
     it "should only pass the attributes it cares about to the group manager" do
-      expect(@group_manager).to receive(:update).with(anything, hash_excluding(:gid), anything)
+      @group_manager.expects(:update).with { |name, attrs| attrs[:gid].nil? }
 
       @instance.groups = "one"
     end
 
     it "should always include :ensure => :present in the current values" do
-      expect(@group_manager).to receive(:update).with(anything, hash_including(ensure: :present), anything)
+      @group_manager.expects(:update).with { |name, is, should| is[:ensure] == :present }
 
       @instance.groups = "one"
     end
 
     it "should always include :ensure => :present in the desired values" do
-      expect(@group_manager).to receive(:update).with(anything, anything, hash_including(ensure: :present))
+      @group_manager.expects(:update).with { |name, is, should| should[:ensure] == :present }
 
       @instance.groups = "one"
     end
 
     it "should always pass the group's original member list" do
       @one[:members] = %w{yay ness}
-      expect(@group_manager).to receive(:update).with(anything, hash_including(members: %w{yay ness}), anything)
+      @group_manager.expects(:update).with { |name, is, should| is[:members] == %w{yay ness} }
 
       @instance.groups = "one"
     end
 
     it "should find the group again when resetting its member list, so it has the full member list" do
-      expect(@group_manager).to receive(:find).with("one").and_return(@one)
+      @group_manager.expects(:find).with("one").returns(@one)
 
-      allow(@group_manager).to receive(:update)
+      @group_manager.stubs(:update)
 
       @instance.groups = "one"
     end
 
-    context "for groups that have no members" do
+    describe "for groups that have no members" do
       it "should create a new members attribute with its value being the user's name" do
-        expect(@group_manager).to receive(:update).with(anything, anything, hash_including(members: %w{myname}))
+        @group_manager.expects(:update).with { |name, is, should| should[:members] == %w{myname} }
 
         @instance.groups = "one"
       end
     end
 
-    context "for groups it is being removed from" do
+    describe "for groups it is being removed from" do
       it "should replace the group's member list with one missing the user's name" do
         @one[:members] = %w{myname a}
         @two[:members] = %w{myname b}
 
-        expect(@group_manager).to receive(:update).with("two", anything, hash_including(members: %w{b}))
+        @group_manager.expects(:update).with { |name, is, should| name == "two" and should[:members] == %w{b} }
 
-        allow(@instance).to receive(:groups).and_return("one,two")
+        @instance.stubs(:groups).returns "one,two"
         @instance.groups = "one"
       end
 
@@ -251,33 +254,33 @@ describe Puppet::Type.type(:user).provider(:ldap) do
         @one[:members] = %w{myname}
         @two[:members] = %w{myname b}
 
-        expect(@group_manager).to receive(:update).with("one", anything, hash_including(members: :absent))
+        @group_manager.expects(:update).with { |name, is, should| name == "one" and should[:members] == :absent }
 
-        allow(@instance).to receive(:groups).and_return("one,two")
+        @instance.stubs(:groups).returns "one,two"
         @instance.groups = "two"
       end
     end
 
-    context "for groups that already have members" do
+    describe "for groups that already have members" do
       it "should replace each group's member list with a new list including the user's name" do
         @one[:members] = %w{a b}
-        expect(@group_manager).to receive(:update).with(anything, anything, hash_including(members: %w{a b myname}))
+        @group_manager.expects(:update).with { |name, is, should| should[:members] == %w{a b myname} }
         @two[:members] = %w{b c}
-        expect(@group_manager).to receive(:update).with(anything, anything, hash_including(members: %w{b c myname}))
+        @group_manager.expects(:update).with { |name, is, should| should[:members] == %w{b c myname} }
 
         @instance.groups = "one,two"
       end
     end
 
-    context "for groups of which it is a member" do
+    describe "for groups of which it is a member" do
       it "should do nothing" do
         @one[:members] = %w{a b}
-        expect(@group_manager).to receive(:update).with(anything, anything, hash_including(members: %w{a b myname}))
+        @group_manager.expects(:update).with { |name, is, should| should[:members] == %w{a b myname} }
 
         @two[:members] = %w{c myname}
-        expect(@group_manager).not_to receive(:update).with("two", any_args)
+        @group_manager.expects(:update).with { |name, *other| name == "two" }.never
 
-        allow(@instance).to receive(:groups).and_return("two")
+        @instance.stubs(:groups).returns "two"
 
         @instance.groups = "one,two"
       end

data/spec/unit/provider/user/openbsd_spec.rb

@@ -1,11 +1,13 @@
+#! /usr/bin/env ruby
 require 'spec_helper'
 
 describe Puppet::Type.type(:user).provider(:openbsd) do
+
   before :each do
-    allow(described_class).to receive(:command).with(:password).and_return('/usr/sbin/passwd')
-    allow(described_class).to receive(:command).with(:add).and_return('/usr/sbin/useradd')
-    allow(described_class).to receive(:command).with(:modify).and_return('/usr/sbin/usermod')
-    allow(described_class).to receive(:command).with(:delete).and_return('/usr/sbin/userdel')
+    described_class.stubs(:command).with(:password).returns '/usr/sbin/passwd'
+    described_class.stubs(:command).with(:add).returns '/usr/sbin/useradd'
+    described_class.stubs(:command).with(:modify).returns '/usr/sbin/usermod'
+    described_class.stubs(:command).with(:delete).returns '/usr/sbin/userdel'
   end
 
   let(:resource) do
@@ -31,21 +33,20 @@ describe Puppet::Type.type(:user).provider(:openbsd) do
   describe "#expiry=" do
     it "should pass expiry to usermod as MM/DD/YY" do
       resource[:expiry] = '2014-11-05'
-      expect(provider).to receive(:execute).with(['/usr/sbin/usermod', '-e', 'November 05 2014', 'myuser'], hash_including(custom_environment: {}))
+      provider.expects(:execute).with(['/usr/sbin/usermod', '-e', 'November 05 2014', 'myuser'], has_entry(:custom_environment, {}))
       provider.expiry = '2014-11-05'
     end
 
     it "should use -e with an empty string when the expiry property is removed" do
       resource[:expiry] = :absent
-      expect(provider).to receive(:execute).with(['/usr/sbin/usermod', '-e', '', 'myuser'], hash_including(custom_environment: {}))
+      provider.expects(:execute).with(['/usr/sbin/usermod', '-e', '', 'myuser'], has_entry(:custom_environment, {}))
       provider.expiry = :absent
     end
   end
 
   describe "#addcmd" do
     it "should return an array with the full command and expiry as MM/DD/YY" do
-      allow(Facter).to receive(:value).with(:osfamily).and_return('OpenBSD')
-      allow(Facter).to receive(:value).with(:operatingsystemmajrelease)
+      Facter.stubs(:value).with(:osfamily).returns('OpenBSD')
       resource[:expiry] = "1997-06-01"
       expect(provider.addcmd).to eq(['/usr/sbin/useradd', '-e', 'June 01 1997', 'myuser'])
     end
@@ -57,19 +58,19 @@ describe Puppet::Type.type(:user).provider(:openbsd) do
     end
 
     it "should return the loginclass if set", :if => Puppet.features.libshadow? do
-      expect(Shadow::Passwd).to receive(:getspnam).with('myuser').and_return(shadow_entry)
+      Shadow::Passwd.expects(:getspnam).with('myuser').returns shadow_entry
       provider.send(:loginclass).should == 'staff'
     end
 
     it "should return the empty string when loginclass isn't set", :if => Puppet.features.libshadow? do
       shadow_entry[:sp_loginclass] = ''
-      expect(Shadow::Passwd).to receive(:getspnam).with('myuser').and_return(shadow_entry)
+      Shadow::Passwd.expects(:getspnam).with('myuser').returns shadow_entry
       provider.send(:loginclass).should == ''
     end
 
     it "should return nil when loginclass isn't available", :if => Puppet.features.libshadow? do
       shadow_entry[:sp_loginclass] = nil
-      expect(Shadow::Passwd).to receive(:getspnam).with('myuser').and_return(shadow_entry)
+      Shadow::Passwd.expects(:getspnam).with('myuser').returns shadow_entry
       provider.send(:loginclass).should be_nil
     end
   end