diff options
author | Sean Karlage <skarlage@fb.com> | 2018-01-04 23:17:22 -0800 |
---|---|---|
committer | Sean Karlage <skarlage@fb.com> | 2018-01-04 23:28:51 -0800 |
commit | 3f54ed4ddfc6f394906cc160ebff2e41448fbd4e (patch) | |
tree | 3d2343020668104f1389ecc0d107087ea1548c4f /spec/unit/provider/user/dscl_spec.rb | |
parent | c6433605bae0a9f52c843ca8cb97e64b2bcf5f0a (diff) | |
download | chef-3f54ed4ddfc6f394906cc160ebff2e41448fbd4e.tar.gz |
Check for set home property before file existence (fixes #5777)
Signed-off-by: Sean Karlage <skarlage@fb.com>
Diffstat (limited to 'spec/unit/provider/user/dscl_spec.rb')
-rw-r--r-- | spec/unit/provider/user/dscl_spec.rb | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/spec/unit/provider/user/dscl_spec.rb b/spec/unit/provider/user/dscl_spec.rb index f59709e717..956c32664d 100644 --- a/spec/unit/provider/user/dscl_spec.rb +++ b/spec/unit/provider/user/dscl_spec.rb @@ -213,6 +213,32 @@ ea18e18b720e358e7fbe3cfbeaa561456f6ba008937a30" end end + describe "current_home_exists?" do + let(:current_resource) do + new_resource.dup + end + + before do + provider.current_resource = current_resource + end + + it "returns false for nil home dir" do + current_resource.home nil + expect(provider.current_home_exists?).to be_falsey + end + + it "is false for empty string" do + current_resource.home "" + expect(provider.current_home_exists?).to be_falsey + end + + it "is true for existing directory" do + current_resource.home "/Users/blah" + allow(::File).to receive(:exist?).with("/Users/blah").and_return(true) + expect(provider.current_home_exists?).to be_truthy + end + end + describe "when modifying the home directory" do let(:current_resource) do new_resource.dup |