@defra-fish/sales-api-service 1.63.0-rc.4 → 1.63.0-rc.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/sales-api-service",
-  "version": "1.63.0-rc.4",
+  "version": "1.63.0-rc.6",
   "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.63.0-rc.4",
-    "@defra-fish/connectors-lib": "1.63.0-rc.4",
-    "@defra-fish/dynamics-lib": "1.63.0-rc.4",
+    "@defra-fish/business-rules-lib": "1.63.0-rc.6",
+    "@defra-fish/connectors-lib": "1.63.0-rc.6",
+    "@defra-fish/dynamics-lib": "1.63.0-rc.6",
     "@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": "19c5ac3cf85aa9f3dc5e7ebc0f14cfe88936125c"
+  "gitHead": "bb4bf03e0fdf3e8c04b00a1bf10b844475796434"
 }

package/src/schema/__tests__/transaction.schema.spec.js

@@ -1,4 +1,9 @@
-import { createTransactionSchema, createTransactionResponseSchema, finaliseTransactionResponseSchema } from '../transaction.schema.js'
+import {
+  createTransactionSchema,
+  createTransactionResponseSchema,
+  finaliseTransactionResponseSchema,
+  retrieveStagedTransactionParamsSchema
+} from '../transaction.schema.js'
 import { mockTransactionPayload, mockStagedTransactionRecord, mockFinalisedTransactionRecord } from '../../__mocks__/test-data.js'
 
 jest.mock('../validators/validators.js', () => ({
@@ -110,12 +115,9 @@ describe('createTransactionSchema', () => {
     await expect(createTransactionSchema.validateAsync(mockPayload)).resolves.not.toThrow()
   })
 
-  it.each([
-    ['agreement id', { id: 'fdc73d20-a0bf-4da6-9a49-2f0a24bd3509' }],
-    ['id', { agreementId: 'jhy7u8ii87uyhjui87u89ui8ie' }]
-  ])('fails validation if %s is omitted from recurring payment detail', async (_d, recurringPayment) => {
+  it('fails validation if agreement id is omitted from recurring payment detail', async () => {
     const mockPayload = mockTransactionPayload()
-    mockPayload.recurringPayment = recurringPayment
+    mockPayload.recurringPayment = { id: 'fdc73d20-a0bf-4da6-9a49-2f0a24bd3509' }
     await expect(() => createTransactionSchema.validateAsync(mockPayload)).rejects.toThrow()
   })
 
@@ -153,3 +155,32 @@ describe('finaliseTransactionResponseSchema', () => {
     expect(result).toBeInstanceOf(Object)
   })
 })
+
+describe('retrieveStagedTransactionParamsSchema', () => {
+  it.each([
+    ['36fb757c-6377-49c5-ab6e-32eb9782fcf0'],
+    ['c290b78d-3bbc-4445-b4dd-b36f6ee044a2'],
+    ['2323a890-b36f-47b1-ab9f-d60e292ac4ae'],
+    ['9c6b79be-28be-4916-aa5c-08520aa1e804']
+  ])('validates successfully when a uuid v4 transactionId is %s', async transactionId => {
+    const sampleData = { id: transactionId }
+    await expect(retrieveStagedTransactionParamsSchema.validateAsync(sampleData)).resolves.not.toThrow()
+  })
+
+  it.each([
+    ['uuid1 string', '5a429f62-871b-11ef-b864-0242ac120002'],
+    ['uuid2 string', '000003e8-871b-21ef-8000-325096b39f47'],
+    ['uuid3 string', 'a3bb189e-8bf9-3888-9912-ace4e6543002'],
+    ['uuid5 string', 'a6edc906-2f9f-5fb2-a373-efac406f0ef2'],
+    ['uuid6 string', 'a3bb189e-8bf9-3888-9912-ace4e6543002'],
+    ['uuid7 string', '01927705-ffac-77b5-89af-c97451b1bbe2'],
+    ['numeric', 4567]
+  ])('fails validation when provided with a %s for transactionId', async (_d, transactionId) => {
+    const sampleData = { id: transactionId }
+    await expect(() => retrieveStagedTransactionParamsSchema.validateAsync(sampleData)).rejects.toThrow()
+  })
+
+  it('throws an error if id missing', async () => {
+    await expect(() => retrieveStagedTransactionParamsSchema.validateAsync({}).rejects.toThrow())
+  })
+})

package/src/schema/transaction.schema.js

@@ -38,7 +38,7 @@ const createTransactionRequestSchemaContent = {
   transactionId: Joi.string().guid({ version: 'uuidv4' }).optional(),
   recurringPayment: Joi.object({
     agreementId: Joi.string().alphanum().length(AGREEMENT_ID_LENGTH).required(),
-    id: Joi.string().guid().required()
+    id: Joi.string().guid()
   }).optional()
 }
 
@@ -156,3 +156,7 @@ export const finaliseTransactionResponseSchema = Joi.object({
     .required()
     .label('finalise-transaction-status')
 }).label('finalise-transaction-response')
+
+export const retrieveStagedTransactionParamsSchema = Joi.object({
+  id: Joi.string().guid({ version: 'uuidv4' }).required()
+})

package/src/server/routes/__tests__/transactions.spec.js

@@ -1,6 +1,10 @@
 import initialiseServer from '../../server.js'
 import { mockTransactionPayload, mockStagedTransactionRecord } from '../../../__mocks__/test-data.js'
 import { v4 as uuidv4 } from 'uuid'
+import { retrieveStagedTransactionParamsSchema } from '../../../schema/transaction.schema.js'
+import { retrieveStagedTransaction } from '../../../services/transactions/retrieve-transaction.js'
+import transactions from '../transactions.js'
+
 jest.mock('../../../services/transactions/transactions.service.js', () => ({
   createTransaction: jest.fn(async () => mockStagedTransactionRecord()),
   createTransactions: jest.fn(async payloads => Array(payloads.length).fill(mockStagedTransactionRecord())),
@@ -9,6 +13,10 @@ jest.mock('../../../services/transactions/transactions.service.js', () => ({
   processDlq: jest.fn(async () => {})
 }))
 
+jest.mock('../../../services/transactions/retrieve-transaction.js', () => ({
+  retrieveStagedTransaction: jest.fn()
+}))
+
 jest.mock('../../../schema/validators/validators.js', () => ({
   ...jest.requireActual('../../../schema/validators/validators.js'),
   createOptionSetValidator: () => async () => undefined,
@@ -18,6 +26,20 @@ jest.mock('../../../schema/validators/validators.js', () => ({
   createPermitConcessionValidator: () => async () => undefined
 }))
 
+jest.mock('@defra-fish/connectors-lib', () => {
+  const awsMock = {
+    docClient: {
+      get: jest.fn(() => ({ Item: { id: 'abc123' } }))
+    }
+  }
+  return {
+    AWS: jest.fn(() => awsMock),
+    airbrake: {
+      initialise: jest.fn()
+    }
+  }
+})
+
 let server = null
 
 describe('transaction handler', () => {
@@ -214,4 +236,30 @@ describe('transaction handler', () => {
       expect(result).toBeUnprocessableEntityErrorResponse()
     })
   })
+
+  describe('retrieveStagedTransaction', () => {
+    const getMockRequest = ({ id = 'abc123' }) => ({ params: { id } })
+    const getMockResponseToolkit = () => ({ response: jest.fn() })
+    const retrieveHandler = transactions[transactions.length - 1].options.handler
+
+    it('handler should return continue response', async () => {
+      const request = getMockRequest({})
+      const responseToolkit = getMockResponseToolkit()
+      expect(await retrieveHandler(request, responseToolkit)).toEqual(responseToolkit.continue)
+    })
+
+    it('should call retrieveStagedTransaction with id', async () => {
+      const id = 'transaction-id'
+      const request = getMockRequest({ id })
+      await retrieveHandler(request, getMockResponseToolkit())
+      expect(retrieveStagedTransaction).toHaveBeenCalledWith(id)
+    })
+
+    it('should validate with cancelRecurringPaymentRequestParamsSchema', async () => {
+      const id = 'transaction-id'
+      const request = getMockRequest({ id })
+      await retrieveHandler(request, getMockResponseToolkit())
+      expect(transactions[5].options.validate.params).toBe(retrieveStagedTransactionParamsSchema)
+    })
+  })
 })

package/src/server/routes/transactions.js

@@ -7,6 +7,7 @@ import {
   processQueue,
   processDlq
 } from '../../services/transactions/transactions.service.js'
+import { retrieveStagedTransaction } from '../../services/transactions/retrieve-transaction.js'
 import {
   createTransactionSchema,
   createTransactionResponseSchema,
@@ -14,6 +15,7 @@ import {
   createTransactionBatchResponseSchema,
   finaliseTransactionRequestSchema,
   finaliseTransactionResponseSchema,
+  retrieveStagedTransactionParamsSchema,
   BATCH_CREATE_MAX_COUNT
 } from '../../schema/transaction.schema.js'
 import db from 'debug'
@@ -174,5 +176,29 @@ export default [
         }
       }
     }
+  },
+  {
+    method: 'GET',
+    path: '/retrieveStagedTransaction/{id}',
+    options: {
+      handler: async (request, h) => {
+        const { id } = request.params
+        const result = await retrieveStagedTransaction(id)
+        return h.response(result)
+      },
+      description: 'Retrieve a staged transaction',
+      tags: ['api', 'transactions'],
+      validate: {
+        params: retrieveStagedTransactionParamsSchema
+      },
+      plugins: {
+        'hapi-swagger': {
+          responses: {
+            200: { description: 'Staged transaction retreived' }
+          },
+          order: 5
+        }
+      }
+    }
   }
 ]

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

@@ -392,7 +392,9 @@ describe('recurring payments service', () => {
           ...permission
         }
       ],
-      agreementId,
+      recurringPayment: {
+        agreementId
+      },
       payment: {
         amount: 35.8,
         source: 'Gov Pay',

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

@@ -37,8 +37,8 @@ const getNextDueDate = (startDate, issueDate, endDate) => {
 }
 
 export const generateRecurringPaymentRecord = async (transactionRecord, permission) => {
-  if (transactionRecord.agreementId) {
-    const agreementResponse = await getRecurringPaymentAgreement(transactionRecord.agreementId)
+  if (transactionRecord.recurringPayment.agreementId) {
+    const agreementResponse = await getRecurringPaymentAgreement(transactionRecord.recurringPayment.agreementId)
     const lastDigitsCardNumbers = agreementResponse.payment_instrument?.card_details?.last_digits_card_number
     const [{ startDate, issueDate, endDate }] = transactionRecord.permissions
     return {
@@ -49,7 +49,7 @@ export const generateRecurringPaymentRecord = async (transactionRecord, permissi
           cancelledDate: null,
           cancelledReason: null,
           endDate,
-          agreementId: transactionRecord.agreementId,
+          agreementId: transactionRecord.recurringPayment.agreementId,
           status: 1,
           last_digits_card_number: lastDigitsCardNumbers
         }