@defra-fish/sales-api-service 1.61.0-rc.0 → 1.61.0-rc.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/sales-api-service",
-  "version": "1.61.0-rc.0",
+  "version": "1.61.0-rc.10",
   "description": "Rod Licensing Sales API",
   "type": "module",
   "engines": {
@@ -35,9 +35,9 @@
     "test": "echo \"Error: run tests from root\" && exit 1"
   },
   "dependencies": {
-    "@defra-fish/business-rules-lib": "1.61.0-rc.0",
-    "@defra-fish/connectors-lib": "1.61.0-rc.0",
-    "@defra-fish/dynamics-lib": "1.61.0-rc.0",
+    "@defra-fish/business-rules-lib": "1.61.0-rc.10",
+    "@defra-fish/connectors-lib": "1.61.0-rc.10",
+    "@defra-fish/dynamics-lib": "1.61.0-rc.10",
     "@hapi/boom": "^9.1.2",
     "@hapi/hapi": "^20.1.3",
     "@hapi/inert": "^6.0.3",
@@ -52,5 +52,5 @@
     "moment-timezone": "^0.5.34",
     "uuid": "^8.3.2"
   },
-  "gitHead": "67185f8a721c75f38052b63bdb92a5549622367c"
+  "gitHead": "27650c5d19c86f1621c7c1367fafb7d090da3d02"
 }

package/src/schema/__tests__/recurring-payments.schema.spec.js

@@ -1,7 +1,8 @@
 import {
   dueRecurringPaymentsRequestParamsSchema,
   dueRecurringPaymentsResponseSchema,
-  processRPResultRequestParamsSchema
+  processRPResultRequestParamsSchema,
+  linkRecurringPaymentsRequestParamsSchema
 } from '../recurring-payments.schema.js'
 
 jest.mock('../validators/validators.js', () => ({
@@ -62,6 +63,11 @@ const getProcessRPResultSampleData = () => ({
   createdDate: '2025-01-01T00:00:00.000Z'
 })
 
+const getLinkRecurringPaymentsSampleData = () => ({
+  existingRecurringPaymentId: 'ghi123',
+  agreementId: 'jkl456'
+})
+
 describe('getDueRecurringPaymentsSchema', () => {
   it('validates expected object', async () => {
     expect(() => dueRecurringPaymentsResponseSchema.validateAsync(getResponseSampleData())).not.toThrow()
@@ -149,3 +155,24 @@ describe('processRPResultRequestParamsSchema', () => {
     expect(() => processRPResultRequestParamsSchema.validateAsync(sampleData).rejects.toThrow())
   })
 })
+
+describe('linkRecurringPaymentsRequestParamsSchema', () => {
+  it('validates expected object', async () => {
+    expect(() => linkRecurringPaymentsRequestParamsSchema.validateAsync(getLinkRecurringPaymentsSampleData())).not.toThrow()
+  })
+
+  it.each([['existingRecurringPaymentId'], ['agreementId']])('throws an error if %s is missing', async property => {
+    const sampleData = getLinkRecurringPaymentsSampleData()
+    delete sampleData[property]
+    expect(() => linkRecurringPaymentsRequestParamsSchema.validateAsync(sampleData).rejects.toThrow())
+  })
+
+  it.each([
+    ['existingRecurringPaymentId', 99],
+    ['agreementId', 99]
+  ])('throws an error if %s is not the correct type', async (property, value) => {
+    const sampleData = getLinkRecurringPaymentsSampleData()
+    sampleData[property] = value
+    expect(() => linkRecurringPaymentsRequestParamsSchema.validateAsync(sampleData).rejects.toThrow())
+  })
+})

package/src/schema/recurring-payments.schema.js

@@ -29,3 +29,8 @@ export const processRPResultRequestParamsSchema = Joi.object({
   paymentId: Joi.string().required(),
   createdDate: Joi.string().isoDate().required()
 })
+
+export const linkRecurringPaymentsRequestParamsSchema = Joi.object({
+  existingRecurringPaymentId: Joi.string().required(),
+  agreementId: Joi.string().required()
+})

package/src/server/routes/__tests__/recurring-payments.spec.js

@@ -1,6 +1,10 @@
 import recurringPayments from '../recurring-payments.js'
-import { getRecurringPayments, processRPResult } from '../../../services/recurring-payments.service.js'
-import { dueRecurringPaymentsRequestParamsSchema, processRPResultRequestParamsSchema } from '../../../schema/recurring-payments.schema.js'
+import { getRecurringPayments, processRPResult, linkRecurringPayments } from '../../../services/recurring-payments.service.js'
+import {
+  dueRecurringPaymentsRequestParamsSchema,
+  processRPResultRequestParamsSchema,
+  linkRecurringPaymentsRequestParamsSchema
+} from '../../../schema/recurring-payments.schema.js'
 
 const [
   {
@@ -8,26 +12,33 @@ const [
   },
   {
     options: { handler: prpHandler }
+  },
+  {
+    options: { handler: lrpHandler }
   }
 ] = recurringPayments
 
 jest.mock('../../../services/recurring-payments.service.js', () => ({
   getRecurringPayments: jest.fn(),
-  processRPResult: jest.fn()
+  processRPResult: jest.fn(),
+  linkRecurringPayments: jest.fn()
 }))
 
 jest.mock('../../../schema/recurring-payments.schema.js', () => ({
   dueRecurringPaymentsRequestParamsSchema: jest.fn(),
-  processRPResultRequestParamsSchema: jest.fn()
+  processRPResultRequestParamsSchema: jest.fn(),
+  linkRecurringPaymentsRequestParamsSchema: jest.fn()
 }))
 
 const getMockRequest = ({
   date = '2023-10-19',
   transactionId = 'transaction-id',
   paymentId = 'payment-id',
-  createdDate = 'created-date'
+  createdDate = 'created-date',
+  existingRecurringPaymentId = 'existing-recurring-payment-id',
+  agreementId = 'agreement-id'
 }) => ({
-  params: { date, transactionId, paymentId, createdDate }
+  params: { date, transactionId, paymentId, createdDate, existingRecurringPaymentId, agreementId }
 })
 
 const getMockResponseToolkit = () => ({
@@ -75,11 +86,37 @@ describe('recurring payments', () => {
       expect(processRPResult).toHaveBeenCalledWith(transactionId, paymentId, createdDate)
     })
 
-    it('should validate with dueRecurringPaymentsRequestParamsSchema', async () => {
-      const date = Symbol('date')
-      const request = getMockRequest({ date })
-      await drpHandler(request, getMockResponseToolkit())
+    it('should validate with processRPResultRequestParamsSchema', async () => {
+      const transactionId = Symbol('transaction-id')
+      const paymentId = Symbol('payment-id')
+      const createdDate = Symbol('created-date')
+      const request = getMockRequest({ transactionId, paymentId, createdDate })
+      await prpHandler(request, getMockResponseToolkit())
       expect(recurringPayments[1].options.validate.params).toBe(processRPResultRequestParamsSchema)
     })
   })
+
+  describe('linkRecurringPayments', () => {
+    it('handler should return continue response', async () => {
+      const request = getMockRequest({})
+      const responseToolkit = getMockResponseToolkit()
+      expect(await lrpHandler(request, responseToolkit)).toEqual(responseToolkit.continue)
+    })
+
+    it('should call linkRecurringPayments with existingRecurringPaymentId and agreementId', async () => {
+      const existingRecurringPaymentId = Symbol('existing-recurring-payment')
+      const agreementId = Symbol('agreement-id')
+      const request = getMockRequest({ existingRecurringPaymentId, agreementId })
+      await lrpHandler(request, getMockResponseToolkit())
+      expect(linkRecurringPayments).toHaveBeenCalledWith(existingRecurringPaymentId, agreementId)
+    })
+
+    it('should validate with linkRecurringPaymentsRequestParamsSchema', async () => {
+      const existingRecurringPaymentId = Symbol('existing-recurring-payment')
+      const agreementId = Symbol('agreement-id')
+      const request = getMockRequest({ existingRecurringPaymentId, agreementId })
+      await lrpHandler(request, getMockResponseToolkit())
+      expect(recurringPayments[2].options.validate.params).toBe(linkRecurringPaymentsRequestParamsSchema)
+    })
+  })
 })

package/src/server/routes/recurring-payments.js

@@ -1,9 +1,12 @@
 import {
   dueRecurringPaymentsRequestParamsSchema,
   dueRecurringPaymentsResponseSchema,
+  linkRecurringPaymentsRequestParamsSchema,
   processRPResultRequestParamsSchema
 } from '../../schema/recurring-payments.schema.js'
-import { getRecurringPayments, processRPResult } from '../../services/recurring-payments.service.js'
+import { getRecurringPayments, linkRecurringPayments, processRPResult } from '../../services/recurring-payments.service.js'
+
+const SWAGGER_TAGS = ['api', 'recurring-payments']
 
 export default [
   {
@@ -16,7 +19,7 @@ export default [
         return h.response(result)
       },
       description: 'Retrieve recurring payments due for the specified date',
-      tags: ['api', 'recurring-payments'],
+      tags: SWAGGER_TAGS,
       validate: {
         params: dueRecurringPaymentsRequestParamsSchema
       },
@@ -40,7 +43,7 @@ export default [
         return h.response(result)
       },
       description: 'Generate a permission from a recurring payment record',
-      tags: ['api', 'recurring-payments'],
+      tags: SWAGGER_TAGS,
       validate: {
         params: processRPResultRequestParamsSchema
       },
@@ -53,5 +56,29 @@ export default [
         }
       }
     }
+  },
+  {
+    method: 'GET',
+    path: '/linkRecurringPayments/{existingRecurringPaymentId}/{agreementId}',
+    options: {
+      handler: async (request, h) => {
+        const { existingRecurringPaymentId, agreementId } = request.params
+        const result = await linkRecurringPayments(existingRecurringPaymentId, agreementId)
+        return h.response(result)
+      },
+      description: 'Link an old RecurringPayment to its replacement',
+      tags: SWAGGER_TAGS,
+      validate: {
+        params: linkRecurringPaymentsRequestParamsSchema
+      },
+      plugins: {
+        'hapi-swagger': {
+          responses: {
+            200: { description: 'Old RecurringPayment linked to new RecurringPayment successfully' }
+          },
+          order: 3
+        }
+      }
+    }
   }
 ]

package/src/services/__tests__/recurring-payments.service.spec.js

@@ -1,9 +1,10 @@
-import { findDueRecurringPayments, Permission } from '@defra-fish/dynamics-lib'
+import { executeQuery, findDueRecurringPayments, findRecurringPaymentsByAgreementId, Permission } from '@defra-fish/dynamics-lib'
 import {
   getRecurringPayments,
   processRecurringPayment,
   generateRecurringPaymentRecord,
-  processRPResult
+  processRPResult,
+  linkRecurringPayments
 } from '../recurring-payments.service.js'
 import { calculateEndDate, generatePermissionNumber } from '../permissions.service.js'
 import { getObfuscatedDob } from '../contacts.service.js'
@@ -20,7 +21,8 @@ jest.mock('@defra-fish/dynamics-lib', () => ({
   ...jest.requireActual('@defra-fish/dynamics-lib'),
   executeQuery: jest.fn(),
   findById: jest.fn(),
-  findDueRecurringPayments: jest.fn()
+  findDueRecurringPayments: jest.fn(),
+  findRecurringPaymentsByAgreementId: jest.fn(() => ['foo'])
 }))
 
 jest.mock('@defra-fish/connectors-lib', () => {
@@ -77,7 +79,7 @@ jest.mock('@defra-fish/business-rules-lib', () => ({
 
 const dynamicsLib = jest.requireMock('@defra-fish/dynamics-lib')
 
-const getMockRecurringPayment = () => ({
+const getMockRecurringPayment = (overrides = {}) => ({
   name: 'Test Name',
   nextDueDate: '2019-12-14T00:00:00Z',
   cancelledDate: null,
@@ -93,7 +95,8 @@ const getMockRecurringPayment = () => ({
     activePermission: {
       entity: getMockPermission()
     }
-  }
+  },
+  ...overrides
 })
 
 const getMockRPContactPermission = (contact, permission) => ({
@@ -619,4 +622,43 @@ describe('recurring payments service', () => {
       })
     })
   })
+
+  describe('linkRecurringPayments', () => {
+    it('should call executeQuery with findRecurringPaymentsByAgreementId and the provided agreementId', async () => {
+      const agreementId = 'abc123'
+      await linkRecurringPayments('existing-recurring-payment-id', agreementId)
+      expect(executeQuery).toHaveBeenCalledWith(findRecurringPaymentsByAgreementId(agreementId))
+    })
+
+    it('should log a RecurringPayment record when there is one match', async () => {
+      const consoleLogSpy = jest.spyOn(console, 'log').mockImplementation(jest.fn())
+      const onlyRecord = { entity: getMockRecurringPayment() }
+      dynamicsLib.executeQuery.mockReturnValueOnce([onlyRecord])
+
+      await linkRecurringPayments('existing-recurring-payment-id', 'agreement-id')
+
+      expect(consoleLogSpy).toHaveBeenCalledWith('newRecurringPaymentId: ', onlyRecord.id)
+    })
+
+    it('should log the RecurringPayment record with the latest endDate when there are multiple matches', async () => {
+      const consoleLogSpy = jest.spyOn(console, 'log').mockImplementation(jest.fn())
+      const oldestRecord = { entity: getMockRecurringPayment({ id: Symbol('oldest'), endDate: new Date('2021-12-15T00:00:00Z') }) }
+      const newestRecord = { entity: getMockRecurringPayment({ id: Symbol('newest'), endDate: new Date('2023-12-15T00:00:00Z') }) }
+      const middlestRecord = { entity: getMockRecurringPayment({ id: Symbol('middlest'), endDate: new Date('2022-12-15T00:00:00Z') }) }
+      dynamicsLib.executeQuery.mockReturnValueOnce([oldestRecord, newestRecord, middlestRecord])
+
+      await linkRecurringPayments('existing-recurring-payment-id', 'agreement-id')
+
+      expect(consoleLogSpy).toHaveBeenCalledWith('newRecurringPaymentId: ', newestRecord.id)
+    })
+
+    it('should log no matches when there are no matches', async () => {
+      const consoleLogSpy = jest.spyOn(console, 'log').mockImplementation(jest.fn())
+      dynamicsLib.executeQuery.mockReturnValueOnce([])
+
+      await linkRecurringPayments('existing-recurring-payment-id', 'agreement-id')
+
+      expect(consoleLogSpy).toHaveBeenCalledWith('No matches found')
+    })
+  })
 })

package/src/services/recurring-payments.service.js

@@ -1,4 +1,4 @@
-import { executeQuery, findDueRecurringPayments, RecurringPayment } from '@defra-fish/dynamics-lib'
+import { executeQuery, findDueRecurringPayments, findRecurringPaymentsByAgreementId, RecurringPayment } from '@defra-fish/dynamics-lib'
 import { calculateEndDate, generatePermissionNumber } from './permissions.service.js'
 import { getObfuscatedDob } from './contacts.service.js'
 import { createHash } from 'node:crypto'
@@ -126,3 +126,23 @@ export const processRPResult = async (transactionId, paymentId, createdDate) =>
 
   return { permission }
 }
+
+export const linkRecurringPayments = async (existingRecurringPaymentId, agreementId) => {
+  console.log('existingRecurringPaymentId: ', existingRecurringPaymentId)
+  console.log('agreementId: ', agreementId)
+  const newRecurringPayment = await findNewestExistingRecurringPaymentInCrm(agreementId)
+  if (newRecurringPayment) {
+    console.log('newRecurringPaymentId: ', newRecurringPayment.id)
+  } else {
+    console.log('No matches found')
+  }
+}
+
+const findNewestExistingRecurringPaymentInCrm = async agreementId => {
+  const matchingRecords = await executeQuery(findRecurringPaymentsByAgreementId(agreementId))
+  if (matchingRecords?.length) {
+    return [...matchingRecords].sort((a, b) => a.entity.endDate - b.entity.endDate).pop()
+  } else {
+    return false
+  }
+}

package/src/services/transactions/__tests__/process-transaction-queue.spec.js

@@ -7,10 +7,10 @@ import {
   FulfilmentRequest,
   Permission,
   PoclFile,
+  RecurringPayment,
   RecurringPaymentInstruction,
   Transaction,
-  TransactionJournal,
-  RecurringPayment
+  TransactionJournal
 } from '@defra-fish/dynamics-lib'
 import {
   mockFinalisedTransactionRecord,
@@ -65,7 +65,11 @@ jest.mock('@defra-fish/business-rules-lib', () => ({
   START_AFTER_PAYMENT_MINUTES: 30
 }))
 
-jest.mock('../../recurring-payments.service.js')
+jest.mock('../../recurring-payments.service.js', () => ({
+  findNewestExistingRecurringPaymentInCrm: jest.fn(),
+  generateRecurringPaymentRecord: jest.fn(),
+  processRecurringPayment: jest.fn()
+}))
 
 describe('transaction service', () => {
   beforeAll(() => {