BNGP-4970: LPA reference data should not store in db

packages/webapp/src/routes/__tests__/credits-purchase/development-project-information.spec.js

@@ -1,6 +1,9 @@
 import { submitGetRequest, submitPostRequest } from '../helpers/server.js'
 import constants from '../../../utils/credits-purchase-constants.js'
-import lojConstants from '../../../utils/loj-constants.js'
+import { getLpaNames } from '../../../utils/get-lpas.js'
+
+jest.mock('../../../utils/get-lpas.js')
+
 const url = constants.routes.CREDITS_PURCHASE_DEVELOPMENT_PROJECT_INFORMATION
 
 describe(url, () => {
@@ -23,14 +26,13 @@ describe(url, () => {
 
     redisMap = new Map()
     redisMap.set(constants.redisKeys.PLANNING_AUTHORTITY_LIST, ['Planning Authority 1', 'Planning Authority 2'])
-    redisMap.set(lojConstants.redisKeys.REF_LPA_NAMES, ['Northumberland LPA', 'Middlesbrough LPA', 'Planning Authority 1', 'Planning Authority 2', 'Planning Authority 3'])
 
     developmentProjectInformation = require('../../credits-purchase/development-project-information.js')
+
+    getLpaNames.mockReturnValue(['Northumberland LPA', 'Middlesbrough LPA', 'Planning Authority 1', 'Planning Authority 2', 'Planning Authority 3'])
   })
 
   describe('GET', () => {
-    jest.mock('../../../utils/get-lpas.js')
-
     it(`should render the ${url.substring(1)} view`, async () => {
       await submitGetRequest({ url })
     })

packages/webapp/src/routes/__tests__/developer/development-project-information.spec.js

@@ -1,6 +1,9 @@
 import { submitGetRequest, submitPostRequest } from '../helpers/server.js'
 import constants from '../../../utils/constants.js'
-import lojConstants from '../../../utils/loj-constants.js'
+import { getLpaNames } from '../../../utils/get-lpas.js'
+
+jest.mock('../../../utils/get-lpas.js')
+
 const url = constants.routes.DEVELOPER_DEVELOPMENT_PROJECT_INFORMATION
 
 describe(url, () => {
@@ -23,14 +26,13 @@ describe(url, () => {
 
     redisMap = new Map()
     redisMap.set(constants.redisKeys.PLANNING_AUTHORTITY_LIST, ['Planning Authority 1', 'Planning Authority 2'])
-    redisMap.set(lojConstants.redisKeys.REF_LPA_NAMES, ['Northumberland LPA', 'Middlesbrough LPA', 'Planning Authority 1', 'Planning Authority 2', 'Planning Authority 3'])
 
     developmentProjectInformation = require('../../developer/development-project-information.js')
+
+    getLpaNames.mockReturnValue(['Northumberland LPA', 'Middlesbrough LPA', 'Planning Authority 1', 'Planning Authority 2', 'Planning Authority 3'])
   })
 
   describe('GET', () => {
-    jest.mock('../../../utils/get-lpas.js')
-
     it(`should render the ${url.substring(1)} view`, async () => {
       await submitGetRequest({ url })
     })

packages/webapp/src/routes/__tests__/land/add-planning-authority.spec.js

@@ -1,5 +1,9 @@
 import { submitGetRequest, submitPostRequest } from '../helpers/server.js'
 import constants from '../../../utils/constants.js'
+import { getLpaNames } from '../../../utils/get-lpas.js'
+
+jest.mock('../../../utils/get-lpas.js')
+
 const url = constants.routes.ADD_PLANNING_AUTHORITY
 
 describe(url, () => {
@@ -22,14 +26,13 @@ describe(url, () => {
 
     redisMap = new Map()
     redisMap.set(constants.redisKeys.PLANNING_AUTHORTITY_LIST, ['Planning Authority 1', 'Planning Authority 2'])
-    redisMap.set(constants.redisKeys.REF_LPA_NAMES, ['Northumberland LPA', 'Middlesbrough LPA', 'Planning Authority 1', 'Planning Authority 2', 'Planning Authority 3'])
 
     addPlanningAuthority = require('../../land/add-planning-authority.js')
+
+    getLpaNames.mockReturnValue(['Northumberland LPA', 'Middlesbrough LPA', 'Planning Authority 1', 'Planning Authority 2', 'Planning Authority 3'])
   })
 
   describe('GET', () => {
-    jest.mock('../../../utils/get-lpas.js')
-
     it(`should render the ${url.substring(1)} view`, async () => {
       await submitGetRequest({ url })
     })

packages/webapp/src/routes/credits-purchase/development-project-information.js

@@ -1,5 +1,4 @@
 import creditsConstants from '../../utils/credits-purchase-constants.js'
-import constants from '../../utils/loj-constants.js'
 import { getLpaNames } from '../../utils/get-lpas.js'
 import {
   validateIdGetSchemaOptional, getValidReferrerUrl
@@ -10,8 +9,6 @@ const handlers = {
   get: (request, h) => {
     const lpaNames = getLpaNames(filePathAndName)
 
-    request.yar.set(constants.redisKeys.REF_LPA_NAMES, lpaNames)
-
     const selectedLpa = request.yar.get(creditsConstants.redisKeys.CREDITS_PURCHASE_PLANNING_AUTHORITY_LIST)
     const planningApplicationRef = request.yar.get(creditsConstants.redisKeys.CREDITS_PURCHASE_PLANNING_APPLICATION_REF)
     const developmentName = request.yar.get(creditsConstants.redisKeys.CREDITS_PURCHASE_DEVELOPMENT_NAME)
@@ -26,10 +23,10 @@ const handlers = {
   post: (request, h) => {
     const { localPlanningAuthority, planningApplicationRef, developmentName } = request.payload
 
-    const refLpaNames = request.yar.get(constants.redisKeys.REF_LPA_NAMES) ?? []
+    const lpaNames = getLpaNames(filePathAndName) ?? []
     const selectedLpa = Array.isArray(localPlanningAuthority) ? localPlanningAuthority[0] : localPlanningAuthority
 
-    const errors = lpaErrorHandler(selectedLpa, refLpaNames)
+    const errors = lpaErrorHandler(selectedLpa, lpaNames)
 
     if (!planningApplicationRef) {
       errors.planningApplicationRefError = {
@@ -54,7 +51,7 @@ const handlers = {
       return h.view(creditsConstants.views.CREDITS_PURCHASE_DEVELOPMENT_PROJECT_INFORMATION, {
         err,
         errors,
-        lpaNames: refLpaNames,
+        lpaNames,
         selectedLpa,
         planningApplicationRef,
         developmentName
@@ -84,10 +81,10 @@ export default [{
 /**
  * Handles Local Planning Authority errors
  * @param {string} selectedLpa
- * @param {Array<string>} refLpaNames
+ * @param {Array<string>} lpaNames
  * @returns {Object} Local planning authority errors object
  */
-const lpaErrorHandler = (selectedLpa, refLpaNames) => {
+const lpaErrorHandler = (selectedLpa, lpaNames) => {
   const errors = {}
 
   if (!selectedLpa) {
@@ -99,7 +96,7 @@ const lpaErrorHandler = (selectedLpa, refLpaNames) => {
     return errors
   }
 
-  if (refLpaNames.length > 0 && !refLpaNames.includes(selectedLpa)) {
+  if (lpaNames.length > 0 && !lpaNames.includes(selectedLpa)) {
     errors.invalidLocalPlanningAuthorityError = {
       text: 'Enter a valid local planning authority',
       href: '#invalidLocalPlanningAuthorityError'

packages/webapp/src/routes/developer/development-project-information.js

@@ -10,8 +10,6 @@ const handlers = {
   get: (request, h) => {
     const lpaNames = getLpaNames(filePathAndName)
 
-    request.yar.set(constants.redisKeys.REF_LPA_NAMES, lpaNames)
-
     const selectedLpa = request.yar.get(constants.redisKeys.DEVELOPER_PLANNING_AUTHORITY_LIST)
     const planningApplicationRef = request.yar.get(constants.redisKeys.DEVELOPER_PLANNING_APPLICATION_REF)
     const developmentName = request.yar.get(constants.redisKeys.DEVELOPER_DEVELOPMENT_NAME)
@@ -25,10 +23,10 @@ const handlers = {
   post: (request, h) => {
     const { localPlanningAuthority, planningApplicationRef, developmentName } = request.payload
 
-    const refLpaNames = request.yar.get(constants.redisKeys.REF_LPA_NAMES) ?? []
+    const lpaNames = getLpaNames(filePathAndName) ?? []
     const selectedLpa = Array.isArray(localPlanningAuthority) ? localPlanningAuthority[0] : localPlanningAuthority
 
-    const errors = lpaErrorHandler(selectedLpa, refLpaNames)
+    const errors = lpaErrorHandler(selectedLpa, lpaNames)
 
     if (!planningApplicationRef) {
       errors.planningApplicationRefError = {
@@ -53,7 +51,7 @@ const handlers = {
       return h.view(constants.views.DEVELOPER_DEVELOPMENT_PROJECT_INFORMATION, {
         err,
         errors,
-        lpaNames: refLpaNames,
+        lpaNames,
         selectedLpa,
         planningApplicationRef,
         developmentName
@@ -83,10 +81,10 @@ export default [{
 /**
  * Handles Local Planning Authority errors
  * @param {string} selectedLpa
- * @param {Array<string>} refLpaNames
+ * @param {Array<string>} lpaNames
  * @returns {Object} Local planning authority errors object
  */
-const lpaErrorHandler = (selectedLpa, refLpaNames) => {
+const lpaErrorHandler = (selectedLpa, lpaNames) => {
   const errors = {}
 
   if (!selectedLpa) {
@@ -98,7 +96,7 @@ const lpaErrorHandler = (selectedLpa, refLpaNames) => {
     return errors
   }
 
-  if (refLpaNames.length > 0 && !refLpaNames.includes(selectedLpa)) {
+  if (lpaNames.length > 0 && !lpaNames.includes(selectedLpa)) {
     errors.invalidLocalPlanningAuthorityError = {
       text: 'Enter a valid local planning authority',
       href: '#localPlanningAuthority'

packages/webapp/src/routes/land/add-planning-authority.js

@@ -12,7 +12,6 @@ const handlers = {
     const { id } = request.query
     const lpaNames = getLpaNames(filePathAndName)
 
-    request.yar.set(constants.redisKeys.REF_LPA_NAMES, lpaNames)
     const legalAgreementType = getLegalAgreementDocumentType(
       request.yar.get(constants.redisKeys.LEGAL_AGREEMENT_DOCUMENT_TYPE))?.toLowerCase()
     const lpaList = request.yar.get(constants.redisKeys.PLANNING_AUTHORTITY_LIST)
@@ -36,7 +35,7 @@ const handlers = {
     const selectedLpa = Array.isArray(localPlanningAuthority) ? localPlanningAuthority[0] : localPlanningAuthority
     const lpaList = request.yar.get(constants.redisKeys.PLANNING_AUTHORTITY_LIST) ?? []
     const errors = {}
-    const refLpaNames = request.yar.get(constants.redisKeys.REF_LPA_NAMES) ?? []
+    const lpaNames = getLpaNames(filePathAndName) ?? []
 
     if (!selectedLpa) {
       errors.emptyLocalPlanningAuthority = {
@@ -47,11 +46,11 @@ const handlers = {
         err: Object.values(errors),
         errors,
         legalAgreementType,
-        lpaNames: refLpaNames
+        lpaNames
       })
     }
 
-    if (refLpaNames.length > 0 && !refLpaNames.includes(selectedLpa)) {
+    if (lpaNames.length > 0 && !lpaNames.includes(selectedLpa)) {
       errors.invalidLocalPlanningAuthorityError = {
         text: 'Enter a valid local planning authority',
         href: '#invalidLocalPlanningAuthorityError'
@@ -60,7 +59,7 @@ const handlers = {
         err: Object.values(errors),
         errors,
         legalAgreementType,
-        lpaNames: refLpaNames
+        lpaNames
       })
     }
 
@@ -78,7 +77,7 @@ const handlers = {
         err: Object.values(duplicateError),
         errors,
         legalAgreementType,
-        lpaNames: refLpaNames
+        lpaNames
       })
     }
     if (id) {

packages/webapp/src/utils/loj-constants.js

@@ -150,7 +150,6 @@ const WRITTEN_AUTHORISATION_FILE_TYPE = 'written-authorisation-file-type'
 const WRITTEN_AUTHORISATION_CHECKED = 'written-authorisation-checked'
 const ALL_LPA = 'all-lpa'
 const PLANNING_AUTHORTITY_LIST = 'planning-authority-list'
-const REF_LPA_NAMES = 'ref-lpa-names'
 const LAND_OWNERSHIP_PROOFS = 'land-ownership-proofs'
 const ANY_OTHER_LANDOWNERS_CHECKED = 'la-any-other-landowners-checked'
 const LAND_APPLICATION_SUBMITTED = 'land-application-submitted'
@@ -271,7 +270,6 @@ export default {
     CHECK_PLANNING_AUTHORITIES,
     PLANNING_AUTHORTITY_LIST,
     ALL_LPA,
-    REF_LPA_NAMES,
     IS_ADDRESS_UK_KEY,
     UK_ADDRESS_KEY,
     NON_UK_ADDRESS_KEY,