From 5a0ee9342ee2d8505e0e8c08e26b8df67ce37941 Mon Sep 17 00:00:00 2001
From: Audrius Vaitonis <audrius.vaitonis@ukexportfinance.gov.uk>
Date: Thu, 25 Apr 2024 09:38:06 +0100
Subject: [PATCH] feat(DTFS2-7052): comments copy and variable name
 improvements

---
 src/helper-modules/ordnance-survey/ordnance-survey.service.ts | 1 -
 src/modules/geospatial/geospatial.service.ts                  | 2 +-
 test/support/generator/get-geospatial-addresses-generator.ts  | 4 ++--
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/helper-modules/ordnance-survey/ordnance-survey.service.ts b/src/helper-modules/ordnance-survey/ordnance-survey.service.ts
index f8ad1a78..8da5ee13 100644
--- a/src/helper-modules/ordnance-survey/ordnance-survey.service.ts
+++ b/src/helper-modules/ordnance-survey/ordnance-survey.service.ts
@@ -25,7 +25,6 @@ export class OrdnanceSurveyService {
       path,
       headers: { 'Content-Type': 'application/json' },
       onError: (error: Error) => {
-        console.error('Http call error happened, error %o', error);
         throw new OrdnanceSurveyException('Failed to get response from Ordnance Survey API.', error);
       },
     });
diff --git a/src/modules/geospatial/geospatial.service.ts b/src/modules/geospatial/geospatial.service.ts
index b566e7f0..cf2199a8 100644
--- a/src/modules/geospatial/geospatial.service.ts
+++ b/src/modules/geospatial/geospatial.service.ts
@@ -18,7 +18,7 @@ export class GeospatialService {
     }
 
     response.results.forEach((item) => {
-      // Item can have key DPA or LPI, get data dynamically, even if we expect key to always be DPA.
+      // Item can have key DPA or LPI, so we get data dynamically, even if we expect key to always be DPA.
       const item_data = item[Object.keys(item)[0]];
       addresses.push({
         organisationName: item_data.ORGANISATION_NAME || null,
diff --git a/test/support/generator/get-geospatial-addresses-generator.ts b/test/support/generator/get-geospatial-addresses-generator.ts
index 6a8c8e22..51522d9f 100644
--- a/test/support/generator/get-geospatial-addresses-generator.ts
+++ b/test/support/generator/get-geospatial-addresses-generator.ts
@@ -109,7 +109,7 @@ export class GetGeospatialAddressesGenerator extends AbstractGenerator<AddressVa
       ],
     }));
 
-    const getAddressesOrdnanceSurveyMultipleResponse: GetAddressesOrdnanceSurveyResponse = {
+    const getAddressesOrdnanceSurveyMultipleMatchingAddressesResponse: GetAddressesOrdnanceSurveyResponse = {
       header: {
         uri: 'test',
         query: 'test',
@@ -195,7 +195,7 @@ export class GetGeospatialAddressesGenerator extends AbstractGenerator<AddressVa
       getAddressByPostcodeMultipleResponse: getAddressesByPostcodeMultipleResponse,
       getAddressOrdnanceSurveyResponse: getAddressesOrdnanceSurveyResponse,
       getAddressOrdnanceSurveyEmptyResponse: getAddressesOrdnanceSurveyEmptyResponse,
-      getAddressOrdnanceSurveyMultipleResponse: getAddressesOrdnanceSurveyMultipleResponse,
+      getAddressOrdnanceSurveyMultipleResponse: getAddressesOrdnanceSurveyMultipleMatchingAddressesResponse,
       ordnanceSurveyAuthErrorResponse,
     };
   }