diff options
| author | Dominic Cleal <dcleal@redhat.com> | 2011-08-06 17:46:20 +0100 |
|---|---|---|
| committer | Dominic Cleal <dcleal@redhat.com> | 2011-08-07 09:45:29 +0100 |
| commit | 9d09b077a33c1324d53f2725fcf61b529deafb4b (patch) | |
| tree | e553d3d34e58e69ccc3fbe1a5cec20574c542031 /spec | |
| parent | 2b9b7c114e7c599f88be4f3be70f504add8072f8 (diff) | |
| download | puppet-9d09b077a33c1324d53f2725fcf61b529deafb4b.tar.gz puppet-9d09b077a33c1324d53f2725fcf61b529deafb4b.tar.xz puppet-9d09b077a33c1324d53f2725fcf61b529deafb4b.zip | |
(#8808) Fail Augeas resource when unable to save changes
Raise a failure when Augeas changes cannot be saved (due to invalid layout of
the tree, permissions etc). Fixes a regression.
Diffstat (limited to 'spec')
| -rwxr-xr-x | spec/unit/provider/augeas/augeas_spec.rb | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/spec/unit/provider/augeas/augeas_spec.rb b/spec/unit/provider/augeas/augeas_spec.rb index 434a99d70..874f70a8d 100755 --- a/spec/unit/provider/augeas/augeas_spec.rb +++ b/spec/unit/provider/augeas/augeas_spec.rb @@ -442,6 +442,20 @@ describe provider_class do @provider.expects(:diff).with("#{file}", "#{file}.augnew").returns("") @provider.should be_need_to_run end + + it "should fail with an error if saving fails" do + file = "/etc/hosts" + + @resource[:context] = "/files" + @resource[:changes] = ["set #{file}/foo bar"] + + @augeas_stub.stubs(:save).returns(false) + @augeas_stub.stubs(:match).with("/augeas/events/saved").returns([]) + @augeas_stub.expects(:close) + + @provider.expects(:diff).never() + lambda { @provider.need_to_run? }.should raise_error + end end end |
