From 78356de11f928bd673ac9e76328092ab74049c9d Mon Sep 17 00:00:00 2001 From: Bryan McLellan Date: Fri, 16 Nov 2012 09:50:50 -0800 Subject: CHEF-3582: Fix #check_lock under why-run We assume in whyrun mode that a non-existant user would be created before we attempt to lock it, so we return false from #check_lock when user does not exist and we are in why_run mode. Conflicts: chef/lib/chef/provider/user/useradd.rb --- .../unit/provider/useradd_based_user_provider.rb | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) (limited to 'spec/support/shared/unit/provider/useradd_based_user_provider.rb') diff --git a/spec/support/shared/unit/provider/useradd_based_user_provider.rb b/spec/support/shared/unit/provider/useradd_based_user_provider.rb index 3b8f8678bf..e9fe06dfd7 100644 --- a/spec/support/shared/unit/provider/useradd_based_user_provider.rb +++ b/spec/support/shared/unit/provider/useradd_based_user_provider.rb @@ -335,6 +335,24 @@ shared_examples_for "a useradd-based user provider" do |supported_useradd_option lambda { provider.check_lock }.should raise_error(Mixlib::ShellOut::ShellCommandFailed) end end + + context "when in why run mode" do + before do + passwd_status = mock("Mixlib::ShellOut command", :exitstatus => 0, :stdout => "", :stderr => "passwd: user 'chef-test' does not exist\n") + provider.should_receive(:shell_out!). + with("passwd", "-S", @new_resource.username, {:returns=>[0, 1]}). + and_return(passwd_status) + Chef::Config[:why_run] = true + end + + it "should return false if the user does not exist" do + provider.check_lock.should eql(false) + end + + it "should not raise an error if the user does not exist" do + lambda { provider.check_lock }.should_not raise_error + end + end end describe "when locking the user" do -- cgit v1.2.1