Revert "Revert "Remove concept of submitted application""

This reverts commit 3a3b38b.

app/controllers/system_admin/applicants_controller.rb

@@ -9,7 +9,7 @@ class ApplicantsController < AdminController
     include Pagy::Backend
 
     def index
-      results = Application.submitted
+      results = Application.all
       .search(params[:search])
       .filter_by_status(params[:status])
       .order(created_at: :desc)

app/models/application.rb

@@ -30,10 +30,6 @@ class Application < ApplicationRecord
   delegate :sla_breached?, to: :application_progress
   delegate :status, to: :application_progress, allow_nil: false
 
-  default_scope { submitted }
-  scope :submitted, -> { where.not(urn: nil) }
-  scope :in_progress, -> { unscoped.where(urn: nil) }
-
   scope :search, lambda { |term|
                    return if term.blank?
 
@@ -64,22 +60,11 @@ def mark_as_qa!
                        unique_by: %i[application_id status])
   end
 
-  with_options if: :submitted? do
-    validates(:application_date, presence: true)
-    validates(:application_route, presence: true)
-    validates(:date_of_entry, presence: true)
-    validates(:start_date, presence: true)
-    validates(:subject, presence: true)
-    validates(:visa_type, presence: true)
-    validates(:urn, presence: true)
-    validates(:applicant, presence: true)
-  end
-
-  def assign_urn!
-    update!(urn: Urn.generate(application_route))
-  end
-
-  def submitted?
-    urn.present?
-  end
+  validates(:application_date, presence: true)
+  validates(:application_route, presence: true)
+  validates(:date_of_entry, presence: true)
+  validates(:start_date, presence: true)
+  validates(:subject, presence: true)
+  validates(:visa_type, presence: true)
+  validates(:applicant, presence: true)
 end

app/models/kpis.rb

@@ -4,7 +4,7 @@ def initialize
   end
 
   def total_applications
-    Application.submitted.count
+    Application.count
   end
 
   def total_rejections

app/queries/average_age_query.rb

@@ -1,6 +1,6 @@
 class AverageAgeQuery
   def initialize(relation = Applicant.all)
-    @relation = relation.joins(:application).merge(Application.submitted)
+    @relation = relation.joins(:application).merge(Application.all)
   end
 
   def call

app/queries/gender_breakdown_query.rb

@@ -1,6 +1,6 @@
 class GenderBreakdownQuery
   def initialize(relation = Applicant.all)
-    @relation = relation.joins(:application).merge(Application.submitted)
+    @relation = relation.joins(:application).merge(Application.all)
   end
 
   def call

app/queries/nationality_breakdown_query.rb

@@ -1,6 +1,6 @@
 class NationalityBreakdownQuery
   def initialize(relation = Applicant.all)
-    @relation = relation.joins(:application).merge(Application.submitted)
+    @relation = relation.joins(:application).merge(Application.all)
   end
 
   def call

app/queries/rejection_reason_breakdown_query.rb

@@ -1,6 +1,6 @@
 class RejectionReasonBreakdownQuery
   def initialize(relation = ApplicationProgress.all)
-    @relation = relation.joins(:application).merge(Application.submitted).where.not(rejection_reason: nil)
+    @relation = relation.joins(:application).merge(Application.all).where.not(rejection_reason: nil)
   end
 
   def call

app/queries/route_breakdown_query.rb

@@ -1,6 +1,6 @@
 class RouteBreakdownQuery
   def initialize(relation = Application.all)
-    @relation = relation.submitted
+    @relation = relation
   end
 
   def call

app/queries/subject_breakdown_query.rb

@@ -1,6 +1,6 @@
 class SubjectBreakdownQuery
   def initialize(relation = Application.all)
-    @relation = relation.submitted
+    @relation = relation
   end
 
   def call

app/queries/visa_breakdown_query.rb

@@ -1,6 +1,6 @@
 class VisaBreakdownQuery
   def initialize(relation = Application.all)
-    @relation = relation.submitted
+    @relation = relation
   end
 
   def call

spec/factories/applications.rb

@@ -31,7 +31,7 @@
     visa_type { VisaStep::VALID_ANSWERS_OPTIONS.reject { _1 == "Other" }.sample }
     date_of_entry { Time.zone.today }
     start_date { 1.month.from_now.to_date }
-    submitted
+    urn { Urn.generate(application_route) }
 
     factory :teacher_application do
       application_route { "teacher" }
@@ -45,14 +45,6 @@
       applicant strategy: :create, factory: :applicant
     end
 
-    trait :submitted do
-      urn { Urn.generate(application_route) }
-    end
-
-    trait :not_submitted do
-      urn { nil }
-    end
-
     trait :with_initial_checks_completed do
       application_progress strategy: :build, factory: %i[application_progress initial_checks_completed]
     end

spec/features/admin_console/applications_list_spec.rb

@@ -60,13 +60,13 @@
   def given_there_are_few_applications
     # Create 2 specific applications for search tests
     unique_applicant = create(:applicant, given_name: "Unique Given Name", middle_name: "Unique Middle Name", family_name: "Unique Family Name", email_address: "[email protected]")
-    create(:application, :submitted, applicant: unique_applicant, urn: "Unique Urn 1")
+    create(:application, applicant: unique_applicant, urn: "Unique Urn 1")
 
     another_applicant = create(:applicant, given_name: "Another Given Name", middle_name: "Another Middle Name", family_name: "Another Family Name", email_address: "[email protected]")
-    create(:application, :submitted, applicant: another_applicant, urn: "Unique Urn 2")
+    create(:application, applicant: another_applicant, urn: "Unique Urn 2")
 
     # Create 19 more applications for pagination test
-    create_list(:application, 19, :submitted)
+    create_list(:application, 19)
   end
 
   def given_there_is_an_application_that_breached_sla

spec/features/admin_console/download_qa_csv_file_spec.rb

@@ -36,7 +36,7 @@
   end
 
   def given_there_are_few_applications
-    create_list(:application, 5, :submitted)
+    create_list(:application, 5)
   end
 
   def when_i_am_in_the_applications_list_page

spec/features/admin_console/duplicates_search_spec.rb

@@ -10,10 +10,10 @@
   let!(:applicant_four) { build(:applicant, email_address: "[email protected]", passport_number: "123456", phone_number: "999999999") }
 
   before do
-    create(:application, :submitted, applicant: applicant_one)
-    create(:application, :submitted, applicant: applicant_two)
-    create(:application, :submitted, applicant: applicant_three)
-    create(:application, :submitted, applicant: applicant_four)
+    create(:application, applicant: applicant_one)
+    create(:application, applicant: applicant_two)
+    create(:application, applicant: applicant_three)
+    create(:application, applicant: applicant_four)
   end
 
   it "Admin can search for duplicates by email" do
@@ -34,20 +34,6 @@
     then_i_see_matching_duplicates_by_passport_number
   end
 
-  it "the view renders even where there are 'in progress' applications" do
-    given_i_am_signed_as_an_admin
-    when_there_are_in_progress_applications
-    when_i_visit_the_duplicates_page
-    then_i_see_the_in_progress_applications
-  end
-
-  it "shows duplicates only once even if they match multiple times" do
-    given_i_am_signed_as_an_admin
-    when_there_are_in_progress_applications
-    when_i_visit_the_duplicates_page
-    then_i_see_the_in_progress_applications_only_once
-  end
-
   def when_i_search_for_a_duplicate_by(type)
     visit duplicates_path
     case type
@@ -60,18 +46,10 @@ def when_i_search_for_a_duplicate_by(type)
     end
   end
 
-  def when_there_are_in_progress_applications
-    create(:application, applicant: nil, urn: nil, application_progress: nil)
-  end
-
   def when_i_visit_the_duplicates_page
     visit duplicates_path
   end
 
-  def then_i_see_the_in_progress_applications
-    expect(page).to have_content("Duplicated applications")
-  end
-
   def then_i_see_matching_duplicates
     expect(page).to have_content("[email protected]")
     expect(page).to have_content("123456")

spec/models/application_spec.rb

@@ -26,7 +26,7 @@
 RSpec.describe Application do
   describe "validations" do
     context "when the application has been submitted" do
-      subject(:application) { build(:teacher_application, :submitted) }
+      subject(:application) { build(:teacher_application) }
 
       it { expect(application).to validate_presence_of(:application_date) }
       it { expect(application).to validate_presence_of(:application_route) }
@@ -36,30 +36,9 @@
       it { expect(application).to validate_presence_of(:visa_type) }
       it { expect(application).to validate_presence_of(:applicant) }
     end
-
-    context "when the application has not been submitted" do
-      it "is valid" do
-        expect(described_class.new).to be_valid
-      end
-    end
   end
 
   describe "scopes" do
-    describe ".submitted" do
-      it "returns applications with a URN" do
-        create_list(:application, 2, :submitted)
-        create(:application, :not_submitted)
-
-        expect(described_class.submitted.count).to eq 2
-      end
-
-      it "does not return applications without a URN" do
-        create(:application, :not_submitted)
-
-        expect(described_class.submitted.count).to eq 0
-      end
-    end
-
     describe ".filter_by_status" do
       it "returns applications with the specified status" do
         initial_checks_application = build(:application)
@@ -95,22 +74,6 @@
     end
   end
 
-  describe "#assign_urn!" do
-    it "matches the required format for a teacher" do
-      application = create(:teacher_application)
-      application.assign_urn!
-
-      expect(application.reload.urn).to match(/^IRPTE[A-Z0-9]{5}$/)
-    end
-
-    it "matches the required format for a trainee" do
-      application = create(:salaried_trainee_application)
-      application.assign_urn!
-
-      expect(application.reload.urn).to match(/^IRPST[A-Z0-9]{5}$/)
-    end
-  end
-
   describe ".search" do
     let(:applicant_john) { create(:applicant, given_name: "John", family_name: "Doe", email_address: "[email protected]", passport_number: "123456") }
     let(:application_john) { create(:application, urn: "APP123", applicant: applicant_john) }

spec/models/kpis_spec.rb

@@ -3,7 +3,6 @@
 RSpec.describe Kpis do
   describe "#total_applications" do
     it "returns the total number of applications" do
-      create(:application, :not_submitted)
       create_list(:application, 5)
 
       kpis = described_class.new

spec/queries/average_age_query_spec.rb

@@ -4,8 +4,6 @@
   describe "#call" do
     context "when there are a few applicants" do
       before do
-        create(:application, :not_submitted, applicant: create(:applicant, date_of_birth: 25.years.ago))
-
         create(:application, applicant: create(:applicant, date_of_birth: 35.years.ago))
         create(:application, applicant: create(:applicant, date_of_birth: 45.years.ago))
         create(:application, applicant: create(:applicant, date_of_birth: 52.years.ago))

spec/queries/gender_breakdown_query_spec.rb

@@ -4,8 +4,6 @@
   describe "#call" do
     context "when there are a few applicants" do
       before do
-        create(:application, :not_submitted, applicant: create(:applicant, sex: "male"))
-
         create(:application, applicant: create(:applicant, sex: "male"))
         create(:application, applicant: create(:applicant, sex: "female"))
         create(:application, applicant: create(:applicant, sex: "male"))

spec/queries/nationality_breakdown_query_spec.rb

@@ -4,8 +4,6 @@
   describe "#call" do
     context "when there are a few applicants" do
       before do
-        create(:application, :not_submitted, applicant: create(:applicant, nationality: "Nationality 1"))
-
         create(:application, applicant: create(:applicant, nationality: "Nationality 1"))
         create(:application, applicant: create(:applicant, nationality: "Nationality 3"))
         create(:application, applicant: create(:applicant, nationality: "Nationality 4"))

spec/queries/rejection_reason_breakdown_query_spec.rb

@@ -8,7 +8,6 @@
         create(:application_progress, application: create(:application, applicant:), rejection_reason: "suspected_fraud")
         create(:application_progress, application: create(:application, applicant:), rejection_reason: "suspected_fraud")
         create(:application_progress, application: create(:application, applicant:), rejection_reason: "ineligible_school")
-        create(:application_progress, application: create(:application, :not_submitted, applicant:), rejection_reason: "duplicate_submission") # This one should not be included in the count because the associated application is not submitted
       end
 
       it "returns the correct rejection reason breakdown" do

spec/queries/route_breakdown_query_spec.rb

@@ -4,7 +4,6 @@
   describe "#call" do
     context "when there are a few applicants" do
       before do
-        create(:teacher_application, :not_submitted)
         create(:teacher_application)
         create(:salaried_trainee_application)
         create(:teacher_application)

spec/queries/subject_breakdown_query_spec.rb

@@ -4,7 +4,6 @@
   describe "#call" do
     context "when there are a few applicants" do
       before do
-        create(:application, :not_submitted)
         create(:application, subject: :physics)
         create(:application, subject: :languages)
         create(:application, subject: :general_science)

spec/queries/visa_breakdown_query_spec.rb

@@ -4,7 +4,6 @@
   describe "#call" do
     context "when there are a few applicants" do
       before do
-        create(:application, :not_submitted, visa_type: "visa_1")
         create(:application, visa_type: "visa_1")
         create_list(:application, 2, visa_type: "visa_2")
         create_list(:application, 4, visa_type: "visa_3")