make reactivating_soon message refer to the soonest, not most recently created

app/models/location.rb

@@ -220,8 +220,8 @@ class Location < ApplicationRecord
     location_deactivation_periods.deactivations_without_reactivation.first
   end
 
-  def last_deactivation_before(date)
+  def soonest_reactivation(date)
-    location_deactivation_periods.where("deactivation_date <= ?", date).order("created_at").last
+    location_deactivation_periods.deactivations_with_reactivation.where(date < reactivation_date).order(reactivation_date).first
   end
 
   def status

app/models/scheme.rb

@@ -315,8 +315,8 @@ class Scheme < ApplicationRecord
     scheme_deactivation_periods.deactivations_without_reactivation.first
   end
 
-  def last_deactivation_before(date)
+  def soonest_reactivation(date)
-    scheme_deactivation_periods.where("deactivation_date <= ?", date).order("created_at").last
+    scheme_deactivation_periods.deactivations_with_reactivation.where(date < reactivation_date).order(reactivation_date).first
   end
 
   def last_deactivation_date

app/models/validations/shared_validations.rb

@@ -73,7 +73,7 @@ module Validations::SharedValidations
     status = resource.status_at(date)
     return unless %i[reactivating_soon activating_soon deactivated].include?(status)
 
-    closest_reactivation = resource.last_deactivation_before(date)
+    closest_reactivation = resource.soonest_reactivation(date)
     open_deactivation = if resource.is_a?(Location)
                           resource.open_deactivation || resource.scheme.open_deactivation
                         else