From e3f55349c838e73081b020ca512fea9048db4909 Mon Sep 17 00:00:00 2001 From: Phil Lee Date: Wed, 14 Dec 2022 15:19:10 +0000 Subject: [PATCH] coverage for bulk upload filename on results --- spec/factories/bulk_upload.rb | 1 + .../bulk_upload_lettings_results_controller_spec.rb | 7 +++++++ 2 files changed, 8 insertions(+) diff --git a/spec/factories/bulk_upload.rb b/spec/factories/bulk_upload.rb index 26b80c172..d9721988b 100644 --- a/spec/factories/bulk_upload.rb +++ b/spec/factories/bulk_upload.rb @@ -6,5 +6,6 @@ FactoryBot.define do log_type { BulkUpload.log_types.values.sample } year { 2022 } identifier { SecureRandom.uuid } + sequence(:filename) { |n| "bulk-upload-#{n}.csv" } end end diff --git a/spec/requests/bulk_upload_lettings_results_controller_spec.rb b/spec/requests/bulk_upload_lettings_results_controller_spec.rb index c9e384261..673c4f5bc 100644 --- a/spec/requests/bulk_upload_lettings_results_controller_spec.rb +++ b/spec/requests/bulk_upload_lettings_results_controller_spec.rb @@ -23,5 +23,12 @@ RSpec.describe BulkUploadLettingsResultsController, type: :request do expect(response).to be_successful expect(response.body).to include("We found 2 errors in your file") end + + it "renders filename of the upload" do + get "/lettings-logs/bulk-upload-results/#{bulk_upload.id}" + + expect(response).to be_successful + expect(response.body).to include(bulk_upload.filename) + end end end