rubocop

Author: reemi1510

app/controllers/admin/urns_controller.rb

@@ -8,7 +8,7 @@ def index
   end
 
   def download
-    send_data urn_csv, 
+    send_data urn_csv,
               type: 'text/csv',
               disposition: 'attachment',
               filename: "customer_urns_#{Time.zone.today}.csv"
@@ -25,4 +25,4 @@ def urn_csv
       end
     end
   end
-end
+end

spec/features/admin_can_search_urns_spec.rb

@@ -2,9 +2,12 @@
 
 RSpec.feature 'Admin can search for URNs' do
   before do
-    @customer1 = FactoryBot.create(:customer, urn: '123', name: 'Customer One', postcode: 'AB1 2CD', sector: :central_government)
-    @customer2 = FactoryBot.create(:customer, urn: '456', name: 'Customer Two', postcode: 'EF3 4GH', sector: :wider_public_sector)
-    @customer3 = FactoryBot.create(:customer, urn: '789', name: 'Customer Three', postcode: 'IJ5 6KL', sector: :wider_public_sector)
+    @customer1 = FactoryBot.create(:customer, urn: '123', name: 'Customer One', postcode: 'AB1 2CD',
+sector: :central_government)
+    @customer2 = FactoryBot.create(:customer, urn: '456', name: 'Customer Two', postcode: 'EF3 4GH',
+sector: :wider_public_sector)
+    @customer3 = FactoryBot.create(:customer, urn: '789', name: 'Customer Three', postcode: 'IJ5 6KL',
+sector: :wider_public_sector)
     sign_in_as_admin
   end
 
@@ -41,4 +44,4 @@
     expect(page).to_not have_content '456'
     expect(page).to have_content '789'
   end
-end
+end

spec/requests/admin/urns_spec.rb

@@ -9,8 +9,8 @@
     mock_sso_with(email: 'admin@example.com')
     get '/auth/google_oauth2/callback'
   end
-  
-  describe  'GET /admin/urns' do
+
+  describe 'GET /admin/urns' do
     it 'renders the URN search page' do
       get admin_urns_path
 
@@ -22,15 +22,20 @@
   end
 
   describe 'GET /admin/urns/download' do
-    let!(:active_customer) { create(:customer, urn: '123', name: 'Active Customer One', postcode: 'AB1 2CD', sector: :central_government) }
-    let!(:deleted_customer) { create(:customer, urn: '456', name: 'Deleted Customer', postcode: 'IJ5 6KL', sector: :wider_public_sector, deleted: true) }
+    let!(:active_customer) do
+      create(:customer, urn: '123', name: 'Active Customer One', postcode: 'AB1 2CD', sector: :central_government)
+    end
+    let!(:deleted_customer) do
+      create(:customer, urn: '456', name: 'Deleted Customer', postcode: 'IJ5 6KL', sector: :wider_public_sector,
+     deleted: true)
+    end
 
     it 'returns a CSV file with active customers' do
       get download_admin_urns_path
 
       expect(response).to have_http_status(:ok)
       expect(response.headers['Content-Type']).to include('text/csv')
-      expect(response.headers['Content-Disposition']).to include("attachment; filename=\"customer_urns_#{Time.zone.today}.csv\"")
+      expect(response.headers['Content-Disposition']).to include("filename=\"customer_urns_#{Time.zone.today}.csv\"")
 
       csv = CSV.parse(response.body, headers: true)
       expect(csv.headers).to eq(['URN', 'CustomerName', 'PostCode', 'Sector', 'Published'])
@@ -49,4 +54,4 @@
       end
     end
   end
-end
+end