@defra-fish/gafl-webapp-service 1.58.0-rc.0 → 1.58.0-rc.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/gafl-webapp-service",
-  "version": "1.58.0-rc.0",
+  "version": "1.58.0-rc.2",
   "description": "The websales frontend for the GAFL service",
   "type": "module",
   "engines": {
@@ -36,8 +36,8 @@
     "prepare": "gulp --gulpfile build/gulpfile.cjs"
   },
   "dependencies": {
-    "@defra-fish/business-rules-lib": "1.58.0-rc.0",
-    "@defra-fish/connectors-lib": "1.58.0-rc.0",
+    "@defra-fish/business-rules-lib": "1.58.0-rc.2",
+    "@defra-fish/connectors-lib": "1.58.0-rc.2",
     "@defra/hapi-gapi": "^2.0.0",
     "@hapi/boom": "^9.1.2",
     "@hapi/catbox-redis": "^6.0.2",
@@ -80,5 +80,5 @@
       "./gafl-jest-matchers.js"
     ]
   },
-  "gitHead": "718b66d9c6be30a461d7649331dfc91b75d34e54"
+  "gitHead": "06d1e00b91644b9bfaf04b34c26fd45cba8ba198"
 }

package/src/pages/order-complete/order-complete/__tests__/order-complete.spec.js

@@ -5,7 +5,6 @@ import { COMPLETION_STATUS, FEEDBACK_URI_DEFAULT } from '../../../../constants.j
 import { displayStartTime } from '../../../../processors/date-and-time-display.js'
 import { LICENCE_TYPE } from '../../../../processors/mapping-constants.js'
 import { displayPrice } from '../../../../processors/price-display.js'
-import { validForRecurringPayment } from '../../../../processors/recurring-pay-helper.js'
 
 jest.mock('../../../../processors/recurring-pay-helper.js')
 jest.mock('../../../../processors/date-and-time-display.js')
@@ -50,14 +49,16 @@ const getSamplePermission = ({
   licenceType = LICENCE_TYPE['trout-and-coarse'],
   isLicenceForYou = true,
   licenceLength = '12M',
-  licensee = getSampleLicensee()
+  licensee = getSampleLicensee(),
+  isRecurringPayment = false
 } = {}) => ({
   startDate: '2019-12-14T00:00:00Z',
   licensee,
   isLicenceForYou,
   licenceType,
   referenceNumber,
-  licenceLength
+  licenceLength,
+  isRecurringPayment
 })
 
 const getSampleCompletionStatus = ({ agreed = true, posted = true, finalised = true, setUpPayment = true } = {}) => ({
@@ -203,27 +204,23 @@ describe('The order completion handler', () => {
     expect(displayStartTime).toHaveBeenCalledWith(request, permission)
   })
 
-  it('validForRecurringPayment is called with a permission', async () => {
-    const permission = getSamplePermission()
-
-    await getData(getSampleRequest({ permission }))
-
-    expect(validForRecurringPayment).toHaveBeenCalledWith(permission)
-  })
-
   it.each`
-    agreement | valid    | expected
-    ${true}   | ${true}  | ${true}
-    ${true}   | ${false} | ${false}
-    ${false}  | ${false} | ${false}
-    ${false}  | ${true}  | ${false}
+    show     | recurring | expected
+    ${true}  | ${true}   | ${true}
+    ${true}  | ${false}  | ${false}
+    ${false} | ${false}  | ${false}
+    ${false} | ${true}   | ${false}
   `(
-    'recurringPayment returns $expected when status for setup of recurring payment agreement is $agreement and valid for recurring payment is $valid',
-    async ({ agreement, valid, expected }) => {
-      validForRecurringPayment.mockReturnValueOnce(valid)
-      const completionStatus = getSampleCompletionStatus({ setUpPayment: agreement })
-      const { recurringPayment } = await getData(getSampleRequest({ completionStatus }))
+    'recurringPayment returns $expected when SHOW_RECURRING_PAYMENTS is $show and the permission recurring payment is $recurring',
+    async ({ show, recurring, expected }) => {
+      process.env.SHOW_RECURRING_PAYMENTS = show
+      const permission = getSamplePermission({ isRecurringPayment: recurring })
+      const request = getSampleRequest({ permission })
+      const { recurringPayment } = await getData(request)
+
       expect(recurringPayment).toBe(expected)
+
+      delete process.env.SHOW_RECURRING_PAYMENTS
     }
   )
 

package/src/pages/order-complete/order-complete/route.js

@@ -1,5 +1,4 @@
 import pageRoute from '../../../routes/page-route.js'
-import { validForRecurringPayment } from '../../../processors/recurring-pay-helper.js'
 import Boom from '@hapi/boom'
 import { COMPLETION_STATUS, FEEDBACK_URI_DEFAULT } from '../../../constants.js'
 import { ORDER_COMPLETE, NEW_TRANSACTION, LICENCE_DETAILS } from '../../../uri.js'
@@ -42,7 +41,7 @@ export const getData = async request => {
     digitalConfirmation: digital && permission.licensee.postalFulfilment,
     digitalLicence: digital && !permission.licensee.postalFulfilment,
     postalLicence: permission.licensee.postalFulfilment,
-    recurringPayment: isRecurringPayment(status, permission),
+    recurringPayment: isRecurringPayment(permission),
     uri: {
       feedback: process.env.FEEDBACK_URI || FEEDBACK_URI_DEFAULT,
       licenceDetails: addLanguageCodeToUri(request, LICENCE_DETAILS.uri),
@@ -59,7 +58,7 @@ const postalFulfilment = permission => {
   }
 }
 
-const isRecurringPayment = (status, permission) => validForRecurringPayment(permission) && status.permissions[0]['set-up-payment']
+const isRecurringPayment = permission => process.env.SHOW_RECURRING_PAYMENTS?.toLowerCase() === 'true' && permission.isRecurringPayment
 
 const digitalConfirmation = permission =>
   permission.licensee.preferredMethodOfConfirmation === HOW_CONTACTED.email ||