Obey rubocop

app/controllers/plans_controller.rb

@@ -60,7 +60,7 @@ def add_plan(id)
   end
 
   def check_user_owns_plan
-    render status: :bad_request, body: nil  if @plan.nil?
+    render status: :bad_request, body: nil if @plan.nil?
     render status: :forbidden, body: nil unless @user.plans.include?(@plan)
   end
 

config/routes.rb

@@ -31,7 +31,7 @@
     resources :items, only: %i[update destroy]
   end
 
-  resolve('PlanSchedule') { [:event, :item] }
+  resolve('PlanSchedule') { %i[event item] }
 
   resources :teams, except: :index do
     resources :members, only: %i[create update destroy]

test/controllers/items_controller_test.rb

@@ -10,28 +10,32 @@ def setup
 
   test 'Add new item to plan' do
     assert_difference('PlanSchedule.count', 1) do
-      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:six).id }
+      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+           params: { schedule_id: schedules(:six).id }
     end
     assert_redirected_to event_plan_path(plans(:one), event_name: events(:party).name)
   end
 
   test 'Add new item to plan with turbo frames' do
     assert_difference('PlanSchedule.count', 1) do
-      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:six).id }, as: :turbo_stream
+      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+           params: { schedule_id: schedules(:six).id }, as: :turbo_stream
     end
     assert_response :ok
   end
 
   test 'Reject add new item to plan when crossover item has exists' do
     assert_no_difference('PlanSchedule.count') do
-      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:one_crossover).id }
+      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+           params: { schedule_id: schedules(:one_crossover).id }
     end
     assert_redirected_to event_plan_path(plans(:one), event_name: events(:party).name)
   end
 
   test 'Reject add new item to plan when crossover item has exists with turbo frames' do
     assert_no_difference('PlanSchedule.count') do
-      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:one_crossover).id }, as: :turbo_stream
+      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+           params: { schedule_id: schedules(:one_crossover).id }, as: :turbo_stream
     end
     assert_redirected_to event_plan_path(plans(:one), event_name: events(:party).name)
   end
@@ -41,7 +45,8 @@ def setup
     plan.initial = true
     plan.save!
 
-    post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:six).id }
+    post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+         params: { schedule_id: schedules(:six).id }
     plan.reload
 
     assert_not plan.initial
@@ -52,22 +57,25 @@ def setup
     plan.initial = true
     plan.save!
 
-    post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:one_crossover).id }
+    post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+         params: { schedule_id: schedules(:one_crossover).id }
     plan.reload
 
     assert plan.initial
   end
 
   test 'Reject add item to plan when item is not belongs to event' do
     assert_no_difference('PlanSchedule.count') do
-      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id), params: { schedule_id: schedules(:dojo_one).id }
+      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:one).id),
+           params: { schedule_id: schedules(:dojo_one).id }
     end
     assert_response :not_found
   end
 
   test 'Reject add new item to plan that not owned' do
     assert_no_difference('PlanSchedule.count') do
-      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:two).id), params: { schedule_id: schedules(:six).id }
+      post event_plan_items_url(event_name: events(:party).name, plan_id: plans(:two).id),
+           params: { schedule_id: schedules(:six).id }
     end
     assert_response :forbidden
   end

test/controllers/plans_controller_test.rb

@@ -20,15 +20,18 @@ def setup
 
   test 'Create plan' do
     assert_difference('Plan.count') do
-      post event_plans_path(event_name: events(:dojo).name), params: { plan: { title: 'test', description: 'test', public: true, initial: false } }
+      post event_plans_path(event_name: events(:dojo).name),
+           params: { plan: { title: 'test', description: 'test', public: true, initial: false } }
     end
 
     assert_redirected_to event_schedules_path(event_name: events(:dojo).name)
   end
 
   test 'Create plan with first item' do
     assert_difference('PlanSchedule.count') do
-      post event_plans_path(event_name: events(:dojo).name), params: { plan: { title: 'test', description: 'test', public: true, initial: false, add_schedule_id: schedules(:dojo_one).id }}
+      post event_plans_path(event_name: events(:dojo).name),
+           params: { plan: { title: 'test', description: 'test', public: true, initial: false,
+                             add_schedule_id: schedules(:dojo_one).id } }
     end
 
     assert_redirected_to event_schedules_path(event_name: events(:dojo).name)