From 1a098dfcab0243837d9f5f726e4c0b28d22bfe7b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Javi=20Mart=C3=ADn?= <javim@elretirao.net>
Date: Sat, 1 Jul 2023 18:18:22 +0200
Subject: [PATCH] Add and apply MultilineMethodCallBraceLayout rule

In order for this rule to work effectively when running `--autocorrect`,
we also need to enable the `ClosingParenthesisIndentation` rule.
---
 .rubocop.yml                                             | 6 ++++++
 .../budgets/investments/form_component.html.erb          | 3 +--
 .../budgets/investments/my_ballot_component.html.erb     | 3 ++-
 app/components/debates/form_component.html.erb           | 3 +--
 app/components/proposals/form_component.html.erb         | 3 +--
 app/controllers/admin/budgets_controller.rb              | 3 ++-
 app/models/comment.rb                                    | 3 ++-
 app/models/organization.rb                               | 6 ++++--
 app/models/user.rb                                       | 3 ++-
 app/views/admin/officials/search.html.erb                | 2 +-
 app/views/layouts/application.html.erb                   | 3 +--
 app/views/legislation/proposals/_form.html.erb           | 3 +--
 app/views/legislation/proposals/_proposal.html.erb       | 3 ++-
 app/views/mailer/budget_investment_created.html.erb      | 3 ++-
 app/views/mailer/budget_investment_unfeasible.html.erb   | 3 ++-
 app/views/mailer/comment.html.erb                        | 3 ++-
 app/views/mailer/direct_message_for_receiver.html.erb    | 6 ++++--
 app/views/mailer/email_verification.html.erb             | 6 ++++--
 app/views/mailer/evaluation_comment.html.erb             | 3 +--
 app/views/mailer/newsletter.html.erb                     | 3 ++-
 app/views/mailer/proposal_notification_digest.html.erb   | 8 +++++---
 app/views/mailer/reply.html.erb                          | 3 ++-
 app/views/organizations/registrations/new.html.erb       | 3 +--
 app/views/polls/_advanced_stats.html.erb                 | 3 ++-
 app/views/users/registrations/new.html.erb               | 6 +++---
 app/views/verification/residence/new.html.erb            | 6 +++---
 spec/components/sdg/tag_list_component_spec.rb           | 3 +--
 .../sdg/targets/plain_tag_list_component_spec.rb         | 3 +--
 spec/components/sdg/targets/tag_list_component_spec.rb   | 3 +--
 spec/components/shared/banner_component_spec.rb          | 9 +++------
 spec/models/budget/stats_spec.rb                         | 3 +--
 spec/models/geozone_spec.rb                              | 3 ++-
 spec/system/admin/budget_investments_spec.rb             | 6 ++----
 spec/system/emails_spec.rb                               | 3 ++-
 spec/system/legislation/processes_spec.rb                | 3 +--
 spec/system/legislation/summary_spec.rb                  | 3 +--
 36 files changed, 74 insertions(+), 65 deletions(-)

diff --git a/.rubocop.yml b/.rubocop.yml
index b0b25095f..7d16caa06 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -30,6 +30,9 @@ Layout/AssignmentIndentation:
 Layout/ClosingHeredocIndentation:
   Enabled: true
 
+Layout/ClosingParenthesisIndentation:
+  Enabled: true
+
 Layout/DotPosition:
   Enabled: true
 
@@ -108,6 +111,9 @@ Layout/MultilineHashBraceLayout:
 Layout/MultilineHashKeyLineBreaks:
   Enabled: true
 
+Layout/MultilineMethodCallBraceLayout:
+  Enabled: true
+
 Layout/MultilineMethodCallIndentation:
   Enabled: true
 
diff --git a/app/components/budgets/investments/form_component.html.erb b/app/components/budgets/investments/form_component.html.erb
index 9b28cb0ed..daa0aec4a 100644
--- a/app/components/budgets/investments/form_component.html.erb
+++ b/app/components/budgets/investments/form_component.html.erb
@@ -97,8 +97,7 @@
                         title: t("form.accept_terms_title"),
                         label: t("form.accept_terms",
                                  policy: link_to(t("form.policy"), "/privacy", target: "blank"),
-                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")
-                                ) %>
+                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")) %>
       </div>
     <% end %>
 
diff --git a/app/components/budgets/investments/my_ballot_component.html.erb b/app/components/budgets/investments/my_ballot_component.html.erb
index bf141f63e..307be97cc 100644
--- a/app/components/budgets/investments/my_ballot_component.html.erb
+++ b/app/components/budgets/investments/my_ballot_component.html.erb
@@ -25,7 +25,8 @@
   <p>
     <%= sanitize(ballot.change_vote_info(
         link: link_to(t("budgets.investments.index.sidebar.change_vote_link"),
-                      budget_ballot_path(budget)))) %>
+                      budget_ballot_path(budget))
+      )) %>
   </p>
 
   <ul class="ballot-list">
diff --git a/app/components/debates/form_component.html.erb b/app/components/debates/form_component.html.erb
index 3733960cd..851269d1b 100644
--- a/app/components/debates/form_component.html.erb
+++ b/app/components/debates/form_component.html.erb
@@ -45,8 +45,7 @@
                         title: t("form.accept_terms_title"),
                         label: t("form.accept_terms",
                                  policy: link_to(t("form.policy"), "/privacy", target: "blank"),
-                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")
-                                ) %>
+                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")) %>
       </div>
     <% end %>
 
diff --git a/app/components/proposals/form_component.html.erb b/app/components/proposals/form_component.html.erb
index 5a99d7e37..f6084f3b8 100644
--- a/app/components/proposals/form_component.html.erb
+++ b/app/components/proposals/form_component.html.erb
@@ -104,8 +104,7 @@
                         title: t("form.accept_terms_title"),
                         label: t("form.accept_terms",
                                  policy: link_to(t("form.policy"), "/privacy", target: "blank"),
-                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")
-                                ) %>
+                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")) %>
       </div>
     <% end %>
 
diff --git a/app/controllers/admin/budgets_controller.rb b/app/controllers/admin/budgets_controller.rb
index 6ce7b2721..c0facf84b 100644
--- a/app/controllers/admin/budgets_controller.rb
+++ b/app/controllers/admin/budgets_controller.rb
@@ -29,7 +29,8 @@ class Admin::BudgetsController < Admin::BaseController
     @budget.headings.each { |heading| Budget::Result.new(@budget, heading).delay.calculate_winners }
     redirect_to admin_budget_budget_investments_path(
                   budget_id: @budget.id,
-                  advanced_filters: ["winners"]),
+                  advanced_filters: ["winners"]
+                ),
                 notice: I18n.t("admin.budgets.winners.calculated")
   end
 
diff --git a/app/models/comment.rb b/app/models/comment.rb
index 762f1bdef..cad1bf8fe 100644
--- a/app/models/comment.rb
+++ b/app/models/comment.rb
@@ -148,7 +148,8 @@ class Comment < ApplicationRecord
     def validate_body_length
       validator = ActiveModel::Validations::LengthValidator.new(
         attributes: :body,
-        maximum: Comment.body_max_length)
+        maximum: Comment.body_max_length
+      )
       validator.validate(self)
     end
 
diff --git a/app/models/organization.rb b/app/models/organization.rb
index 62b510a4d..436521fd4 100644
--- a/app/models/organization.rb
+++ b/app/models/organization.rb
@@ -52,14 +52,16 @@ class Organization < ApplicationRecord
     def validate_name_length
       validator = ActiveModel::Validations::LengthValidator.new(
         attributes: :name,
-        maximum: Organization.name_max_length)
+        maximum: Organization.name_max_length
+      )
       validator.validate(self)
     end
 
     def validate_responsible_name_length
       validator = ActiveModel::Validations::LengthValidator.new(
         attributes: :responsible_name,
-        maximum: Organization.responsible_name_max_length)
+        maximum: Organization.responsible_name_max_length
+      )
       validator.validate(self)
     end
 end
diff --git a/app/models/user.rb b/app/models/user.rb
index 881ace74e..ecc2833e8 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -427,7 +427,8 @@ class User < ApplicationRecord
     def validate_username_length
       validator = ActiveModel::Validations::LengthValidator.new(
         attributes: :username,
-        maximum: User.username_max_length)
+        maximum: User.username_max_length
+      )
       validator.validate(self)
     end
 end
diff --git a/app/views/admin/officials/search.html.erb b/app/views/admin/officials/search.html.erb
index 5e361d9e7..8c5d8d013 100644
--- a/app/views/admin/officials/search.html.erb
+++ b/app/views/admin/officials/search.html.erb
@@ -37,7 +37,7 @@
               actions: [:edit],
               edit_path: edit_admin_official_path(user),
               edit_text: (t("admin.officials.search.make_official") unless user.official?)
-              ) %>
+            ) %>
           </td>
         </tr>
       <% end %>
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index eb84e2688..8cdffdce9 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -35,8 +35,7 @@
                            firefox: link_to(t("layouts.application.firefox"),
                                             "https://www.mozilla.org/firefox",
                                             title: t("shared.target_blank"),
-                                            target: "_blank")
-            )) %>
+                                            target: "_blank"))) %>
           </p>
         </div>
       <![endif]-->
diff --git a/app/views/legislation/proposals/_form.html.erb b/app/views/legislation/proposals/_form.html.erb
index 8d9760c08..7fa3d3a4b 100644
--- a/app/views/legislation/proposals/_form.html.erb
+++ b/app/views/legislation/proposals/_form.html.erb
@@ -66,8 +66,7 @@
                         title: t("form.accept_terms_title"),
                         label: t("form.accept_terms",
                                  policy: link_to(t("form.policy"), "/privacy", target: "blank"),
-                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")
-                                ) %>
+                                 conditions: link_to(t("form.conditions"), "/conditions", target: "blank")) %>
       <% end %>
     </div>
 
diff --git a/app/views/legislation/proposals/_proposal.html.erb b/app/views/legislation/proposals/_proposal.html.erb
index 5537fbba5..a2a985f44 100644
--- a/app/views/legislation/proposals/_proposal.html.erb
+++ b/app/views/legislation/proposals/_proposal.html.erb
@@ -26,7 +26,8 @@
                 url: legislation_process_proposal_path(
                        proposal.legislation_process_id,
                        proposal,
-                       anchor: "comments")
+                       anchor: "comments"
+                     )
               ) %>
 
               <% if proposal.author.hidden? || proposal.author.erased? %>
diff --git a/app/views/mailer/budget_investment_created.html.erb b/app/views/mailer/budget_investment_created.html.erb
index 12f945b41..7c0c1c8e6 100644
--- a/app/views/mailer/budget_investment_created.html.erb
+++ b/app/views/mailer/budget_investment_created.html.erb
@@ -20,7 +20,8 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.budget_investment_created.follow",
+      t(
+        "mailers.budget_investment_created.follow",
         link: link_to(t("mailers.budget_investment_created.follow_link"),
                       budgets_url,
                       style: css_for_mailer_link)
diff --git a/app/views/mailer/budget_investment_unfeasible.html.erb b/app/views/mailer/budget_investment_unfeasible.html.erb
index 0fd60982d..444861672 100644
--- a/app/views/mailer/budget_investment_unfeasible.html.erb
+++ b/app/views/mailer/budget_investment_unfeasible.html.erb
@@ -14,7 +14,8 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.budget_investment_unfeasible.new",
+      t(
+        "mailers.budget_investment_unfeasible.new",
         url: link_to(t("mailers.budget_investment_unfeasible.new_href"),
                      new_budget_investment_url(@investment.budget),
                      style: css_for_mailer_link)
diff --git a/app/views/mailer/comment.html.erb b/app/views/mailer/comment.html.erb
index 3c277bbd8..3121e2ce3 100644
--- a/app/views/mailer/comment.html.erb
+++ b/app/views/mailer/comment.html.erb
@@ -19,7 +19,8 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.config.unsubscribe_text",
+      t(
+        "mailers.config.unsubscribe_text",
         notifications: link_to(
           t("mailers.config.notifications_link"),
           edit_subscriptions_url(token: @token),
diff --git a/app/views/mailer/direct_message_for_receiver.html.erb b/app/views/mailer/direct_message_for_receiver.html.erb
index 33c64f941..5a8799607 100644
--- a/app/views/mailer/direct_message_for_receiver.html.erb
+++ b/app/views/mailer/direct_message_for_receiver.html.erb
@@ -22,11 +22,13 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.direct_message_for_receiver.unsubscribe_text",
+      t(
+        "mailers.direct_message_for_receiver.unsubscribe_text",
         notifications: link_to(
           t("mailers.config.notifications_link"),
           edit_subscriptions_url(token: @token),
-          style: css_for_mailer_link)
+          style: css_for_mailer_link
+        )
       ),
       attributes: %w[href style]
     ) %>
diff --git a/app/views/mailer/email_verification.html.erb b/app/views/mailer/email_verification.html.erb
index fe1525a3f..d4305d77e 100644
--- a/app/views/mailer/email_verification.html.erb
+++ b/app/views/mailer/email_verification.html.erb
@@ -10,11 +10,13 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.email_verification.instructions",
+      t(
+        "mailers.email_verification.instructions",
         verification_link: link_to(
           t("mailers.email_verification.click_here_to_verify"),
           email_url(email_verification_token: @token),
-          style: css_for_mailer_link)
+          style: css_for_mailer_link
+        )
       ),
       attributes: %w[href style]
     ) %>
diff --git a/app/views/mailer/evaluation_comment.html.erb b/app/views/mailer/evaluation_comment.html.erb
index fa95263ed..c1ae1b0eb 100644
--- a/app/views/mailer/evaluation_comment.html.erb
+++ b/app/views/mailer/evaluation_comment.html.erb
@@ -12,8 +12,7 @@
     <%= sanitize(
       t("mailers.evaluation_comment.new_comment_by",
         commenter: @email.comment.author.name,
-        investment: valuation_comments_link(@email.commentable)
-      )
+        investment: valuation_comments_link(@email.commentable))
     ) %>
   </p>
 
diff --git a/app/views/mailer/newsletter.html.erb b/app/views/mailer/newsletter.html.erb
index 7b7d5130d..8c6c3dfee 100644
--- a/app/views/mailer/newsletter.html.erb
+++ b/app/views/mailer/newsletter.html.erb
@@ -6,7 +6,8 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.config.unsubscribe_text",
+      t(
+        "mailers.config.unsubscribe_text",
         notifications: link_to(
           t("mailers.config.notifications_link"),
           edit_subscriptions_url(token: @token),
diff --git a/app/views/mailer/proposal_notification_digest.html.erb b/app/views/mailer/proposal_notification_digest.html.erb
index fee371793..bc2620d37 100644
--- a/app/views/mailer/proposal_notification_digest.html.erb
+++ b/app/views/mailer/proposal_notification_digest.html.erb
@@ -64,12 +64,14 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.proposal_notification_digest.unsubscribe_text",
+      t(
+        "mailers.proposal_notification_digest.unsubscribe_text",
         notifications: link_to(
           t("mailers.config.notifications_link"),
           edit_subscriptions_url(token: @token),
-          style: css_for_mailer_link)
-        ),
+          style: css_for_mailer_link
+        )
+      ),
       attributes: %w[href style]
     ) %>
   </p>
diff --git a/app/views/mailer/reply.html.erb b/app/views/mailer/reply.html.erb
index c25fcb1bc..2af83667a 100644
--- a/app/views/mailer/reply.html.erb
+++ b/app/views/mailer/reply.html.erb
@@ -19,7 +19,8 @@
 
   <p style="<%= css_for_mailer_text %>">
     <%= sanitize(
-      t("mailers.config.unsubscribe_text",
+      t(
+        "mailers.config.unsubscribe_text",
         notifications: link_to(
           t("mailers.config.notifications_link"),
           edit_subscriptions_url(token: @token),
diff --git a/app/views/organizations/registrations/new.html.erb b/app/views/organizations/registrations/new.html.erb
index 09b462f6d..b36fba601 100644
--- a/app/views/organizations/registrations/new.html.erb
+++ b/app/views/organizations/registrations/new.html.erb
@@ -30,8 +30,7 @@
                                terms: link_to(t("devise_views.users.registrations.new.terms_link"),
                                               "/conditions",
                                               title: t("shared.target_blank"),
-                                              target: "_blank")
-                              ) %>
+                                              target: "_blank")) %>
 
       <div class="small-12 medium-6 small-centered">
         <%= f.submit t("devise_views.organizations.registrations.new.submit"), class: "button expanded" %>
diff --git a/app/views/polls/_advanced_stats.html.erb b/app/views/polls/_advanced_stats.html.erb
index 5b66db75a..7bfd0f06f 100644
--- a/app/views/polls/_advanced_stats.html.erb
+++ b/app/views/polls/_advanced_stats.html.erb
@@ -7,7 +7,8 @@
     <% stats.channels.each do |channel| %>
       <%= number_with_info_tags(
         stats.send("total_participants_#{channel}"),
-        t("stats.polls.#{channel}_percentage",
+        t(
+          "stats.polls.#{channel}_percentage",
           percentage: number_to_stats_percentage(stats.send(:"total_participants_#{channel}_percentage"))
         ),
         html_class: channel
diff --git a/app/views/users/registrations/new.html.erb b/app/views/users/registrations/new.html.erb
index 6f028e199..0afca1696 100644
--- a/app/views/users/registrations/new.html.erb
+++ b/app/views/users/registrations/new.html.erb
@@ -37,10 +37,10 @@
       <%= f.check_box :terms_of_service,
                       title: t("devise_views.users.registrations.new.terms_title"),
                       label: t("devise_views.users.registrations.new.terms",
-                               terms: link_to(t("devise_views.users.registrations.new.terms_link"), "/conditions",
+                               terms: link_to(t("devise_views.users.registrations.new.terms_link"),
+                                              "/conditions",
                                               title: t("shared.target_blank"),
-                                              target: "_blank")
-                              ) %>
+                                              target: "_blank")) %>
 
       <div class="small-12 medium-6 small-centered">
         <%= f.submit t("devise_views.users.registrations.new.submit"), class: "button expanded" %>
diff --git a/app/views/verification/residence/new.html.erb b/app/views/verification/residence/new.html.erb
index e62e890f4..b528598f9 100644
--- a/app/views/verification/residence/new.html.erb
+++ b/app/views/verification/residence/new.html.erb
@@ -71,10 +71,10 @@
         <%= f.check_box :terms_of_service,
                         title: t("verification.residence.new.accept_terms_text_title"),
                         label: t("verification.residence.new.accept_terms_text",
-                                 terms_url: link_to(t("verification.residence.new.terms"), page_path("census_terms"),
+                                 terms_url: link_to(t("verification.residence.new.terms"),
+                                                    page_path("census_terms"),
                                                     title: t("shared.target_blank"),
-                                                    target: "_blank")
-                                ) %>
+                                                    target: "_blank")) %>
       </div>
 
       <div class="small-12 medium-3 clear">
diff --git a/spec/components/sdg/tag_list_component_spec.rb b/spec/components/sdg/tag_list_component_spec.rb
index 1139ae6ef..cc43c0c0e 100644
--- a/spec/components/sdg/tag_list_component_spec.rb
+++ b/spec/components/sdg/tag_list_component_spec.rb
@@ -4,8 +4,7 @@ describe SDG::TagListComponent do
   let(:debate) do
     create(:debate,
            sdg_goals: [SDG::Goal[3]],
-           sdg_targets: [SDG::Target[3.2], create(:sdg_local_target, code: "3.2.1")]
-          )
+           sdg_targets: [SDG::Target[3.2], create(:sdg_local_target, code: "3.2.1")])
   end
   let(:component) { SDG::TagListComponent.new(debate) }
 
diff --git a/spec/components/sdg/targets/plain_tag_list_component_spec.rb b/spec/components/sdg/targets/plain_tag_list_component_spec.rb
index 04bd08cad..722bfdb3f 100644
--- a/spec/components/sdg/targets/plain_tag_list_component_spec.rb
+++ b/spec/components/sdg/targets/plain_tag_list_component_spec.rb
@@ -3,8 +3,7 @@ require "rails_helper"
 describe SDG::Targets::PlainTagListComponent do
   let(:debate) do
     create(:debate,
-           sdg_targets: [SDG::Target[1.1], SDG::Target[3.2], create(:sdg_local_target, code: "3.2.1")]
-          )
+           sdg_targets: [SDG::Target[1.1], SDG::Target[3.2], create(:sdg_local_target, code: "3.2.1")])
   end
   let(:component) { SDG::Targets::PlainTagListComponent.new(debate) }
 
diff --git a/spec/components/sdg/targets/tag_list_component_spec.rb b/spec/components/sdg/targets/tag_list_component_spec.rb
index b2dcf1146..5ae7f7b89 100644
--- a/spec/components/sdg/targets/tag_list_component_spec.rb
+++ b/spec/components/sdg/targets/tag_list_component_spec.rb
@@ -3,8 +3,7 @@ require "rails_helper"
 describe SDG::Targets::TagListComponent do
   let(:debate) do
     create(:debate,
-           sdg_targets: [SDG::Target[1.1], SDG::Target[3.2], create(:sdg_local_target, code: "3.2.1")]
-          )
+           sdg_targets: [SDG::Target[1.1], SDG::Target[3.2], create(:sdg_local_target, code: "3.2.1")])
   end
   let(:component) { SDG::Targets::TagListComponent.new(debate) }
 
diff --git a/spec/components/shared/banner_component_spec.rb b/spec/components/shared/banner_component_spec.rb
index 0d3436329..ef36de26a 100644
--- a/spec/components/shared/banner_component_spec.rb
+++ b/spec/components/shared/banner_component_spec.rb
@@ -9,8 +9,7 @@ describe Shared::BannerComponent do
                     post_started_at: (Date.current - 4.days),
                     post_ended_at: (Date.current + 10.days),
                     background_color: "#FF0000",
-                    font_color: "#FFFFFF"
-                   )
+                    font_color: "#FFFFFF")
 
     render_inline Shared::BannerComponent.new(banner)
 
@@ -38,15 +37,13 @@ describe Shared::BannerComponent do
              web_sections: [WebSection.find_by!(name: "debates")],
              title: "First banner",
              description: "First description",
-             target_url: "/first_target"
-            )
+             target_url: "/first_target")
 
       create(:banner,
              web_sections: [WebSection.find_by!(name: "debates")],
              title: "Second banner",
              description: "Second description",
-             target_url: "/second_target"
-            )
+             target_url: "/second_target")
     end
 
     it "only renders one banner" do
diff --git a/spec/models/budget/stats_spec.rb b/spec/models/budget/stats_spec.rb
index d59ef6e7c..476969876 100644
--- a/spec/models/budget/stats_spec.rb
+++ b/spec/models/budget/stats_spec.rb
@@ -77,8 +77,7 @@ describe Budget::Stats do
 
     it "doesn't count nil user ids" do
       create(:budget_ballot_line, investment: investment,
-                                  ballot: create(:budget_ballot, budget: budget.reload, user: nil, physical: true)
-      )
+                                  ballot: create(:budget_ballot, budget: budget.reload, user: nil, physical: true))
 
       expect(stats.total_participants_vote_phase).to be 0
     end
diff --git a/spec/models/geozone_spec.rb b/spec/models/geozone_spec.rb
index 65c269610..c2b441f10 100644
--- a/spec/models/geozone_spec.rb
+++ b/spec/models/geozone_spec.rb
@@ -73,7 +73,8 @@ describe Geozone do
       expect(geozone.outline_points).to eq(
         [[-3.9259027239257, 40.8792937308316],
          [-3.9249047078766, 40.8788966596619],
-         [-3.9247799675785, 40.8789131852224]])
+         [-3.9247799675785, 40.8789131852224]]
+      )
     end
   end
 end
diff --git a/spec/system/admin/budget_investments_spec.rb b/spec/system/admin/budget_investments_spec.rb
index 1f1514990..3002ae987 100644
--- a/spec/system/admin/budget_investments_spec.rb
+++ b/spec/system/admin/budget_investments_spec.rb
@@ -937,8 +937,7 @@ describe "Admin budget investments", :admin do
                                  price: 1234,
                                  price_first_year: 1000,
                                  administrator: administrator,
-                                 valuators: [valuator]
-                                )
+                                 valuators: [valuator])
 
       visit admin_budget_budget_investments_path(budget_investment.budget)
 
@@ -1757,8 +1756,7 @@ describe "Admin budget investments", :admin do
              :winner,
              :visible_to_valuators,
              budget: budget,
-             author: create(:user, username: "Jon Doe")
-            )
+             author: create(:user, username: "Jon Doe"))
     end
     let(:default_columns) do
       %w[id title supports admin valuator geozone feasibility price
diff --git a/spec/system/emails_spec.rb b/spec/system/emails_spec.rb
index a439921b7..38f1ea940 100644
--- a/spec/system/emails_spec.rb
+++ b/spec/system/emails_spec.rb
@@ -496,7 +496,8 @@ describe "Emails" do
       expect(email.body.encoded).to include("This is a different body")
       expect(email).to have_body_text("To unsubscribe from these emails, visit")
       expect(email).to have_body_text(
-                        edit_subscriptions_path(token: user_with_newsletter_in_segment_2.subscriptions_token))
+                        edit_subscriptions_path(token: user_with_newsletter_in_segment_2.subscriptions_token)
+                      )
       expect(email).to have_body_text('and uncheck "Receive relevant information by email"')
     end
   end
diff --git a/spec/system/legislation/processes_spec.rb b/spec/system/legislation/processes_spec.rb
index a0b2829d1..6540ae3e5 100644
--- a/spec/system/legislation/processes_spec.rb
+++ b/spec/system/legislation/processes_spec.rb
@@ -401,8 +401,7 @@ describe "Legislation" do
         create(:milestone,
                milestoneable: process,
                description: "Something important happened",
-               publication_date: Date.new(2018, 3, 22)
-              )
+               publication_date: Date.new(2018, 3, 22))
 
         visit legislation_process_path(process)
 
diff --git a/spec/system/legislation/summary_spec.rb b/spec/system/legislation/summary_spec.rb
index 5bab2d84d..37618b9e1 100644
--- a/spec/system/legislation/summary_spec.rb
+++ b/spec/system/legislation/summary_spec.rb
@@ -22,8 +22,7 @@ describe "Legislation" do
   scenario "empty process" do
     process = create(:legislation_process, :empty,
                      result_publication_enabled: true,
-                     end_date: Date.current - 1.day
-    )
+                     end_date: Date.current - 1.day)
 
     visit summary_legislation_process_path(process)