diff options
Diffstat (limited to 'spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb')
-rw-r--r-- | spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb b/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb index 700d7fa7ed..d96cfc1ede 100644 --- a/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb +++ b/spec/unit/formatters/error_inspectors/cookbook_resolve_error_inspector_spec.rb @@ -19,7 +19,6 @@ require "spec_helper" describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do - before do @expanded_run_list = Chef::RunList.new("recipe[annoyances]", "recipe[apache2]", "recipe[users]", "recipe[chef::client]") @@ -31,7 +30,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do describe "when explaining a 403 error" do before do - @response_body = %q({"error": [{"message": "gtfo"}]) @response = Net::HTTPForbidden.new("1.1", "403", "(response) forbidden") allow(@response).to receive(:body).and_return(@response_body) @@ -44,7 +42,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do it "prints a nice message" do expect { @description.display(@outputter) }.not_to raise_error end - end describe "when explaining a PreconditionFailed (412) error with current error message style" do @@ -52,7 +49,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do # which must be re-parsed to get the actual data. before do - @response_body = "{\"error\":[\"{\\\"non_existent_cookbooks\\\":[\\\"apache2\\\"],\\\"cookbooks_with_no_versions\\\":[\\\"users\\\"],\\\"message\\\":\\\"Run list contains invalid items: no such cookbook nope.\\\"}\"]}" @response = Net::HTTPPreconditionFailed.new("1.1", "412", "(response) unauthorized") allow(@response).to receive(:body).and_return(@response_body) @@ -70,7 +66,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do expect(observed_output).to include("users") expect(observed_output).not_to include("Run list contains invalid items: no such cookbook nope.") end - end describe "when explaining a PreconditionFailed (412) error with current error message style without cookbook details" do @@ -80,7 +75,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do # details. But we should still print a pretty error message. before do - @response_body = "{\"error\":[{\"non_existent_cookbooks\":[],\"cookbooks_with_no_versions\":[],\"message\":\"unable to solve dependencies in alotted time.\"}]}" @response = Net::HTTPPreconditionFailed.new("1.1", "412", "(response) unauthorized") allow(@response).to receive(:body).and_return(@response_body) @@ -95,7 +89,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do @outputter_output.rewind expect(@outputter_output.read).to include("unable to solve dependencies in alotted time.") end - end describe "when explaining a PreconditionFailed (412) error with single encoded JSON" do @@ -103,7 +96,6 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do # which must be re-parsed to get the actual data. before do - @response_body = "{\"error\":[{\"non_existent_cookbooks\":[\"apache2\"],\"cookbooks_with_no_versions\":[\"users\"],\"message\":\"Run list contains invalid items: no such cookbook nope.\"}]}" @response = Net::HTTPPreconditionFailed.new("1.1", "412", "(response) unauthorized") allow(@response).to receive(:body).and_return(@response_body) @@ -121,9 +113,5 @@ describe Chef::Formatters::ErrorInspectors::CookbookResolveErrorInspector do expect(observed_output).to include("users") expect(observed_output).not_to include("Run list contains invalid items: no such cookbook nope.") end - end end - - - |