Write specs for the new helper methods

Author: tiagopog

spec/controllers/users_controller_spec.rb

@@ -282,21 +282,31 @@
 
   describe '#update' do
     let(:user) { User.first }
+    let(:post) { user.posts.first }
 
     let(:update_params) do
       user_params.tap do |params|
         params[:data][:id] = user.id
         params[:data][:attributes].merge!(first_name: 'Yukihiro')
+        params[:data][:relationships] = relationship_params
         params.merge!(id: user.id)
       end
     end
 
+    let(:relationship_params) do
+      { posts: { data: [{ id: post.id, type: "posts" }] } }
+    end
+
     it 'update an existing user' do
       patch :update, update_params
+
       expect(response).to have_http_status :ok
       expect(response).to have_primary_data('users')
       expect(response).to have_data_attributes(fields)
       expect(data['attributes']['first_name']).to eq(user_params[:data][:attributes][:first_name])
+
+      expect(user.reload.posts.count).to eq(1)
+      expect(user.posts.first).to eq(post)
     end
 
     context 'when resource was not found' do

spec/support/controllers.rb

@@ -86,7 +86,7 @@ def create
   # PATCH /users/:id
   def update
     user = User.find(params[:id])
-    if user.update(resource_params)
+    if user.update(resource_params) && update_relationships(user)
       jsonapi_render json: user
     else
       # Example of error rendering for exceptions or any object
@@ -97,7 +97,8 @@ def update
 
   private
 
-  def user_params
-    params.require(:data).require(:attributes).permit(:first_name, :last_name, :admin)
+  def update_relationships(user)
+    return user if relationship_params[:posts].blank?
+    user.posts = Post.where(id: relationship_params[:posts])
   end
 end