diff options
| author | Lamont Granquist <lamont@scriptkiddie.org> | 2017-02-13 09:52:10 -0800 |
|---|---|---|
| committer | Lamont Granquist <lamont@scriptkiddie.org> | 2017-02-13 09:52:10 -0800 |
| commit | 404a9bc88be538769c6c80b3b31f39a6582991d2 (patch) | |
| tree | 2f8795e5f64153440c22a522d669c235a627f289 /lib/chef/formatters | |
| parent | b949a48acc21d4b64869bd7b834708d5232b1f2a (diff) | |
| download | chef-404a9bc88be538769c6c80b3b31f39a6582991d2.tar.gz | |
fix specs: RedundantReturn, RedundantSelf, RedundantBegin
department of redundancy department
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Diffstat (limited to 'lib/chef/formatters')
| -rw-r--r-- | lib/chef/formatters/error_inspectors/api_error_formatting.rb | 9 | ||||
| -rw-r--r-- | lib/chef/formatters/error_mapper.rb | 12 |
2 files changed, 10 insertions, 11 deletions
diff --git a/lib/chef/formatters/error_inspectors/api_error_formatting.rb b/lib/chef/formatters/error_inspectors/api_error_formatting.rb index 2415d0f4bb..53549ee77c 100644 --- a/lib/chef/formatters/error_inspectors/api_error_formatting.rb +++ b/lib/chef/formatters/error_inspectors/api_error_formatting.rb @@ -172,11 +172,10 @@ E # .../lib/ruby/1.9.1/net/http.rb:2709:in `read_body' # .../lib/ruby/1.9.1/net/http.rb:2736:in `body' # .../lib/chef/formatters/error_inspectors/api_error_formatting.rb:91:in `rescue in format_rest_error' - begin - exception.response.body - rescue Exception - "Cannot fetch the contents of the response." - end + + exception.response.body + rescue Exception + "Cannot fetch the contents of the response." end end diff --git a/lib/chef/formatters/error_mapper.rb b/lib/chef/formatters/error_mapper.rb index 4786a20465..eb89662308 100644 --- a/lib/chef/formatters/error_mapper.rb +++ b/lib/chef/formatters/error_mapper.rb @@ -30,7 +30,7 @@ class Chef headline = "Chef encountered an error attempting to create the client \"#{node_name}\"" description = ErrorDescription.new(headline) error_inspector.add_explanation(description) - return description + description end def self.node_load_failed(node_name, exception, config) @@ -38,7 +38,7 @@ class Chef headline = "Chef encountered an error attempting to load the node data for \"#{node_name}\"" description = ErrorDescription.new(headline) error_inspector.add_explanation(description) - return description + description end def self.run_list_expand_failed(node, exception) @@ -46,7 +46,7 @@ class Chef headline = "Error expanding the run_list:" description = ErrorDescription.new(headline) error_inspector.add_explanation(description) - return description + description end def self.cookbook_resolution_failed(expanded_run_list, exception) @@ -54,7 +54,7 @@ class Chef headline = "Error Resolving Cookbooks for Run List:" description = ErrorDescription.new(headline) error_inspector.add_explanation(description) - return description + description end def self.cookbook_sync_failed(cookbooks, exception) @@ -62,7 +62,7 @@ class Chef headline = "Error Syncing Cookbooks:" description = ErrorDescription.new(headline) error_inspector.add_explanation(description) - return description + description end def self.resource_failed(resource, action, exception) @@ -70,7 +70,7 @@ class Chef headline = "Error executing action `#{action}` on resource '#{resource}'" description = ErrorDescription.new(headline) error_inspector.add_explanation(description) - return description + description end def self.file_load_failed(path, exception) |