Adding logic to prevent email being scheduled if user can start exam

Author: msquance-stem

app/jobs/cs_accelerator/check_next_steps_job.rb

@@ -3,11 +3,13 @@ class CheckNextStepsJob < ApplicationJob
     queue_as :default
 
     def perform(achievement_id)
+      programme = Programme.cs_accelerator
       achievement = Achievement.find(achievement_id)
       user = achievement.user
-      enrolment = user.user_programme_enrolments.find_by(programme_id: Programme.cs_accelerator.id)
+      enrolment = user.user_programme_enrolments.find_by(programme_id: programme.id)
 
       return unless enrolment
+      return if programme.enough_credits_for_test?(user)
       return if enrolment.current_state == "complete"
 
       AchievementMailer.with(user_id: user.id).completed_course.deliver_now

spec/jobs/cs_accelerator/check_next_steps_job_spec.rb

@@ -22,12 +22,12 @@
         enrolment
       end
 
-      it "sends online course completion emails" do
+      it "sends email for face-to-face course" do
         expect { described_class.perform_now(achievement.id) }
           .to change { ActionMailer::Base.deliveries.count }.by(1)
       end
 
-      it "sends face to face course completion emails" do
+      it "sends email for online course" do
         expect { described_class.perform_now(achievement_2.id) }
           .to change { ActionMailer::Base.deliveries.count }.by(1)
       end
@@ -36,7 +36,6 @@
     context "when the user has completed CSA" do
       before do
         user
-        allow_any_instance_of(Programmes::CSAccelerator).to receive(:enough_activities_for_test?).with(user).and_return(true)
         allow_any_instance_of(Programmes::CSAccelerator).to receive(:user_meets_completion_requirement?).with(user).and_return(true)
         enrolment.transition_to(:complete)
       end
@@ -51,6 +50,24 @@
           .not_to change { ActionMailer::Base.deliveries.count }
       end
     end
+
+    context "when the user has enough credits for the exam" do
+      before do
+        user
+        allow_any_instance_of(Programmes::CSAccelerator).to receive(:enough_credits_for_test?).with(user).and_return(true)
+      end
+
+      it "does not send the online course completion emails" do
+        expect { described_class.perform_now(achievement.id) }
+          .not_to change { ActionMailer::Base.deliveries.count }
+      end
+
+      it "does not send the face to face course completion emails" do
+        expect { described_class.perform_now(achievement_2.id) }
+          .not_to change { ActionMailer::Base.deliveries.count }
+      end
+    end
+
   end
 
   context "when the user is not enrolled on CSA" do