Merge pull request #537 from companieshouse/IDVA5-1373-International-…

…Correspondence-Address Fixed country name and premise not being retained on error
companieshouse · Nov 21, 2024 · f8c9ba8 · f8c9ba8
2 parents 44eaee7 + 420b8e1
commit f8c9ba8
Show file tree

Hide file tree

Showing 7 changed files with 30 additions and 9 deletions.
diff --git a/locales/cy/address-look-up.json b/locales/cy/address-look-up.json
@@ -4,6 +4,7 @@
     "correspondenceLookUpAddressPostCodeInput": "Cod post y DU",
     "correspondenceLookUpAddressInputHint1": "Enw neu rif eiddo",
     "correspondenceLookUpAddressInputHint2": "Bydd angen i chi cofnodi'r cyfeiriad â llaw os nad yw yn y Deyrnas Unedig",
+    "businessLookUpAddressPostcodeHint": "Er enghraifft, 'Y Felin', '116' neu 'Fflat 37a'",
     "correspondenceLookUpAddressFindAddressBtn": "Dod o hyd i gyfeiriad",
     "correspondenceLookUpAddressManuallyBtn": "Cofnodwch y cyfeiriad â llaw",
     "correspondenceLookUpAddressListTitle": "Dewiswch y cyfeiriad gohebiaeth",

diff --git a/locales/en/address-look-up.json b/locales/en/address-look-up.json
@@ -4,6 +4,7 @@
     "correspondenceLookUpAddressPostCodeInput": "UK postcode",
     "correspondenceLookUpAddressInputHint1": "Property name or number",
     "correspondenceLookUpAddressInputHint2": "You'll need to enter the address manually if it's not in the UK",
+    "businessLookUpAddressPostcodeHint": "For example, The Mill', '116' or 'Flat 37a'",
     "correspondenceLookUpAddressFindAddressBtn": "Find address",
     "correspondenceLookUpAddressManuallyBtn": "Enter address manually",
     "correspondenceLookUpAddressListTitle": "Select the correspondence address",

diff --git a/src/services/correspondence-address/correspondence-address-manual.ts b/src/services/correspondence-address/correspondence-address-manual.ts
@@ -10,7 +10,7 @@ export class CorrespondenceAddressManualService {
             addressLine2: req.body.addressLine2,
             locality: req.body.addressTown,
             region: req.body.addressCounty,
-            country: req.body.addressCountry,
+            country: req.body.countryInput,
             postalCode: req.body.addressPostcode
         };
         const applicantDetails = acspData.applicantDetails || {};
@@ -20,12 +20,12 @@ export class CorrespondenceAddressManualService {
 
     public getCorrespondenceManualAddress (acspData: AcspData) {
         return {
-            propertyDetails: acspData?.applicantDetails?.correspondenceAddress?.premises,
+            addressPropertyDetails: acspData?.applicantDetails?.correspondenceAddress?.premises,
             addressLine1: acspData?.applicantDetails?.correspondenceAddress?.addressLine1,
             addressLine2: acspData?.applicantDetails?.correspondenceAddress?.addressLine2,
             addressTown: acspData?.applicantDetails?.correspondenceAddress?.locality,
             addressCounty: acspData?.applicantDetails?.correspondenceAddress?.region,
-            addressCountry:
+            countryInput:
               acspData?.applicantDetails?.correspondenceAddress?.country,
             addressPostcode:
               acspData?.applicantDetails?.correspondenceAddress?.postalCode

diff --git a/src/views/common/correspondence-address-manual/capture-correspondence-address-manual.njk b/src/views/common/correspondence-address-manual/capture-correspondence-address-manual.njk
@@ -31,7 +31,7 @@
         },
         id: "addressPropertyDetails",
         name: "addressPropertyDetails",
-        value: payload["propertyDetails"],
+        value: payload["addressPropertyDetails"],
         errorMessage: {
             text: errors["addressPropertyDetails"].text
         } if errors.addressPropertyDetails
@@ -85,6 +85,9 @@
     <div id="typeahead-form-group" class="govuk-form-group">
         <label for="countryInput" class="govuk-label">{{ i18n.addressCountry }}</label>
         {% set dropdownDefaultText = i18n.whereDoYouLiveDefaultDropdownText %}
+        {% if payload.countryInput | length %}
+            {% set dropdownValue = payload.countryInput %}
+        {% endif %}
         <p class="govuk-hint" id="typeahead-hint">{{ i18n.whereDoYouLiveHint }}</p>
         <div id="my-autocomplete-container" class="autocomplete-wrapper govuk-!-width-two-thirds"></div>
         {% set errorMessage = errors %}

diff --git a/src/views/features/unincorporated/business-address-auto-lookup/auto-lookup-address.njk b/src/views/features/unincorporated/business-address-auto-lookup/auto-lookup-address.njk
@@ -38,7 +38,7 @@
                 autocomplete: "street-address",
                 value: payload.premise,
                 hint: {
-                  text: i18n.correspondenceLookUpAddressInputHint2,
+                  text: i18n.businessLookUpAddressPostcodeHint,
                   classes: "govuk-hint"
                 }
             }) }}

diff --git a/src/views/partials/country-typeahead-input.njk b/src/views/partials/country-typeahead-input.njk
@@ -2,7 +2,7 @@
 <script type="text/javascript">
   const countryList = "{{countryList}}"
   const error = "{{errors.countryInput.text}}"
-  const countryDefaultValue = "{{countryInput}}"
+  const countryDefaultValue = "{{payload.countryInput}}"
 
   var autocompleteConfig = {
     element: document.querySelector('#my-autocomplete-container'),

diff --git a/test/src/services/correspondence-address/correspondence-address-manual.test.ts b/test/src/services/correspondence-address/correspondence-address-manual.test.ts
@@ -39,7 +39,7 @@ describe("CorrespondenceAddressManualService", () => {
             addressLine2: "Suite 300",
             addressTown: "Example",
             addressCounty: "Example",
-            addressCountry: "Example",
+            countryInput: "Example",
             addressPostcode: "EX1 1EX"
         };
 
@@ -70,12 +70,12 @@ describe("CorrespondenceAddressManualService", () => {
         const retrievedAddress = service.getCorrespondenceManualAddress(acspData);
 
         expect(retrievedAddress).toEqual({
-            propertyDetails: "Suite 100",
+            addressPropertyDetails: "Suite 100",
             addressLine1: "123 Test St",
             addressLine2: "Apt 4",
             addressTown: "Test",
             addressCounty: "Test",
-            addressCountry: "Test",
+            countryInput: "Test",
             addressPostcode: "TE5 5TL"
         });
     });
@@ -111,4 +111,20 @@ describe("CorrespondenceAddressManualService", () => {
             addressPostcode: undefined
         });
     });
+
+    test("getCorrespondenceManualAddress handles undefined acspData", () => {
+        const acspDataUndefined = undefined;
+
+        const retrievedAddress = service.getCorrespondenceManualAddress(acspDataUndefined!);
+
+        expect(retrievedAddress).toEqual({
+            propertyDetails: undefined,
+            addressLine1: undefined,
+            addressLine2: undefined,
+            addressTown: undefined,
+            addressCounty: undefined,
+            addressCountry: undefined,
+            addressPostcode: undefined
+        });
+    });
 });