diff options
author | Lamont Granquist <lamont@opscode.com> | 2013-03-25 13:06:41 -0700 |
---|---|---|
committer | Lamont Granquist <lamont@opscode.com> | 2013-03-25 13:06:41 -0700 |
commit | 9de1da01124e549ac514488d2dc71ffb46c9b57e (patch) | |
tree | b7100ddd1c9eca786d8d5ebf3a9004fe5c214573 /spec/unit/provider/directory_spec.rb | |
parent | db72ae9f28a3eaf27c8e8af5815f7b030186ef07 (diff) | |
download | chef-9de1da01124e549ac514488d2dc71ffb46c9b57e.tar.gz |
moar spec test fixes
Diffstat (limited to 'spec/unit/provider/directory_spec.rb')
-rw-r--r-- | spec/unit/provider/directory_spec.rb | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/spec/unit/provider/directory_spec.rb b/spec/unit/provider/directory_spec.rb index 3936525903..0b7d040210 100644 --- a/spec/unit/provider/directory_spec.rb +++ b/spec/unit/provider/directory_spec.rb @@ -59,7 +59,7 @@ describe Chef::Provider::Directory do end it "describes the access mode as a String of octal integers" do - File.stub!(:exist?).and_return(true) + File.stub!(:exists?).and_return(true) File.should_receive(:stat).and_return(mock_stat) @directory.load_current_resource @directory.current_resource.mode.should == "0755" @@ -67,7 +67,7 @@ describe Chef::Provider::Directory do context "when user and group are specified with UID/GID" do it "describes the current owner and group as UID and GID" do - File.stub!(:exist?).and_return(true) + File.stub!(:exists?).and_return(true) File.should_receive(:stat).and_return(mock_stat) @directory.load_current_resource @directory.current_resource.path.should eql(@new_resource.path) @@ -81,16 +81,16 @@ describe Chef::Provider::Directory do end # Unix only for now. While file security attribute reporting for windows is - # disabled, unix and windows differ in the number of exist? calls that are + # disabled, unix and windows differ in the number of exists? calls that are # made by the provider. it "should create a new directory on create, setting updated to true", :unix_only do @new_resource.path "/tmp/foo" - File.should_receive(:exist?).exactly(2).and_return(false) + File.should_receive(:exists?).at_least(:once).and_return(false) File.should_receive(:directory?).with("/tmp").and_return(true) Dir.should_receive(:mkdir).with(@new_resource.path).once.and_return(true) - @directory.should_receive(:set_all_access_controls) + @directory.should_receive(:do_acl_changes) @directory.stub!(:update_new_file_state) @directory.run_action(:create) @directory.new_resource.should be_updated @@ -103,20 +103,20 @@ describe Chef::Provider::Directory do end # Unix only for now. While file security attribute reporting for windows is - # disabled, unix and windows differ in the number of exist? calls that are + # disabled, unix and windows differ in the number of exists? calls that are # made by the provider. it "should create a new directory when parent directory does not exist if recursive is true and permissions are correct", :unix_only do @new_resource.path "/path/to/dir" @new_resource.recursive true - File.should_receive(:exist?).with(@new_resource.path).ordered.and_return(false) + File.should_receive(:exists?).with(@new_resource.path).ordered.and_return(false) - File.should_receive(:exist?).with('/path/to').ordered.and_return(false) - File.should_receive(:exist?).with('/path').ordered.and_return(true) + File.should_receive(:exists?).with('/path/to').ordered.and_return(false) + File.should_receive(:exists?).with('/path').ordered.and_return(true) File.should_receive(:writable?).with('/path').ordered.and_return(true) - File.should_receive(:exist?).with(@new_resource.path).ordered.and_return(false) + File.should_receive(:exists?).with(@new_resource.path).ordered.and_return(false) FileUtils.should_receive(:mkdir_p).with(@new_resource.path).and_return(true) - @directory.should_receive(:set_all_access_controls) + @directory.should_receive(:do_acl_changes) @directory.stub!(:update_new_file_state) @directory.run_action(:create) @new_resource.should be_updated @@ -131,16 +131,16 @@ describe Chef::Provider::Directory do end # Unix only for now. While file security attribute reporting for windows is - # disabled, unix and windows differ in the number of exist? calls that are + # disabled, unix and windows differ in the number of exists? calls that are # made by the provider. it "should not create the directory if it already exists", :unix_only do stub_file_cstats @new_resource.path "/tmp/foo" - File.should_receive(:directory?).twice.and_return(true) + File.should_receive(:directory?).at_least(:once).and_return(true) File.should_receive(:writable?).with("/tmp").and_return(true) - File.should_receive(:exist?).exactly(3).and_return(true) + File.should_receive(:exists?).at_least(:once).and_return(true) Dir.should_not_receive(:mkdir).with(@new_resource.path) - @directory.should_receive(:set_all_access_controls) + @directory.should_receive(:do_acl_changes) @directory.run_action(:create) end @@ -152,14 +152,14 @@ describe Chef::Provider::Directory do end it "should raise an exception if it cannot delete the directory due to bad permissions" do - File.stub!(:exist?).and_return(true) + File.stub!(:exists?).and_return(true) File.stub!(:writable?).and_return(false) lambda { @directory.run_action(:delete) }.should raise_error(RuntimeError) end it "should take no action when deleting a target directory that does not exist" do @new_resource.path "/an/invalid/path" - File.stub!(:exist?).and_return(false) + File.stub!(:exists?).and_return(false) Dir.should_not_receive(:delete).with(@new_resource.path) @directory.run_action(:delete) @directory.new_resource.should_not be_updated @@ -168,7 +168,7 @@ describe Chef::Provider::Directory do it "should raise an exception when deleting a directory when target directory is a file" do stub_file_cstats @new_resource.path "/an/invalid/path" - File.stub!(:exist?).and_return(true) + File.stub!(:exists?).and_return(true) File.should_receive(:directory?).and_return(false) Dir.should_not_receive(:delete).with(@new_resource.path) lambda { @directory.run_action(:delete) }.should raise_error(RuntimeError) |