Use the correct parameters for the upload set form

Fixes samvera-deprecated/sufia#1434
samvera-deprecated · Dec 10, 2015 · b66d924 · b66d924
1 parent 55686dc
commit b66d924
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/app/controllers/concerns/curation_concerns/upload_sets_controller_behavior.rb b/app/controllers/concerns/curation_concerns/upload_sets_controller_behavior.rb
@@ -43,7 +43,7 @@ def create_update_job
         UploadSetUpdateJob.perform_later(current_user.user_key,
                                          params[:id],
                                          params[:title],
-                                         edit_form_class.model_attributes(params[:work]),
+                                         edit_form_class.model_attributes(params[:upload_set]),
                                          params[:visibility])
       end
   end

diff --git a/spec/controllers/upload_sets_controller_spec.rb b/spec/controllers/upload_sets_controller_spec.rb
@@ -14,7 +14,7 @@
       it "enqueues a upload_set job and redirects to generic_works list with a flash message" do
         expect(UploadSetUpdateJob).to receive(:perform_later).with(user.user_key, upload_set.id, { '1' => 'foo' },
                                                                    { tag: [] }, 'open').once
-        post :update, id: upload_set.id, title: { '1' => 'foo' }, visibility: 'open', work: { tag: [""] }
+        post :update, id: upload_set.id, title: { '1' => 'foo' }, visibility: 'open', upload_set: { tag: [""] }
         expect(response).to redirect_to routes.url_helpers.curation_concerns_generic_works_path
         expect(flash[:notice]).to include("Your files are being processed")
       end
@@ -25,20 +25,20 @@
       let!(:work) { create(:generic_work, user: user, upload_set: upload_set) }
 
       it "they can set public read access" do
-        post :update, id: upload_set, visibility: "open", work: { tag: [""] }
+        post :update, id: upload_set, visibility: "open", upload_set: { tag: [""] }
         expect(work.reload.read_groups).to eq ['public']
       end
 
       it "they can set metadata like title" do
-        post :update, id: upload_set, work: { tag: ["footag", "bartag"] }, title: { work.id => ["New Title"] }
+        post :update, id: upload_set, upload_set: { tag: ["footag", "bartag"] }, title: { work.id => ["New Title"] }
         work.reload
         expect(work.title).to eq ["New Title"]
         # TODO: is order important?
         expect(work.tag).to include("footag", "bartag")
       end
 
       it "they cannot set any tags" do
-        post :update, id: upload_set, work: { tag: [""] }
+        post :update, id: upload_set, upload_set: { tag: [""] }
         expect(work.reload.tag).to be_empty
       end
     end
@@ -48,8 +48,8 @@
       let!(:work) { create(:generic_work, title: ['Original Title'], upload_set: upload_set) }
 
       it "they cannot modify the object" do
-        post :update, id: upload_set, "work" => { "tag" => [""] },
-                      "title" => { work.id => "Title Won't Change" }
+        post :update, id: upload_set, upload_set: { "tag" => [""] },
+                      title: { work.id => "Title Won't Change" }
         work.reload
         expect(work.title).to eq ["Original Title"]
       end