added tests and method update and destroy

test/controllers/api/v1/memberships_controller_test.rb

@@ -67,44 +67,44 @@ class Api::V1::MembershipsControllerTest < ActionDispatch::IntegrationTest
     assert_response :forbidden
   end
 
-  # # UPDATE
+  # UPDATE
-  # test "should update membership" do
+  test "should update membership" do
-  #   patch api_v1_membership_url(@membership),
+    patch api_v1_team_membership_url(@team, @membership),
-  #   headers: { Authorization: JsonWebToken.encode(user_id: @user.id) }, 
+    headers: { Authorization: JsonWebToken.encode(user_id: @user.id) }, 
-  #   params: { membership: { can_edit: true } },
+    params: { membership: { can_edit: true } },
-  #   as: :json
+    as: :json
-  #   assert_response :success
+    assert_response :success
-  # end
+  end
 
-  # test "should forbid update membership" do
+  test "should forbid update membership" do
-  #   patch api_v1_membership_url(@membership),
+    patch api_v1_team_membership_url(@team, @membership),
-  #   params: { membership: { can_edit: true } },
+    params: { membership: { can_edit: true } },
-  #   as: :json
+    as: :json
-  #   assert_response :forbidden
+    assert_response :forbidden
-  # end
+  end
 
-  # test "should forbid update read_only attributes - team_id" do
+  test "should forbid update read_only attributes - team_id" do
-  #   patch api_v1_membership_url(@membership),
+    patch api_v1_team_membership_url(@team, @membership),
-  #   headers: { Authorization: JsonWebToken.encode(user_id: @user.id) }, 
+    headers: { Authorization: JsonWebToken.encode(user_id: @user.id) }, 
-  #   params: { membership: { team_id: @team2.id, member_id: @user2.id } },
+    params: { membership: { team_id: @team2.id, member_id: @user2.id } },
-  #   as: :json
+    as: :json
-  #   assert_response :unprocessable_entity
+    assert_response :unprocessable_entity
-  # end
+  end
 
-  # # DESTROY
+  # DESTROY
-  # test "should destroy membership" do
+  test "should destroy membership" do
-  #   assert_difference('Membership.count', -1) do
+    assert_difference('Membership.count', -1) do
-  #     delete api_v1_membership_url(@membership), 
+      delete api_v1_team_membership_url(@team, @membership), 
-  #     headers: { Authorization: JsonWebToken.encode(user_id: @user.id) },
+      headers: { Authorization: JsonWebToken.encode(user_id: @user.id) },
-  #     as: :json
+      as: :json
-  #   end
+    end
-  #   assert_response :no_content
+    assert_response :no_content
-  # end
+  end
 
-  # test "should forbid destroy membership" do
+  test "should forbid destroy membership" do
-  #   assert_no_difference('Membership.count') do
+    assert_no_difference('Membership.count') do
-  #     delete api_v1_membership_url(@membership), as: :json
+      delete api_v1_team_membership_url(@team, @membership), as: :json
-  #   end
+    end
-  #   assert_response :forbidden
+    assert_response :forbidden
-  # end
+  end
 end