Allow triage team to be optional

This updates the `team_id` column on triages to be `NOT NULL`, allowing
it to not be specified when we don't have a suitable user.

The only place this will be set to `NULL` for now is when creating the
automatic next dose delay triage.

Jira-Issue: MAV-2991

Author: thomasleese

app/models/triage.rb

@@ -17,7 +17,7 @@
 #  updated_at              :datetime         not null
 #  patient_id              :bigint           not null
 #  performed_by_user_id    :bigint
-#  team_id                 :bigint           not null
+#  team_id                 :bigint
 #
 # Indexes
 #
@@ -41,7 +41,7 @@ class Triage < ApplicationRecord
   audited associated_with: :patient
 
   belongs_to :patient
-  belongs_to :team
+  belongs_to :team, optional: true
   belongs_to :performed_by,
              class_name: "User",
              foreign_key: :performed_by_user_id,

app/policies/triage_policy.rb

@@ -11,7 +11,7 @@ def update?
 
   class Scope < ApplicationPolicy::Scope
     def resolve
-      scope.where(team: user.selected_team)
+      scope.where(team: [nil, user.selected_team])
     end
   end
 end

db/migrate/20260108150039_make_triage_team_null.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class MakeTriageTeamNull < ActiveRecord::Migration[8.1]
+  def change
+    change_column_null :triages, :team_id, true
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.1].define(version: 2026_01_08_143705) do
+ActiveRecord::Schema[8.1].define(version: 2026_01_08_150039) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_catalog.plpgsql"
   enable_extension "pg_trgm"
@@ -896,7 +896,7 @@
     t.bigint "performed_by_user_id"
     t.enum "programme_type", null: false, enum_type: "programme_type"
     t.integer "status", null: false
-    t.bigint "team_id", null: false
+    t.bigint "team_id"
     t.datetime "updated_at", null: false
     t.integer "vaccine_method"
     t.boolean "without_gelatine"

spec/factories/triages.rb

@@ -17,7 +17,7 @@
 #  updated_at              :datetime         not null
 #  patient_id              :bigint           not null
 #  performed_by_user_id    :bigint
-#  team_id                 :bigint           not null
+#  team_id                 :bigint
 #
 # Indexes
 #
@@ -38,8 +38,6 @@
     patient
     programme { Programme.sample }
 
-    team
-
     notes { "" }
 
     academic_year { created_at&.to_date&.academic_year || AcademicYear.current }

spec/models/triage_spec.rb

@@ -17,7 +17,7 @@
 #  updated_at              :datetime         not null
 #  patient_id              :bigint           not null
 #  performed_by_user_id    :bigint
-#  team_id                 :bigint           not null
+#  team_id                 :bigint
 #
 # Indexes
 #
@@ -39,7 +39,7 @@
 
   describe "associations" do
     it { should belong_to(:patient) }
-    it { should belong_to(:team) }
+    it { should belong_to(:team).optional(true) }
     it { should belong_to(:performed_by).optional(true) }
   end
 

spec/policies/triage_policy_spec.rb

@@ -60,11 +60,15 @@
       let(:team) { create(:team) }
       let(:user) { create(:user, team:) }
 
-      let(:team_batch) { create(:triage, :safe_to_vaccinate, team:) }
-      let(:non_team_batch) { create(:triage, :safe_to_vaccinate) }
-
-      it { should include(team_batch) }
-      it { should_not include(non_team_batch) }
+      let(:triage_for_team) { create(:triage, :safe_to_vaccinate, team:) }
+      let(:triage_for_different_team) do
+        create(:triage, :safe_to_vaccinate, team: create(:team))
+      end
+      let(:triage_for_all_teams) { create(:triage, :safe_to_vaccinate) }
+
+      it { should include(triage_for_team) }
+      it { should_not include(triage_for_different_team) }
+      it { should include(triage_for_all_teams) }
     end
   end
 end