Use study group external_id

Author: arkirchner

app/mailers/report_mailer.rb

@@ -6,7 +6,7 @@ class ReportMailer < ApplicationMailer
 
   def report_content
     @reported_content = params.fetch(:reported_content)
-    exercise_id = @reported_content.exercise.uuid
+    exercise_id = @reported_content.submission.study_group&.external_id
     base_url = SETTINGS[:exercises_base_url]
     @exercise_url = "#{base_url}/#{exercise_id}" if exercise_id && base_url
 

spec/mailers/report_mailer_spec.rb

@@ -7,9 +7,10 @@
     subject(:mail) { described_class.with(reported_content:).report_content }
 
     context 'when an RfC is reported' do
-      let(:exercise) { create(:fibonacci, uuid: SecureRandom.uuid) }
+      let(:external_id) { '11111111-aaaa-1111-aaaa-111111111111' }
       let(:question) { 'Inappropriate content for RfC.' }
-      let(:reported_content) { create(:rfc, question:, exercise:) }
+      let(:submission) { create(:submission, study_group: create(:study_group, external_id:)) }
+      let(:reported_content) { create(:rfc, question:, submission:) }
 
       it 'sets the correct sender' do
         expect(mail.from).to include('[email protected]')
@@ -27,8 +28,8 @@
       it 'includes the exersice url for the LTI consumer' do
         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}")
+        expect(mail.text_part.body).to include("https://a.com/exersices/#{external_id}")
+        expect(mail.html_part.body).to include("https://a.com/exersices/#{external_id}")
       end
 
       it 'dose not include the exersice URL when not configured' do