Chore: Remove PLF routing TODOs

These were added while the individual pages were implemented and
can now be removed

config/locales/cy/transaction_types.yml

@@ -2,18 +2,6 @@
 cy:
   transaction_types:
     names:
-      citizens:
-        benefits: stifeneB
-        child_care: stnemyap eracdlihC
-        excluded_benefits: stifeneb dedragersiD
-        friends_or_family: ylimaf ro sdneirf morf pleh laicnaniF
-        legal_aid: esac lanimirc a ni dia lagel sdrawot stnemyaP
-        maintenance_in: rentrap remrof a morf stnemyap ecnanetniaM
-        maintenance_out: rentrap remrof a ot stnemyap ecnanetniaM
-        pension: noisneP
-        property_or_lodger: regdol ro ytreporp a morf emocnI
-        rent_or_mortgage: stnemyap gnisuoH
-        salary: segaw ro yralaS
       providers:
         benefits: stifeneB
         child_care: stnemyap eracdlihC

config/locales/en/transaction_types.yml

@@ -1,21 +1,6 @@
 en:
   transaction_types:
     names:
-      # TODO: remove this as we don't actually show transaction categories to citizens anymore, however these categories are still
-      # being used in a few places, and have tests and helper methods which will also need to be updated/removed
-      citizens:
-        benefits: Benefits, charitable or government payments
-        child_care: Childcare payments
-        excluded_benefits: Disregarded benefits
-        friends_or_family: Financial help from friends or family
-        housing_benefit: Housing Benefit
-        legal_aid: Payments towards legal aid in a criminal case
-        maintenance_in: Maintenance payments from a former partner
-        maintenance_out: Maintenance payments to a former partner
-        pension: Pension
-        property_or_lodger: Income from a property or lodger
-        rent_or_mortgage: Housing payments
-        salary: Salary or wages
       providers:
         benefits: Benefits, charitable or government payments
         child_care: Childcare payments

spec/helpers/user_transactions_helper_spec.rb

@@ -16,8 +16,8 @@
   describe "#incomings_list" do
     subject(:helper_incomings_list) { helper.incomings_list(legal_aid_application.transaction_types.credits, locale_namespace:) }
 
-    context "when for a citizen" do
-      let(:locale_namespace) { "transaction_types.names.citizens" }
+    context "when for a provider" do
+      let(:locale_namespace) { "transaction_types.names.providers" }
 
       it "returns correct hash" do
         expect(helper_incomings_list[:items].first.to_h).to match hash_result
@@ -46,8 +46,8 @@
 
     let(:transaction_type) { create(:transaction_type, :maintenance_out) }
 
-    context "when for a citizen" do
-      let(:locale_namespace) { "transaction_types.names.citizens" }
+    context "when for a provider" do
+      let(:locale_namespace) { "transaction_types.names.providers" }
 
       it "returns correct hash" do
         expect(helper_payments_list[:items].first.to_h).to match hash_result

spec/requests/providers/application_merits_task/original_judge_levels_controller_spec.rb

@@ -79,9 +79,9 @@
               .to("district_judge")
         end
 
-        it "redirects to the next page", skip: "TODO: AP-5531/5532" do
+        it "redirects to the next page" do
           post_original_judge_level
-          expect(response).to redirect_to(:appeal_court_type)
+          expect(response).to redirect_to(flow_forward_path)
         end
       end
 

spec/requests/providers/application_merits_task/second_appeals_controller_spec.rb

@@ -70,9 +70,9 @@
           expect(legal_aid_application.reload.appeal.second_appeal).to be true
         end
 
-        it "redirects to the next page", skip: "TODO: AP-5530" do
+        it "redirects to the next page" do
           post_second_appeal
-          expect(response).to redirect_to(:second_appeal_court)
+          expect(response).to redirect_to(providers_legal_aid_application_second_appeal_court_type_path)
         end
       end
 
@@ -88,9 +88,9 @@
           expect(legal_aid_application.reload.appeal.second_appeal).to be false
         end
 
-        it "redirects to the next page", skip: "TODO: AP-5530" do
+        it "redirects to the next page" do
           post_second_appeal
-          expect(response).to redirect_to(:original_judge_level)
+          expect(response).to redirect_to(providers_legal_aid_application_original_judge_level_path)
         end
       end