diff --git a/spec/requests/bulk_upload_lettings_results_controller_spec.rb b/spec/requests/bulk_upload_lettings_results_controller_spec.rb index 91d7a0742..77ee51d98 100644 --- a/spec/requests/bulk_upload_lettings_results_controller_spec.rb +++ b/spec/requests/bulk_upload_lettings_results_controller_spec.rb @@ -63,6 +63,58 @@ RSpec.describe BulkUploadLettingsResultsController, type: :request do end end + describe "GET /lettings-logs/bulk-upload-results/:ID/deletion-report" do + it "renders year combo" do + get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}/deletion-report" + + expect(response).to be_successful + expect(response.body).to include("Bulk upload for lettings (2022/23)") + end + + it "renders the bulk upload filename" do + get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}/deletion-report" + + expect(response.body).to include(bulk_upload.filename) + end + + context "when viewed by support user" do + before do + allow(support_user).to receive(:need_two_factor_authentication?).and_return(false) + end + + let(:viewing_user) { support_user } + + it "is accessible" do + get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}/deletion-report" + + expect(response).to be_successful + expect(response.body).to include(bulk_upload.filename) + end + end + + context "when viewed by some other random user" do + let(:other_user) { create(:user) } + let(:viewing_user) { other_user } + + it "is not accessible" do + get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}/deletion-report" + expect(response).to be_unauthorized + end + end + + context "when viewed by another user in the same org" do + let(:other_user) { create(:user, organisation: user.organisation) } + let(:viewing_user) { other_user } + + it "is accessible" do + get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}/deletion-report" + + expect(response).to be_successful + expect(response.body).to include(bulk_upload.filename) + end + end + end + describe "GET /lettings-logs/bulk-upload-results/:ID" do it "renders correct year" do get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}" diff --git a/spec/requests/bulk_upload_lettings_resume_controller_spec.rb b/spec/requests/bulk_upload_lettings_resume_controller_spec.rb index 009ada45c..bf1019200 100644 --- a/spec/requests/bulk_upload_lettings_resume_controller_spec.rb +++ b/spec/requests/bulk_upload_lettings_resume_controller_spec.rb @@ -42,6 +42,7 @@ RSpec.describe BulkUploadLettingsResumeController, type: :request do expect(response.body).to include("Bulk upload for lettings") expect(response.body).to include("2022/23") + expect(response.body).to include("View the error report") expect(response.body).to include("How would you like to fix the errors?") expect(response.body).to include(bulk_upload.filename) expect(response.body).not_to include("Cancel") @@ -124,6 +125,9 @@ RSpec.describe BulkUploadLettingsResumeController, type: :request do expect(response).to be_successful expect(response.body).to include("Are you sure you want to upload all logs from this bulk upload?") + expect(response.body).to include("View the error report") + expect(response.body).to include("2 answers will be deleted because they are invalid.") + expect(response.body).to include("See which answers will be deleted") end it "sets no cache headers" do