Fix odd closing parenthesis by enabling the Layout/ClosingParenthesisIndentation cop

Author: kamipo

.rubocop.yml

@@ -49,6 +49,9 @@ Layout/CaseIndentation:
 Layout/ClosingHeredocIndentation:
   Enabled: true
 
+Layout/ClosingParenthesisIndentation:
+  Enabled: true
+
 # Align comments with method definitions.
 Layout/CommentIndentation:
   Enabled: true

Gemfile.lock

@@ -395,7 +395,7 @@ GEM
     retriable (3.1.2)
     rexml (3.2.5)
     rouge (3.26.0)
-    rubocop (1.17.0)
+    rubocop (1.18.1)
       parallel (~> 1.10)
       parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
@@ -411,7 +411,7 @@ GEM
     rubocop-performance (1.11.3)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    rubocop-rails (2.10.1)
+    rubocop-rails (2.11.2)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.7.0, < 2.0)

actionview/test/template/form_helper_test.rb

@@ -836,7 +836,7 @@ def test_radio_button
   def test_radio_button_is_checked_with_integers
     assert_dom_equal('<input checked="checked" id="post_secret_1" name="post[secret]" type="radio" value="1" />',
       radio_button("post", "secret", "1")
-   )
+    )
   end
 
   def test_radio_button_with_negative_integer_value
@@ -847,7 +847,7 @@ def test_radio_button_with_negative_integer_value
   def test_radio_button_respects_passed_in_id
     assert_dom_equal('<input checked="checked" id="foo" name="post[secret]" type="radio" value="1" />',
       radio_button("post", "secret", "1", id: "foo")
-   )
+    )
   end
 
   def test_radio_button_with_booleans
@@ -1486,7 +1486,7 @@ def test_auto_index
     assert_dom_equal(
       %{<input checked="checked" id="post_#{pid}_title_hello_world" name="post[#{pid}][title]" type="radio" value="Hello World" />},
       radio_button("post[]", "title", "Hello World")
-     )
+    )
     assert_dom_equal(
       %{<input id="post_#{pid}_title_goodbye_world" name="post[#{pid}][title]" type="radio" value="Goodbye World" />},
       radio_button("post[]", "title", "Goodbye World")
@@ -1510,7 +1510,7 @@ def test_auto_index_with_nil_id
     assert_dom_equal(
       %{<input checked="checked" name="post[#{pid}][title]" type="radio" value="Hello World" />},
        radio_button("post[]", "title", "Hello World", id: nil)
-     )
+    )
     assert_dom_equal(
       %{<input name="post[#{pid}][title]" type="radio" value="Goodbye World" />},
       radio_button("post[]", "title", "Goodbye World", id: nil)

activerecord/lib/active_record/relation/finder_methods.rb

@@ -435,7 +435,7 @@ def apply_join_dependency(eager_loading: group_values.empty?)
                 ), nil
               ).reflections
             )
-        )
+          )
           relation = skip_query_cache_if_necessary do
             klass.connection.distinct_relation_for_primary_key(relation)
           end

activerecord/test/cases/relations_test.rb

@@ -533,7 +533,8 @@ def test_find_on_hash_conditions
   end
 
   def test_joins_with_string_array
-    person_with_reader_and_post = Post.joins([
+    person_with_reader_and_post = Post.joins(
+      [
         "INNER JOIN categorizations ON categorizations.post_id = posts.id",
         "INNER JOIN categories ON categories.id = categorizations.category_id AND categories.type = 'SpecialCategory'"
       ]