Ensure all school move attributes are reset

This fixes an issue where if a patient already had a school move, when
we set the `school`, we don't clear our the `team_id` and
`home_educated` columns, leading to a validation error.

I thought about adding tests for this, but this code is being removed in
a follow up PR
(#6373)
so I thought it was fine to leave it without. There has been a Sentry
issue raised for this, so we can check to see that those stop.

Sentry-Issue: 7361583189

Author: thomasleese

app/models/patient_changeset.rb

@@ -290,9 +290,12 @@ def school_move
           school_move =
             patient.school_moves.includes(:school_teams).first ||
               SchoolMove.new(patient:)
+
+          # TODO: Simplify this once we no longer have changesets with
+          #  `school_id` set to `nil`.
           school_move.assign_attributes(
-            # TODO: Simplify this once we no longer have changesets with
-            #  `school_id` set to `nil`.
+            academic_year:,
+            home_educated: nil,
             school:
               school ||
                 (
@@ -302,9 +305,10 @@ def school_move
                     team.unknown_school
                   end
                 ),
-            academic_year:,
+            team_id: nil,
             source: school_move_source
           )
+
           # TODO: Figure out why this is necessary.
           school_move.strict_loading!(false)
           school_move