Enable Layout/ArgumentAlignment cop

deivid-rodriguez · deivid-rodriguez · commit e8174aa12571 · 2020-09-24T12:54:23.000+02:00
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -39,6 +39,9 @@ Lint/ParenthesesAsGroupedExpression:
 Layout/AccessModifierIndentation:
   Enabled: true
 
+Layout/ArgumentAlignment:
+  Enabled: true
+
 Layout/CaseIndentation:
   Enabled: true
 
diff --git a/lib/active_admin/batch_actions/resource_extension.rb b/lib/active_admin/batch_actions/resource_extension.rb
@@ -69,9 +69,9 @@ def add_default_batch_action
 
           redirect_to active_admin_config.route_collection_path(params),
                       notice: I18n.t("active_admin.batch_actions.succesfully_destroyed",
-                                        count: selected_ids.count,
-                                        model: active_admin_config.resource_label.downcase,
-                                        plural_model: active_admin_config.plural_resource_label(count: selected_ids.count).downcase)
+                                     count: selected_ids.count,
+                                     model: active_admin_config.resource_label.downcase,
+                                     plural_model: active_admin_config.plural_resource_label(count: selected_ids.count).downcase)
         end
       end
 
diff --git a/lib/active_admin/views/footer.rb b/lib/active_admin/views/footer.rb
@@ -21,8 +21,8 @@ def footer_text
 
       def powered_by_message
         I18n.t("active_admin.powered_by",
-          active_admin: link_to("Active Admin", "https://activeadmin.info"),
-          version: ActiveAdmin::VERSION).html_safe
+               active_admin: link_to("Active Admin", "https://activeadmin.info"),
+               version: ActiveAdmin::VERSION).html_safe
       end
 
     end
diff --git a/lib/generators/active_admin/webpacker/webpacker_generator.rb b/lib/generators/active_admin/webpacker/webpacker_generator.rb
@@ -12,12 +12,12 @@ def install_assets
         copy_file "#{__dir__}/plugins/jquery.js", Rails.root.join("config/webpack/plugins/jquery.js").to_s
 
         insert_into_file Rails.root.join("config/webpack/environment.js").to_s,
-          "const jquery = require('./plugins/jquery')\n",
-          after: /require\(('|")@rails\/webpacker\1\);?\n/
+                         "const jquery = require('./plugins/jquery')\n",
+                         after: /require\(('|")@rails\/webpacker\1\);?\n/
 
         insert_into_file Rails.root.join("config/webpack/environment.js").to_s,
-          "environment.plugins.prepend('jquery', jquery)\n",
-          before: "module.exports"
+                         "environment.plugins.prepend('jquery', jquery)\n",
+                         before: "module.exports"
 
         run "yarn add @activeadmin/activeadmin"
       end
diff --git a/lib/ransack_ext.rb b/lib/ransack_ext.rb
@@ -11,10 +11,10 @@
   end
 
   config.add_predicate "gteq_datetime",
-    arel_predicate: "gteq",
-    formatter: ->(v) { v.beginning_of_day }
+                       arel_predicate: "gteq",
+                       formatter: ->(v) { v.beginning_of_day }
 
   config.add_predicate "lteq_datetime",
-    arel_predicate: "lt",
-    formatter: ->(v) { v + 1.day }
+                       arel_predicate: "lt",
+                       formatter: ->(v) { v + 1.day }
 end
diff --git a/spec/support/templates_with_data/admin/posts.rb b/spec/support/templates_with_data/admin/posts.rb
@@ -109,7 +109,7 @@
           f.input :title
           f.input :author
           f.input :published_date,
-            hint: f.object.persisted? && "Created at #{f.object.created_at}"
+                  hint: f.object.persisted? && "Created at #{f.object.created_at}"
           f.input :custom_category_id
           f.input :category
           f.input :position
diff --git a/spec/unit/dependency_spec.rb b/spec/unit/dependency_spec.rb
@@ -58,17 +58,17 @@
     describe "`!`" do
       it "raises an error if requirement not met" do
         expect { k.foo! "5" }.to raise_error ActiveAdmin::DependencyError,
-          "You provided foo 1.2.3 but we need: 5."
+                                             "You provided foo 1.2.3 but we need: 5."
       end
 
       it "accepts multiple arguments" do
         expect { k.foo! "> 1", "< 1.2" }.to raise_error ActiveAdmin::DependencyError,
-          "You provided foo 1.2.3 but we need: > 1, < 1.2."
+                                                        "You provided foo 1.2.3 but we need: > 1, < 1.2."
       end
 
       it "raises an error if not provided" do
         expect { k.bar! }.to raise_error ActiveAdmin::DependencyError,
-          "To use bar you need to specify it in your Gemfile."
+                                         "To use bar you need to specify it in your Gemfile."
       end
     end
   end
@@ -98,10 +98,10 @@
       expect(k["a-b"].match! "1.2.3").to eq nil
 
       expect { k["a-b"].match! "2.5" }.to raise_error ActiveAdmin::DependencyError,
-        "You provided a-b 1.2.3 but we need: 2.5."
+                                                      "You provided a-b 1.2.3 but we need: 2.5."
 
       expect { k["b-c"].match! }.to raise_error ActiveAdmin::DependencyError,
-        "To use b-c you need to specify it in your Gemfile."
+                                                "To use b-c you need to specify it in your Gemfile."
     end
 
     # Note: Ruby comparison operators are separate from the `foo? '> 1'` syntax
@@ -137,7 +137,7 @@
 
       it "throws a custom error if the gem is missing" do
         expect { k["b-c"] < 23 }.to raise_error ActiveAdmin::DependencyError,
-          "To use b-c you need to specify it in your Gemfile."
+                                                "To use b-c you need to specify it in your Gemfile."
       end
     end
   end
