Enable Layout/SpaceAroundOperators cop

deivid-rodriguez · deivid-rodriguez · commit 7bd91d046d04 · 2019-06-19T14:28:24.000+02:00
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -111,6 +111,9 @@ Layout/SpaceAfterComma:
 Layout/SpaceAroundEqualsInParameterDefault:
   Enabled: true
 
+Layout/SpaceAroundOperators:
+  Enabled: true
+
 Layout/SpaceBeforeBlockBraces:
   Enabled: true
 
diff --git a/features/step_definitions/action_link_steps.rb b/features/step_definitions/action_link_steps.rb
@@ -7,7 +7,7 @@
 end
 
 Then /^I should see the actions column with the class "([^"]*)" and the title "([^"]*)"$/ do |klass, title|
-  expect(page).to have_css "th#{'.'+klass}", text: title
+  expect(page).to have_css "th#{'.' + klass}", text: title
 end
 
 Then /^I should see a dropdown menu item to "([^"]*)"$/ do |name|
diff --git a/features/step_definitions/format_steps.rb b/features/step_definitions/format_steps.rb
@@ -21,7 +21,7 @@
 
 # Check first rows of the displayed CSV.
 Then /^I should download a CSV file with "([^"]*)" separator for "([^"]*)" containing:$/ do |sep, resource_name, table|
-  body   = page.driver.response.body
+  body = page.driver.response.body
   content_type_header, content_disposition_header = %w[Content-Type Content-Disposition].map do |header_name|
     page.response_headers[header_name]
   end
diff --git a/features/step_definitions/user_steps.rb b/features/step_definitions/user_steps.rb
@@ -36,7 +36,7 @@ def ensure_user_created(email)
 
 Given /^override locale "([^"]*)" with "([^"]*)"$/ do |path, value|
   keys_value  = path.split('.') + [value]
-  locale_hash = keys_value.reverse.inject { |a, n| { n=>a } }
+  locale_hash = keys_value.reverse.inject { |a, n| { n => a } }
   I18n.available_locales
   I18n.backend.store_translations(I18n.locale, locale_hash)
 end
diff --git a/lib/active_admin/view_helpers/breadcrumb_helper.rb b/lib/active_admin/view_helpers/breadcrumb_helper.rb
@@ -12,9 +12,9 @@ def breadcrumb_links(path = request.path)
           # 1. try using `display_name` if we can locate a DB object
           # 2. try using the model name translation
           # 3. default to calling `titlecase` on the URL fragment
-          if part =~ /\A(\d+|[a-f0-9]{24}|(?:[a-f0-9]{8}-(?:[a-f0-9]{4}-){3}[a-f0-9]{12}))\z/ && parts[index-1]
+          if part =~ /\A(\d+|[a-f0-9]{24}|(?:[a-f0-9]{8}-(?:[a-f0-9]{4}-){3}[a-f0-9]{12}))\z/ && parts[index - 1]
             parent = active_admin_config.belongs_to_config.try :target
-            config = parent && parent.resource_name.route_key == parts[index-1] ? parent : active_admin_config
+            config = parent && parent.resource_name.route_key == parts[index - 1] ? parent : active_admin_config
             name   = display_name config.find_resource part
           end
           name ||= I18n.t "activerecord.models.#{part.singularize}", count: ::ActiveAdmin::Helpers::I18n::PLURAL_MANY_COUNT, default: part.titlecase
diff --git a/lib/ransack_ext.rb b/lib/ransack_ext.rb
@@ -2,11 +2,11 @@
 # identically to the versions given in Ransack.
 #
 Ransack.configure do |config|
-  { 'contains'=>'cont', 'starts_with'=>'start', 'ends_with'=>'end' }.each do |old, current|
+  { 'contains' => 'cont', 'starts_with' => 'start', 'ends_with' => 'end' }.each do |old, current|
     config.add_predicate old, Ransack::Constants::DERIVED_PREDICATES.detect { |q, _| q == current }[1]
   end
 
-  { 'equals'=>'eq', 'greater_than'=>'gt', 'less_than'=>'lt' }.each do |old, current|
+  { 'equals' => 'eq', 'greater_than' => 'gt', 'less_than' => 'lt' }.each do |old, current|
     config.add_predicate old, arel_predicate: current
   end
 
