Merge branch 'collection_patch_test' into 'master'

Added collection update test

See merge request !317

test/controllers/v1/collections_controller_test.rb

@@ -42,6 +42,42 @@ class V1::CollectionsControllerTest < ActionController::TestCase
     assert_response :unauthorized
   end
 
+  test 'should user patch collection to update and return :ok code' do
+    auth_request users(:john)
+    post :update, params: {
+        id: collections(:ufpr).id,
+        collection: {
+          name: 'my updated collection',
+          description: 'testing to update collection',
+          owner_type: 'User', owner_id: users(:john).id,
+          items:
+            [
+                {id: learning_objects(:one).id, type: "LearningObject", order: 1},
+                {id: collections(:ufpr).id, type: "Collection", order: 3}
+            ]
+        }
+      }
+    assert_response :ok
+  end
+
+  test 'should user patch collection to update and return :unauthorized code' do
+    auth_request users(:john)
+    post :update, params: {
+        id: collections(:ufpr).id,
+        collection: {
+          name: 'my updated collection',
+          description: 'testing to update collection',
+          owner_type: 'User', owner_id: users(:john).id,
+          items:
+            [
+                {id: learning_objects(:one).id, type: "LearningObject", order: 1},
+                {id: collections(:ufpr).id, type: "Collection", order: 3}
+            ]
+        }
+      }
+    assert_response :unauthorized
+  end
+
   test 'should user add items to collection and return :ok code' do
     auth_request users(:john)
     post :add_object, params: { id: collections(:ufpr).id, collection: {items: [{id: learning_objects(:lo_metadata2).id, type: 'LearningObject'}] }}
@@ -67,7 +103,6 @@ class V1::CollectionsControllerTest < ActionController::TestCase
     assert_response :unprocessable_entity
   end
 
-
   test 'should user tagging a collection' do
     auth_request users(:jack)
     tag = tags(:tag_three).name