Don't review changesets waiting to be processed

MAV-2604

Add tests for MAV-2604

Author: misaka

app/jobs/review_patient_changeset_job.rb

@@ -17,8 +17,7 @@ def perform(patient_changeset_id)
 
     import = patient_changeset.import
 
-    unless import.changesets.calculating_review.any? ||
-             import.changesets.import_invalid.any?
+    if all_jobs_finished_and_import_valid(import)
       if import.is_a?(ClassImport)
         ReviewClassImportSchoolMoveJob.perform_later(import.id)
       elsif import.calculating_re_review?
@@ -28,4 +27,11 @@ def perform(patient_changeset_id)
       end
     end
   end
+
+  private
+
+  def all_jobs_finished_and_import_valid(import)
+    import.changesets.still_processing.none? &&
+      import.changesets.import_invalid.none?
+  end
 end

app/models/patient_changeset.rb

@@ -116,6 +116,8 @@ class PatientChangeset < ApplicationRecord
           ).where.not(status: :processed)
         end
 
+  scope :still_processing, -> { where(status: %i[pending calculating_review]) }
+
   def self.from_import_row(row:, import:, row_number:)
     create!(
       import:,