Fix Rubocop offenses

Author: MrSerth

app/controllers/account_links_controller.rb

@@ -91,6 +91,6 @@ def set_user
 
   # Never trust parameters from the scary internet, only allow the following list through.
   def account_link_params
-    params.require(:account_link).permit(:push_url, :check_uuid_url, :api_key, :name, :proforma_version)
+    params.expect(account_link: %i[push_url check_uuid_url api_key name proforma_version])
   end
 end

app/controllers/collections_controller.rb

@@ -182,7 +182,7 @@ def collection_tasks_params
   end
 
   def collection_params
-    params.require(:collection).permit(:title, :task_ids, :visibility_level, :description,
-      collection_tasks_attributes: collection_tasks_params)
+    params.expect(collection: [:title, :task_ids, :visibility_level, :description,
+                               {collection_tasks_attributes: [collection_tasks_params]}])
   end
 end

app/controllers/comments_controller.rb

@@ -55,6 +55,6 @@ def load_and_authorize_comment
 
   # Never trust parameters from the scary internet, only allow the following list through.
   def comment_params
-    params.require(:comment).permit(:text)
+    params.expect(comment: [:text])
   end
 end

app/controllers/concerns/task_parameters.rb

@@ -10,21 +10,21 @@ def file_params
 
   def test_params
     [:id, :title, :testing_framework_id, :description, :internal_description, :test_type, :xml_id, :validity, :timeout, :_destroy,
-     :parent_id, {files_attributes: file_params}]
+     :parent_id, {files_attributes: [file_params]}]
   end
 
   def model_solution_params
-    [:id, :description, :internal_description, :xml_id, :_destroy, :parent_id, {files_attributes: file_params}]
+    [:id, :description, :internal_description, :xml_id, :_destroy, :parent_id, {files_attributes: [file_params]}]
   end
 
   def task_params
-    params.require(:task).permit(:title, :description, :internal_description, :parent_uuid, :language, :license_id,
-      :programming_language_id, :access_level, files_attributes: file_params, tests_attributes: test_params,
-      model_solutions_attributes: model_solution_params, label_names: []).tap {|parameters| fix_nested_files_params(parameters) }
+    params.expect(task: [:title, :description, :internal_description, :parent_uuid, :language, :license_id,
+                         :programming_language_id, :access_level, {files_attributes: [file_params], tests_attributes: [test_params],
+      model_solutions_attributes: [model_solution_params], label_names: []}]).tap {|parameters| fix_nested_files_params(parameters) }
   end
 
   def group_tasks_params
-    params.require(:group_tasks).permit(group_ids: [])
+    params.expect(group_tasks: [group_ids: []])
   end
 
   def import_confirm_params

app/controllers/groups_controller.rb

@@ -132,6 +132,6 @@ def set_user
 
   # Never trust parameters from the scary internet, only allow the following list through.
   def group_params
-    params.require(:group).permit(:name, :description)
+    params.expect(group: %i[name description])
   end
 end

app/controllers/labels_controller.rb

@@ -50,7 +50,7 @@ def search # rubocop:disable Metrics/AbcSize
   private
 
   def label_params
-    params.require(:label).permit(:color)
+    params.expect(label: [:color])
   end
 
   def load_and_authorize_label

app/controllers/messages_controller.rb

@@ -81,6 +81,6 @@ def load_and_authorize_user
 
   # Never trust parameters from the scary internet, only allow the following list through.
   def message_params
-    params.require(:message).permit(:text, :sender_id, :recipient_id, :sender_status, :recipient_status)
+    params.expect(message: %i[text sender_id recipient_id sender_status recipient_status])
   end
 end

app/controllers/ratings_controller.rb

@@ -26,7 +26,7 @@ def load_and_authorize_task
 
   # Never trust parameters from the scary internet, only allow the following list through.
   def rating_params
-    params.require(:rating).permit(Rating::CATEGORIES)
+    params.expect(rating: Rating::CATEGORIES)
   end
 
   def handle_own_rating