Layout/ExtraSpacing

.rubocop.yml

@@ -34,21 +34,6 @@ Layout/AlignParameters:
     - 'app/models/participant.rb'
     - 'app/models/person.rb'
 
-# Offense count: 20
-# Cop supports --auto-correct.
-# Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.
-Layout/ExtraSpacing:
-  Exclude:
-    - 'app/controllers/activities_controller.rb'
-    - 'db/schema.rb'
-    - 'test/controllers/activities_controller_test.rb'
-    - 'test/controllers/api/activities_controller_test.rb'
-    - 'test/controllers/api/groups_controller_test.rb'
-    - 'test/controllers/api/people_controller_test.rb'
-    - 'test/controllers/groups_controller_test.rb'
-    - 'test/controllers/members_controller_test.rb'
-    - 'test/controllers/people_controller_test.rb'
-
 # Offense count: 2
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, IndentationWidth.

app/controllers/activities_controller.rb

@@ -8,7 +8,7 @@ class ActivitiesController < ApplicationController
     :destroy_subgroup, :immediate_subgroups, :clear_subgroups
   ]
   before_action :set_activity_and_group, only: has_activity_id
-  before_action :set_group,            except: has_activity_id
+  before_action :set_group, except: has_activity_id
 
   before_action :set_subgroup, only: [:update_subgroup, :destroy_subgroup]
   before_action :require_membership!
@@ -156,7 +156,7 @@ class ActivitiesController < ApplicationController
     @organizers = @activity.organizers
 
     @non_organizers_options = @non_organizers.map{|p| [p.person.full_name, p.id] }
-    @organizers_options     =     @organizers.map{|p| [p.person.full_name, p.id] }
+    @organizers_options     = @organizers.map{|p| [p.person.full_name, p.id] }
 
     @non_organizers_options.sort!
     @organizers_options.sort!

test/controllers/activities_controller_test.rb

@@ -17,7 +17,7 @@ class ActivitiesControllerTest < ActionDispatch::IntegrationTest
 
   test "should create activity" do
     assert_difference('Activity.count') do
-      post activities_url, params: { activity: {  } }
+      post activities_url, params: { activity: { } }
     end
 
     assert_redirected_to activity_url(Activity.last)
@@ -34,7 +34,7 @@ class ActivitiesControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update activity" do
-    patch activity_url(@activity), params: { activity: {  } }
+    patch activity_url(@activity), params: { activity: { } }
     assert_redirected_to activity_url(@activity)
   end
 

test/controllers/api/activities_controller_test.rb

@@ -17,7 +17,7 @@ class Api::ActivitiesControllerTest < ActionDispatch::IntegrationTest
 
   test "should create api_activity" do
     assert_difference('Api::Activity.count') do
-      post api_activities_url, params: { api_activity: {  } }
+      post api_activities_url, params: { api_activity: { } }
     end
 
     assert_redirected_to api_activity_url(Api::Activity.last)
@@ -34,7 +34,7 @@ class Api::ActivitiesControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update api_activity" do
-    patch api_activity_url(@api_activity), params: { api_activity: {  } }
+    patch api_activity_url(@api_activity), params: { api_activity: { } }
     assert_redirected_to api_activity_url(@api_activity)
   end
 

test/controllers/api/groups_controller_test.rb

@@ -17,7 +17,7 @@ class Api::GroupsControllerTest < ActionDispatch::IntegrationTest
 
   test "should create api_group" do
     assert_difference('Api::Group.count') do
-      post api_groups_url, params: { api_group: {  } }
+      post api_groups_url, params: { api_group: { } }
     end
 
     assert_redirected_to api_group_url(Api::Group.last)
@@ -34,7 +34,7 @@ class Api::GroupsControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update api_group" do
-    patch api_group_url(@api_group), params: { api_group: {  } }
+    patch api_group_url(@api_group), params: { api_group: { } }
     assert_redirected_to api_group_url(@api_group)
   end
 

test/controllers/api/people_controller_test.rb

@@ -17,7 +17,7 @@ class Api::PeopleControllerTest < ActionDispatch::IntegrationTest
 
   test "should create api_person" do
     assert_difference('Api::Person.count') do
-      post api_people_url, params: { api_person: {  } }
+      post api_people_url, params: { api_person: { } }
     end
 
     assert_redirected_to api_person_url(Api::Person.last)
@@ -34,7 +34,7 @@ class Api::PeopleControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update api_person" do
-    patch api_person_url(@api_person), params: { api_person: {  } }
+    patch api_person_url(@api_person), params: { api_person: { } }
     assert_redirected_to api_person_url(@api_person)
   end
 

test/controllers/groups_controller_test.rb

@@ -17,7 +17,7 @@ class GroupsControllerTest < ActionDispatch::IntegrationTest
 
   test "should create group" do
     assert_difference('Group.count') do
-      post groups_url, params: { group: {  } }
+      post groups_url, params: { group: { } }
     end
 
     assert_redirected_to group_url(Group.last)
@@ -34,7 +34,7 @@ class GroupsControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update group" do
-    patch group_url(@group), params: { group: {  } }
+    patch group_url(@group), params: { group: { } }
     assert_redirected_to group_url(@group)
   end
 

test/controllers/members_controller_test.rb

@@ -17,7 +17,7 @@ class MembersControllerTest < ActionDispatch::IntegrationTest
 
   test "should create member" do
     assert_difference('Member.count') do
-      post members_url, params: { member: {  } }
+      post members_url, params: { member: { } }
     end
 
     assert_redirected_to member_url(Member.last)
@@ -34,7 +34,7 @@ class MembersControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update member" do
-    patch member_url(@member), params: { member: {  } }
+    patch member_url(@member), params: { member: { } }
     assert_redirected_to member_url(@member)
   end
 

test/controllers/people_controller_test.rb

@@ -17,7 +17,7 @@ class PeopleControllerTest < ActionDispatch::IntegrationTest
 
   test "should create person" do
     assert_difference('Person.count') do
-      post people_url, params: { person: {  } }
+      post people_url, params: { person: { } }
     end
 
     assert_redirected_to person_url(Person.last)
@@ -34,7 +34,7 @@ class PeopleControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "should update person" do
-    patch person_url(@person), params: { person: {  } }
+    patch person_url(@person), params: { person: { } }
     assert_redirected_to person_url(@person)
   end