[ExcelAnalyzer] add PII output to report email

mysociety · May 24, 2024 · abe7b77 · abe7b77
1 parent b64239d
commit abe7b77
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/views/excel_analyzer/notifier_mailer/report.text.erb b/lib/views/excel_analyzer/notifier_mailer/report.text.erb
@@ -6,4 +6,6 @@ Admin attachment URL: <%= edit_admin_foi_attachment_url(@foi_attachment) %>
 
 Excel Analyzer metadata: <%= JSON.pretty_generate(@metadata[:excel]) %>
 
+PII Badger metadata: <%= JSON.pretty_generate(@metadata[:pii_badger]) %>
+
 Please review the file carefully.
diff --git a/spec/excel_analyzer/mailers/notifier_mailer_spec.rb b/spec/excel_analyzer/mailers/notifier_mailer_spec.rb
@@ -46,7 +46,10 @@
   end
 
   it 'includes the metadata in the body' do
-    allow(blob).to receive(:metadata).and_return(excel: { foo: 'bar' })
+    allow(blob).to receive(:metadata).and_return(
+      excel: { foo: 'bar' }, pii_badger: { baz: 'qux' }
+    )
     expect(mail.body).to include('"foo": "bar"')
+    expect(mail.body).to include('"baz": "qux"')
   end
 end
Original file line number	Diff line number	Diff line change
Expand Up		@@ -6,4 +6,6 @@ Admin attachment URL: <%= edit_admin_foi_attachment_url(@foi_attachment) %>

		Excel Analyzer metadata: <%= JSON.pretty_generate(@metadata[:excel]) %>

		PII Badger metadata: <%= JSON.pretty_generate(@metadata[:pii_badger]) %>

		Please review the file carefully.