MAP-577 New options for Prisoner Relocation (#666)

* MAP-577 Add Prison Relocation options, bump node/npm versions

* MAP-577 Bump npm in circle pipeline

* MAP-577 Fix postgres version in circle pipeline

* MAP-577 Update assertion messge on invalid json

* MAP-577 Update node in Dockerfile

* MAP-577 Node 20

* package-lock.json

* MAP-577 Fix pipeline node version

.circleci/config.yml

@@ -14,13 +14,13 @@ parameters:
 
   node-version:
     type: string
-    default: 18.17.1-browsers
+    default: 20.11.0-browsers
 
 executors:
   integration-tests:
     docker:
       - image: cimg/node:<< pipeline.parameters.node-version >>
-      - image: circleci/postgres:10.13-ram
+      - image: cimg/postgres:14.7
         environment:
           POSTGRES_PASSWORD: use-of-force
           POSTGRES_USER: use-of-force
@@ -39,7 +39,7 @@ jobs:
       - checkout
       - run:
           name: Update npm
-          command: 'sudo npm install -g npm@9'
+          command: 'sudo npm install -g npm@10'
       - restore_cache:
           key: dependency-cache-{{ checksum "package-lock.json" }}
       - run:

Dockerfile

@@ -1,4 +1,4 @@
-FROM node:18.17.1 as builder
+FROM node:20.11.0 as builder
 
 ARG BUILD_NUMBER
 ARG GIT_REF
@@ -24,7 +24,7 @@ RUN CYPRESS_INSTALL_BINARY=0 npm ci --no-audit && \
 
 RUN npm prune --production
 
-FROM node:18.17.1-slim
+FROM node:20.11.0-slim
 LABEL maintainer="HMPPS Digital Studio <[email protected]>"
 
 # Cache breaking

README.md

@@ -16,7 +16,7 @@ The easiest way to run the app is to use docker compose to start local postgres
 `docker-compose pull`
 `docker-compose up`
 
-Install dependencies using `npm install`, ensuring you are using >= `Node v16`
+Install dependencies using `npm install`, ensuring you are using >= `Node v20`
 
 And then, to build the assets and start the app with nodemon:
 `npm run start:dev`

package.json

@@ -29,8 +29,8 @@
     "security_audit": "npx audit-ci --config audit-ci.json"
   },
   "engines": {
-    "node": "^18",
-    "npm": "<10"
+    "node": "^20",
+    "npm": "^10"
   },
   "jest": {
     "transform": {

server/config/types.ts

@@ -43,6 +43,9 @@ export const RelocationLocation = toEnum({
   SEGREGATION_UNIT: { value: 'SEGREGATION_UNIT', label: 'Segregation unit' },
   SPECIAL_ACCOMMODATION: { value: 'SPECIAL_ACCOMMODATION', label: 'Special accommodation' },
   CELLULAR_VEHICLE: { value: 'CELLULAR_VEHICLE', label: 'Cellular vehicle' },
+  RECEPTION: { value: 'RECEPTION', label: 'Reception' },
+  OTHER_WING: { value: 'OTHER_WING', label: 'Other wing' },
+  FACILITATE_RELEASE: { value: 'FACILITATE_RELEASE', label: 'To facilitate release' },
 })
 
 export const ReportStatus = toEnum({

server/routes/maintainingReports/admin.test.ts

@@ -142,7 +142,7 @@ describe('/:reportId/edit-report', () => {
           expect(reportService.update).not.toHaveBeenCalled()
           expect(flash).toHaveBeenCalledWith('errors', {
             href: '#form',
-            text: 'Unexpected token h in JSON at position 1',
+            text: "Unexpected token 'h', \"this isn't json!\" is not valid JSON",
           })
         })
     })