From d4173904a81ef984d50a40941172c492b91e8d91 Mon Sep 17 00:00:00 2001 From: Hector Correa Date: Mon, 24 Oct 2022 13:27:05 -0400 Subject: [PATCH] Adjust the tests to match the new structure of the class --- spec/services/work_upload_edit_service_spec.rb | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/spec/services/work_upload_edit_service_spec.rb b/spec/services/work_upload_edit_service_spec.rb index d8a5922ea..74fbdaf37 100644 --- a/spec/services/work_upload_edit_service_spec.rb +++ b/spec/services/work_upload_edit_service_spec.rb @@ -2,6 +2,7 @@ require "rails_helper" RSpec.describe WorkUploadsEditService do + let(:user) { FactoryBot.create :research_data_moderator} let(:work) { FactoryBot.create :draft_work } let(:uploaded_file) do fixture_file_upload("us_covid_2019.csv", "text/csv") @@ -32,7 +33,8 @@ let(:params) { { "work_id" => "" }.with_indifferent_access } it "returns all existing files" do - updated_work = described_class.update_precurated_file_list(work, params) + upload_service = described_class.new(work, user) + updated_work = upload_service.update_precurated_file_list(params) list = updated_work.pre_curation_uploads expect(list.map(&:filename)).to eq([uploaded_file.original_filename]) expect(a_request(:delete, attachment_url)).not_to have_been_made @@ -51,7 +53,8 @@ end it "returns all existing files except the deleted one" do - updated_work = described_class.update_precurated_file_list(work, params) + upload_service = described_class.new(work, user) + updated_work = upload_service.update_precurated_file_list(params) list = updated_work.pre_curation_uploads expect(list.map(&:filename)).to eq([uploaded_file2.original_filename]) expect(a_request(:delete, attachment_url)).to have_been_made.once @@ -67,7 +70,8 @@ let(:params) { { "work_id" => "", "replaced_uploads" => { "1" => uploaded_file4 } }.with_indifferent_access } it "replaces the correct file" do - updated_work = described_class.update_precurated_file_list(work, params) + upload_service = described_class.new(work, user) + updated_work = upload_service.update_precurated_file_list(params) list = updated_work.pre_curation_uploads # remeber order of the files will be alphabetical @@ -80,7 +84,8 @@ let(:params) { { "work_id" => "", "pre_curation_uploads" => [uploaded_file2, uploaded_file3] }.with_indifferent_access } it "replaces all the files" do - updated_work = described_class.update_precurated_file_list(work, params) + upload_service = described_class.new(work, user) + updated_work = upload_service.update_precurated_file_list(params) list = updated_work.reload.pre_curation_uploads expect(list.map(&:filename)).to eq([uploaded_file2.original_filename, uploaded_file3.original_filename]) expect(a_request(:delete, attachment_url)).to have_been_made.once @@ -91,7 +96,8 @@ let(:params) { { "work_id" => "", "pre_curation_uploads" => [uploaded_file, uploaded_file3] }.with_indifferent_access } it "replaces all the files" do - updated_work = described_class.update_precurated_file_list(work, params) + upload_service = described_class.new(work, user) + updated_work = upload_service.update_precurated_file_list(params) list = updated_work.pre_curation_uploads expect(list.map(&:filename)).to eq([uploaded_file.original_filename, uploaded_file3.original_filename])