Check if authentication information is absent if not available.

Author: arkirchner

app/mailers/report_mailer.rb

@@ -6,7 +6,9 @@ class ReportMailer < ApplicationMailer
 
   def report_content
     @reported_content = params.fetch(:reported_content)
-    @exercise_url = "#{SETTINGS[:exercises_base_url]}/#{@reported_content.exercise.uuid}"
+    exercise_id = @reported_content.exercise.uuid
+    base_url = SETTINGS[:exercises_base_url]
+    @exercise_url = "#{base_url}/#{exercise_id}" if exercise_id && base_url
 
     mail(subject: I18n.t('report_mailer.report_content.subject', content_name: @reported_content.model_name.human))
   end

app/views/report_mailer/report_content.text.slim

@@ -8,4 +8,5 @@
 == "\n\n"
 - if @exercise_url.present?
   == t('.authentication')
+  == "\n\n"
   == @exercise_url

spec/mailers/report_mailer_spec.rb

@@ -7,9 +7,9 @@
     subject(:mail) { described_class.with(reported_content:).report_content }
 
     context 'when an RfC is reported' do
-      let(:exercise) { reported_content.exercise }
+      let(:exercise) { create(:fibonacci, uuid: SecureRandom.uuid) }
       let(:question) { 'Inappropriate content for RfC.' }
-      let(:reported_content) { create(:rfc, question:) }
+      let(:reported_content) { create(:rfc, question:, exercise:) }
 
       it 'sets the correct sender' do
         expect(mail.from).to include('[email protected]')
@@ -25,10 +25,16 @@
       end
 
       it 'includes the exersice url for the LTI consumer' do
-        stub_const('ReportMailer::SETTINGS', {exercises_base_url: 'https://a.com/exersices/'})
+        stub_const('ReportMailer::SETTINGS', {exercises_base_url: 'https://a.com/exersices'})
+
         expect(mail.text_part.body).to include("https://a.com/exersices/#{exercise.uuid}")
         expect(mail.html_part.body).to include("https://a.com/exersices/#{exercise.uuid}")
       end
+
+      it 'dose not include the exersice URL when not configured' do
+        expect(mail.text_part.body).not_to include(I18n.t('report_mailer.report_content.authentication'))
+        expect(mail.html_part.body).not_to include(I18n.t('report_mailer.report_content.authentication'))
+      end
     end
   end
 end