(feature) Completing a submission also completes the associated task

After we have a valid and completed submission, the task is also
complete.

Author: leeky

spec/models/submission_completion_spec.rb

@@ -3,29 +3,40 @@
 RSpec.describe SubmissionCompletion do
   describe '#perform' do
     let(:complete_submission) { SubmissionCompletion.new(submission) }
+    let(:task) { FactoryBot.create(:task, status: :in_progress) }
 
     context 'given an "in review" submission' do
       context 'with validated entries' do
-        let(:submission) { FactoryBot.create(:submission_with_validated_entries, aasm_state: 'in_review') }
+        let(:submission) { FactoryBot.create(:submission_with_validated_entries, aasm_state: 'in_review', task: task) }
 
         it 'transitions the submission to completed' do
           complete_submission.perform!
 
           expect(submission).to be_completed
         end
+
+        it 'transitions the task to completed' do
+          complete_submission.perform!
+
+          expect(task).to be_completed
+        end
       end
 
       context 'with some invalid entries' do
-        let(:submission) { FactoryBot.create(:submission_with_invalid_entries) }
+        let(:submission) { FactoryBot.create(:submission_with_invalid_entries, task: task) }
 
         it 'leaves the submission in the "in_review" state' do
           expect { complete_submission.perform! }.not_to change { submission.aasm_state }
         end
+
+        it 'leaves the task in the "in_progress" state' do
+          expect { complete_submission.perform! }.not_to change { task.status }
+        end
       end
     end
 
     context 'given a "processing" submission' do
-      let(:submission) { FactoryBot.create(:submission_with_unprocessed_entries, aasm_state: :processing) }
+      let(:submission) { FactoryBot.create(:submission_with_unprocessed_entries, aasm_state: :processing, task: task) }
 
       it 'leaves the submission in its current state' do
         expect { complete_submission.perform! }.not_to change { submission.aasm_state }

spec/requests/v1/submissions_spec.rb

@@ -244,9 +244,12 @@
 
   describe 'POST /submissions/:submission_id/complete' do
     it 'marks the submission as complete' do
+      task = FactoryBot.create(:task, status: :in_progress)
+
       submission = FactoryBot.create(
         :submission_with_validated_entries,
-        aasm_state: :in_review
+        aasm_state: :in_review,
+        task: task
       )
 
       post "/v1/submissions/#{submission.id}/complete"
@@ -256,6 +259,7 @@
       submission.reload
 
       expect(submission).to be_completed
+      expect(submission.task).to be_completed
     end
   end
 end