Replace reference to signed_consent_document_required

This column has gone now and been replaced with the consent_method
column.

app/models/document.rb

@@ -68,7 +68,7 @@ def allow_multiple_uploads?
   end
 
   def optional?
-    (signed_consent? && !documentable.signed_consent_document_required) ||
+    (signed_consent? && !documentable.signed_consent_required?) ||
       (written_statement? && application_form.written_statement_optional)
   end
 

app/models/qualification_request.rb

@@ -50,7 +50,11 @@ class QualificationRequest < ApplicationRecord
        },
        _prefix: true
 
-  scope :consent_required, -> { where(signed_consent_document_required: true) }
+  scope :signed_consent_required,
+        -> do
+          consent_method_signed_ecctis.or(consent_method_signed_institution)
+        end
+
   scope :consent_requested, -> { where.not(consent_requested_at: nil) }
   scope :consent_received, -> { where.not(consent_received_at: nil) }
   scope :consent_respondable,
@@ -70,6 +74,10 @@ def expires_after
     6.weeks
   end
 
+  def signed_consent_required?
+    consent_method_signed_custom? || consent_method_signed_ecctis?
+  end
+
   def consent_requested!
     update!(consent_requested_at: Time.zone.now)
   end

app/view_objects/teacher_interface/application_form_view_object.rb

@@ -156,7 +156,7 @@ def qualification_consent_submitted?
     return false if assessment.nil?
 
     required_qualification_requests =
-      qualification_requests.where(signed_consent_document_required: true)
+      qualification_requests.signed_consent_required
 
     return false if required_qualification_requests.empty?
 

spec/factories/qualification_requests.rb

@@ -40,7 +40,7 @@
     association :qualification, :completed
 
     trait :consent_required do
-      signed_consent_document_required { true }
+      consent_method { %i[signed_ecctis signed_institution].sample }
 
       after(:create) do |qualification_request, _evaluator|
         create(