Set QA threshold per policy

Previously we were QAing 10% of all claims when what we want to do is QA
10% of claims for each policy. This commit moves the MIN_QA_THRESHOLD
constant into the policy such that it's configurable on a per policy
basis.

app/models/claim.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 class Claim < ApplicationRecord
-  MIN_QA_THRESHOLD = 10
   NO_STUDENT_LOAN = "not_applicable"
   STUDENT_LOAN_PLAN_OPTIONS = StudentLoan::PLANS.dup << NO_STUDENT_LOAN
   ADDRESS_ATTRIBUTES = %w[address_line_1 address_line_2 address_line_3 address_line_4 postcode].freeze
@@ -240,12 +239,12 @@ def flaggable_for_qa?
   # Newly approved claims should not be flagged for QA for as long as the method
   # returns `false`; they should be flagged for QA otherwise.
   def below_min_qa_threshold?
-    return false if MIN_QA_THRESHOLD.zero?
+    return false if policy::MIN_QA_THRESHOLD.zero?
 
-    claims_approved_so_far = Claim.current_academic_year.approved.count
+    claims_approved_so_far = Claim.by_policy(policy).current_academic_year.approved.count
     return true if claims_approved_so_far.zero?
 
-    (Claim.current_academic_year.approved.qa_required.count.to_f / claims_approved_so_far) * 100 <= MIN_QA_THRESHOLD
+    (Claim.by_policy(policy).current_academic_year.approved.qa_required.count.to_f / claims_approved_so_far) * 100 <= policy::MIN_QA_THRESHOLD
   end
 
   def qa_completed?

app/models/policies/early_career_payments.rb

@@ -37,6 +37,9 @@ module EarlyCareerPayments
 
     SEARCHABLE_ELIGIBILITY_ATTRIBUTES = %w[teacher_reference_number].freeze
 
+    # Percentage of claims to QA
+    MIN_QA_THRESHOLD = 10
+
     def eligibility_page_url
       "https://www.gov.uk/guidance/early-career-payments-guidance-for-teachers-and-schools"
     end

app/models/policies/further_education_payments.rb

@@ -1,4 +1,6 @@
 module Policies
   module FurtherEducationPayments
+    # Percentage of claims to QA
+    MIN_QA_THRESHOLD = 10
   end
 end

app/models/policies/international_relocation_payments.rb

@@ -6,6 +6,9 @@ module InternationalRelocationPayments
     ELIGIBILITY_MATCHING_ATTRIBUTES = [["passport_number"]].freeze
     OTHER_CLAIMABLE_POLICIES = []
 
+    # Percentage of claims to QA
+    MIN_QA_THRESHOLD = 10
+
     # NOTE RL: currently IRP only has a single reply to address, so notify
     # doesn't show the address id
     def notify_reply_to_id

app/models/policies/levelling_up_premium_payments.rb

@@ -27,6 +27,9 @@ module LevellingUpPremiumPayments
     POLICY_START_YEAR = AcademicYear.new(2022).freeze
     POLICY_END_YEAR = AcademicYear.new(2024).freeze
 
+    # Percentage of claims to QA
+    MIN_QA_THRESHOLD = 10
+
     def notify_reply_to_id
       "03ece7eb-2a5b-461b-9c91-6630d0051aa6"
     end

app/models/policies/student_loans.rb

@@ -37,6 +37,9 @@ module StudentLoans
 
     SEARCHABLE_ELIGIBILITY_ATTRIBUTES = %w[teacher_reference_number].freeze
 
+    # Percentage of claims to QA
+    MIN_QA_THRESHOLD = 10
+
     def eligibility_page_url
       "https://www.gov.uk/guidance/teachers-claim-back-your-student-loan-repayments"
     end

spec/models/claim_spec.rb

@@ -371,15 +371,15 @@
 
       context "when above the min QA threshold" do
         before do
-          stub_const("Claim::MIN_QA_THRESHOLD", 0)
+          stub_const("Policies::#{claim.policy}::MIN_QA_THRESHOLD", 0)
         end
 
         it { is_expected.to eq(false) }
       end
 
       context "when below the min QA threshold" do
         before do
-          stub_const("Claim::MIN_QA_THRESHOLD", 100)
+          stub_const("Policies::#{claim.policy}::MIN_QA_THRESHOLD", 100)
         end
 
         it { is_expected.to eq(true) }
@@ -598,62 +598,208 @@
   end
 
   describe "#below_min_qa_threshold?" do
-    subject { described_class.new.below_min_qa_threshold? }
+    let(:policy) { Policies::EarlyCareerPayments }
+    let(:other_policy) { Policies::POLICIES.detect { |p| p != policy } }
+
+    subject { build(:claim, policy: policy).below_min_qa_threshold? }
 
     context "when the MIN_QA_THRESHOLD is set to zero" do
       before do
-        stub_const("Claim::MIN_QA_THRESHOLD", 0)
+        stub_const("Policies::#{policy}::MIN_QA_THRESHOLD", 0)
       end
 
       it { is_expected.to eq(false) }
     end
 
     context "when the MIN_QA_THRESHOLD is set to 10" do
       before do
-        stub_const("Claim::MIN_QA_THRESHOLD", 10) unless described_class::MIN_QA_THRESHOLD == 10
+        stub_const("Policies::#{policy}::MIN_QA_THRESHOLD", 10)
       end
 
       context "with no previously approved claims" do
+        let!(:claims_for_other_policy) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: other_policy,
+            academic_year: AcademicYear.current
+          )
+        end
         it { is_expected.to eq(true) }
       end
 
       context "with 1 previously approved claim (1 flagged for QA)" do
-        let!(:claims_flagged_for_qa) { create_list(:claim, 1, :approved, :flagged_for_qa, academic_year: AcademicYear.current) }
+        let!(:claims_flagged_for_qa) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
 
         it { is_expected.to eq(false) }
       end
 
       context "with 2 previously approved claims (1 flagged for QA)" do
-        let!(:claims_flagged_for_qa) { create_list(:claim, 1, :approved, :flagged_for_qa, academic_year: AcademicYear.current) }
-        let!(:claims_not_flagged_for_qa) { create_list(:claim, 1, :approved, academic_year: AcademicYear.current) }
+        let!(:claims_flagged_for_qa) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_not_flagged_for_qa) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
 
         it { is_expected.to eq(false) }
       end
 
       context "with 9 previously approved claims (1 flagged for QA)" do
-        let!(:claims_flagged_for_qa) { create_list(:claim, 1, :approved, :flagged_for_qa, academic_year: AcademicYear.current) }
-        let!(:claims_not_flagged_for_qa) { create_list(:claim, 8, :approved, academic_year: AcademicYear.current) }
+        let!(:claims_flagged_for_qa) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+
+        let!(:claims_not_flagged_for_qa) do
+          create_list(
+            :claim,
+            8,
+            :approved,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_for_other_policy) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: other_policy,
+            academic_year: AcademicYear.current
+          )
+        end
 
         it { is_expected.to eq(false) }
       end
 
       context "with 10 previously approved claims (1 flagged for QA)" do
-        let!(:claims_flagged_for_qa) { create_list(:claim, 1, :approved, :flagged_for_qa, academic_year: AcademicYear.current) }
-        let!(:claims_not_flagged_for_qa) { create_list(:claim, 9, :approved, academic_year: AcademicYear.current) }
+        let!(:claims_flagged_for_qa) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_not_flagged_for_qa) do
+          create_list(
+            :claim,
+            9,
+            :approved,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_for_other_policy) do
+          create_list(
+            :claim,
+            1,
+            :approved,
+            :flagged_for_qa,
+            policy: other_policy,
+            academic_year: AcademicYear.current
+          )
+        end
 
         it { is_expected.to eq(true) }
       end
 
       context "with 11 previously approved claims (2 flagged for QA)" do
-        let!(:claims_flagged_for_qa) { create_list(:claim, 2, :approved, :flagged_for_qa, academic_year: AcademicYear.current) }
-        let!(:claims_not_flagged_for_qa) { create_list(:claim, 10, :approved, academic_year: AcademicYear.current) }
+        let!(:claims_flagged_for_qa) do
+          create_list(
+            :claim,
+            2,
+            :approved,
+            :flagged_for_qa,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_not_flagged_for_qa) do
+          create_list(
+            :claim,
+            10,
+            :approved,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_for_other_policy) do
+          create_list(
+            :claim,
+            2,
+            :approved,
+            policy: other_policy,
+            academic_year: AcademicYear.current
+          )
+        end
 
         it { is_expected.to eq(false) }
       end
 
       context "with 21 previously approved claims (2 flagged for QA)" do
-        let!(:claims_flagged_for_qa) { create_list(:claim, 2, :approved, :flagged_for_qa, academic_year: AcademicYear.current) }
-        let!(:claims_not_flagged_for_qa) { create_list(:claim, 19, :approved, academic_year: AcademicYear.current) }
+        let!(:claims_flagged_for_qa) do
+          create_list(
+            :claim,
+            2,
+            :approved,
+            :flagged_for_qa,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_not_flagged_for_qa) do
+          create_list(
+            :claim,
+            19,
+            :approved,
+            policy: policy,
+            academic_year: AcademicYear.current
+          )
+        end
+        let!(:claims_for_other_policy) do
+          create_list(
+            :claim,
+            19,
+            :approved,
+            policy: other_policy,
+            academic_year: AcademicYear.current
+          )
+        end
 
         it { is_expected.to eq(true) }
       end

spec/support/stubbing_helpers.rb

@@ -1,6 +1,8 @@
 module StubbingHelpers
   def disable_claim_qa_flagging
-    stub_const("Claim::MIN_QA_THRESHOLD", 0)
+    Policies::POLICIES.each do |policy|
+      stub_const("Policies::#{policy}::MIN_QA_THRESHOLD", 0)
+    end
   end
 
   def stub_otp_verification(otp_code: "123456", valid: true)