Merge pull request #7194 from coderanger/silent-failure

Allow specifying `ignore_failure :quiet` to disable the error spew

2 files changed, 44 insertions, 5 deletions

diff --git a/spec/unit/formatters/base_spec.rb b/spec/unit/formatters/base_spec.rb
index 30c7757e5a..19182554f9 100644
--- a/spec/unit/formatters/base_spec.rb
+++ b/spec/unit/formatters/base_spec.rb
@@ -23,6 +23,14 @@ describe Chef::Formatters::Base do
   let(:out) { StringIO.new }
   let(:err) { StringIO.new }
   let(:formatter) { Chef::Formatters::Base.new(out, err) }
+  let(:exception) do
+    # An exception with a real backtrace.
+    begin
+      raise EOFError
+    rescue EOFError => exc
+    end
+    exc
+  end
 
   it "starts with an indentation of zero" do
     expect(formatter.output.indent).to eql(0)
@@ -45,27 +53,48 @@ describe Chef::Formatters::Base do
   end
 
   it "humanizes EOFError exceptions for #registration_failed" do
-    formatter.registration_failed("foo.example.com", EOFError.new, double("Chef::Config"))
+    formatter.registration_failed("foo.example.com", exception, double("Chef::Config"))
     expect(out.string).to match(/Received an EOF on transport socket/)
   end
 
   it "humanizes EOFError exceptions for #node_load_failed" do
-    formatter.node_load_failed("foo.example.com", EOFError.new, double("Chef::Config"))
+    formatter.node_load_failed("foo.example.com", exception, double("Chef::Config"))
     expect(out.string).to match(/Received an EOF on transport socket/)
   end
 
   it "humanizes EOFError exceptions for #run_list_expand_failed" do
-    formatter.run_list_expand_failed(double("Chef::Node"), EOFError.new)
+    formatter.run_list_expand_failed(double("Chef::Node"), exception)
     expect(out.string).to match(/Received an EOF on transport socket/)
   end
 
   it "humanizes EOFError exceptions for #cookbook_resolution_failed" do
-    formatter.run_list_expand_failed(double("Expanded Run List"), EOFError.new)
+    formatter.run_list_expand_failed(double("Expanded Run List"), exception)
     expect(out.string).to match(/Received an EOF on transport socket/)
   end
 
   it "humanizes EOFError exceptions for #cookbook_sync_failed" do
-    formatter.cookbook_sync_failed("foo.example.com", EOFError.new)
+    formatter.cookbook_sync_failed("foo.example.com", exception)
     expect(out.string).to match(/Received an EOF on transport socket/)
   end
+
+  it "outputs error information for failed resources with ignore_failure true" do
+    resource = Chef::Resource::RubyBlock.new("test")
+    resource.ignore_failure(true)
+    formatter.resource_failed(resource, :run, exception)
+    expect(out.string).to match(/Error executing action `run` on resource 'ruby_block\[test\]'/)
+  end
+
+  it "does not output error information for failed resources with ignore_failure :quiet" do
+    resource = Chef::Resource::RubyBlock.new("test")
+    resource.ignore_failure(:quiet)
+    formatter.resource_failed(resource, :run, exception)
+    expect(out.string).to eq("")
+  end
+
+  it "does not output error information for failed resources with ignore_failure 'quiet'" do
+    resource = Chef::Resource::RubyBlock.new("test")
+    resource.ignore_failure("quiet")
+    formatter.resource_failed(resource, :run, exception)
+    expect(out.string).to eq("")
+  end
 end
diff --git a/spec/unit/resource_spec.rb b/spec/unit/resource_spec.rb
index fe853922a1..523f9f7365 100644
--- a/spec/unit/resource_spec.rb
+++ b/spec/unit/resource_spec.rb
@@ -555,6 +555,16 @@ end
       resource.ignore_failure(true)
       expect(resource.ignore_failure).to eq(true)
     end
+
+    it "should allow you to set quiet ignore_failure as a symbol" do
+      resource.ignore_failure(:quiet)
+      expect(resource.ignore_failure).to eq(:quiet)
+    end
+
+    it "should allow you to set quiet ignore_failure as a string" do
+      resource.ignore_failure("quiet")
+      expect(resource.ignore_failure).to eq("quiet")
+    end
   end
 
   describe "retries" do