Merge branch 'main' into CLDC-477/reason-for-leaving-because-of-the-benefit-cap

Gemfile.lock

@@ -1,6 +1,6 @@
 GIT
   remote: https://github.com/rspec/rspec-core.git
-  revision: 053fcfeb6b0b6627edf7261737553a6f7df8cc14
+  revision: d57c371ee92b16211b80ac7b0b025968438f5297
   branch: main
   specs:
     rspec-core (3.11.0.pre)
@@ -198,7 +198,7 @@ GEM
     public_suffix (4.0.6)
     puma (5.5.2)
       nio4r (~> 2.0)
-    racc (1.5.2)
+    racc (1.6.0)
     rack (2.2.3)
     rack-mini-profiler (2.3.3)
       rack (>= 1.2.0)
@@ -279,7 +279,7 @@ GEM
       sass (~> 3.5, >= 3.5.5)
     scss_lint-govuk (0.2.0)
       scss_lint
-    selenium-webdriver (4.0.0)
+    selenium-webdriver (4.0.3)
       childprocess (>= 0.5, < 5.0)
       rexml (~> 3.2, >= 3.2.5)
       rubyzip (>= 1.2.2)
@@ -300,7 +300,7 @@ GEM
     stimulus-rails (0.7.1)
       rails (>= 6.0.0)
     thor (1.1.0)
-    turbo-rails (7.1.0)
+    turbo-rails (7.1.1)
       rails (>= 6.0.0)
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
@@ -323,7 +323,7 @@ GEM
     websocket-extensions (0.1.5)
     xpath (3.2.0)
       nokogiri (~> 1.8)
-    zeitwerk (2.4.2)
+    zeitwerk (2.5.1)
 
 PLATFORMS
   x86_64-darwin-19

app/controllers/case_logs_controller.rb

@@ -15,7 +15,7 @@ class CaseLogsController < ApplicationController
         if case_log.persisted?
           render json: case_log, status: :created
         else
-          render json: { errors: case_log.errors.full_messages }, status: :unprocessable_entity
+          render json: { errors: case_log.errors.messages }, status: :unprocessable_entity
         end
       end
     end
@@ -26,7 +26,7 @@ class CaseLogsController < ApplicationController
       if case_log.update(api_case_log_params)
         render json: case_log, status: :ok
       else
-        render json: { errors: case_log.errors.full_messages }, status: :unprocessable_entity
+        render json: { errors: case_log.errors.messages }, status: :unprocessable_entity
       end
     else
       render json: { error: "Case Log #{params[:id]} not found" }, status: :not_found
@@ -74,7 +74,7 @@ class CaseLogsController < ApplicationController
       if case_log.discard
         head :no_content
       else
-        render json: { errors: case_log.errors.full_messages }, status: :unprocessable_entity
+        render json: { errors: case_log.errors.messages }, status: :unprocessable_entity
       end
     else
       render json: { error: "Case Log #{params[:id]} not found" }, status: :not_found

app/models/case_log.rb

@@ -5,26 +5,26 @@ class CaseLogValidator < ActiveModel::Validator
 
   def validate_tenant_age(record)
     if record.tenant_age && !/^[1-9][0-9]?$|^120$/.match?(record.tenant_age.to_s)
-      record.errors.add :tenant_age, "must be between 0 and 120"
+      record.errors.add :tenant_age, "Tenant age must be between 0 and 120"
     end
   end
 
   def validate_property_number_of_times_relet(record)
     if record.property_number_of_times_relet && !/^[1-9]$|^0[1-9]$|^1[0-9]$|^20$/.match?(record.property_number_of_times_relet.to_s)
-      record.errors.add :property_number_of_times_relet, "must be between 0 and 20"
+      record.errors.add :property_number_of_times_relet, "Must be between 0 and 20"
     end
   end
 
   def validate_reasonable_preference(record)
     if record.homelessness == "No" && record.reasonable_preference == "Yes"
-      record.errors.add :reasonable_preference, "can not be Yes if Not Homeless immediately prior to this letting has been selected"
+      record.errors.add :reasonable_preference, "Can not be Yes if Not Homeless immediately prior to this letting has been selected"
     elsif record.reasonable_preference == "Yes"
       if !record.reasonable_preference_reason_homeless && !record.reasonable_preference_reason_unsatisfactory_housing && !record.reasonable_preference_reason_medical_grounds && !record.reasonable_preference_reason_avoid_hardship && !record.reasonable_preference_reason_do_not_know
-        record.errors.add :reasonable_preference_reason, "- if reasonable preference is Yes, a reason must be given"
+        record.errors.add :reasonable_preference_reason, "If reasonable preference is Yes, a reason must be given"
       end
     elsif record.reasonable_preference == "No"
       if record.reasonable_preference_reason_homeless || record.reasonable_preference_reason_unsatisfactory_housing || record.reasonable_preference_reason_medical_grounds || record.reasonable_preference_reason_avoid_hardship || record.reasonable_preference_reason_do_not_know
-        record.errors.add :reasonable_preference_reason, "- if reasonable preference is No, no reasons should be given"
+        record.errors.add :reasonable_preference_reason, "If reasonable preference is No, no reasons should be given"
       end
     end
   end
@@ -45,6 +45,16 @@ class CaseLogValidator < ActiveModel::Validator
     end
   end
 
+  def validate_armed_forces_injured(record)
+    if (record.armed_forces == "Yes - a regular" || record.armed_forces == "Yes - a reserve") && record.armed_forces_injured.blank?
+      record.errors.add :armed_forces_injured, "You must answer the armed forces injury question if the tenant has served in the armed forces"
+    end
+
+    if (record.armed_forces == "No" || record.armed_forces == "Prefer not to say") && record.armed_forces_injured.present?
+      record.errors.add :armed_forces_injured, "You must not answer the armed forces injury question if the tenant has not served in the armed forces or prefer not to say was chosen"
+    end
+  end
+
   def validate(record)
     # If we've come from the form UI we only want to validate the specific fields
     # that have just been submitted. If we're submitting a log via API or Bulk Upload

docs/api/DLUHC-CORE-Data.v1.json

@@ -108,7 +108,8 @@
                 "examples": {
                   "Invalid Age": {
                     "value": {
-                      "errors": [
+                      "errors": {
+                        "tenant_age": [
                           "Tenant age must be between 0 and 120"
                         ]
                       }
@@ -117,6 +118,7 @@
                 }
               }
             }
+          }
         },
         "description": "Update the information of an existing case log",
         "requestBody": {
@@ -211,9 +213,13 @@
                   "properties": {}
                 },
                 "examples": {
-                  "Invalid Age": {
+                  "Invalid Age & Reasonable Preference": {
                     "value": {
-                      "errors": [
+                      "errors": {
+                        "reasonable_preference_reason": [
+                          "If reasonable preference is Yes, a reason must be given"
+                        ],
+                        "tenant_age": [
                           "Tenant age must be between 0 and 120"
                         ]
                       }
@@ -222,6 +228,7 @@
                 }
               }
             }
+          }
         },
         "requestBody": {
           "content": {

spec/helpers/check_answers_helper_spec.rb

@@ -11,7 +11,7 @@ RSpec.describe CheckAnswersHelper do
     )
   end
   let(:case_log_with_met_radio_condition) do
-    FactoryBot.create(:case_log, armed_forces: "Yes - a regular")
+    FactoryBot.create(:case_log, armed_forces: "Yes - a regular", armed_forces_injured: "No")
   end
   let(:subsection) { "income_and_benefits" }
   let(:subsection_with_numeric_conditionals) { "household_characteristics" }

spec/models/case_log_spec.rb

@@ -26,7 +26,7 @@ RSpec.describe Form, type: :model do
       expect { CaseLog.create!(property_number_of_times_relet: 0) }.to raise_error(ActiveRecord::RecordInvalid)
     end
 
-    describe "reasonable preference validation" do
+    context "reasonable preference validation" do
       it "if given reasonable preference is yes a reason must be selected" do
         expect {
           CaseLog.create!(reasonable_preference: "Yes",
@@ -80,6 +80,22 @@ RSpec.describe Form, type: :model do
         }.to raise_error(ActiveRecord::RecordInvalid)
       end
     end
+
+    context "armed forces injured validation" do
+      it "must be anwered if tenant was a regular or reserve in armed forces" do
+        expect {
+          CaseLog.create!(armed_forces: "Yes - a regular",
+                          armed_forces_injured: nil)
+        }.to raise_error(ActiveRecord::RecordInvalid)
+      end
+
+      it "must be anwered if tenant was not a regular or reserve in armed forces" do
+        expect {
+          CaseLog.create!(armed_forces: "No",
+                          armed_forces_injured: "Yes")
+        }.to raise_error(ActiveRecord::RecordInvalid)
+      end
+    end
   end
 
   describe "status" do

spec/requests/case_log_controller_spec.rb

@@ -66,7 +66,7 @@ RSpec.describe CaseLogsController, type: :request do
       it "validates case log parameters" do
         json_response = JSON.parse(response.body)
         expect(response).to have_http_status(:unprocessable_entity)
-        expect(json_response["errors"]).to match_array(["Tenant age must be between 0 and 120", "Property number of times relet must be between 0 and 20"])
+        expect(json_response["errors"]).to match_array([["property_number_of_times_relet", ["Must be between 0 and 20"]], ["tenant_age", ["Tenant age must be between 0 and 120"]]])
       end
     end
 
@@ -157,7 +157,7 @@ RSpec.describe CaseLogsController, type: :request do
 
       it "returns an error message" do
         json_response = JSON.parse(response.body)
-        expect(json_response["errors"]).to eq(["Tenant age must be between 0 and 120"])
+        expect(json_response["errors"]).to eq({ "tenant_age" => ["Tenant age must be between 0 and 120"] })
       end
     end