@defra-fish/sales-api-service 1.63.0-rc.5 → 1.63.0-rc.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/sales-api-service",
-  "version": "1.63.0-rc.5",
+  "version": "1.63.0-rc.7",
   "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.5",
-    "@defra-fish/connectors-lib": "1.63.0-rc.5",
-    "@defra-fish/dynamics-lib": "1.63.0-rc.5",
+    "@defra-fish/business-rules-lib": "1.63.0-rc.7",
+    "@defra-fish/connectors-lib": "1.63.0-rc.7",
+    "@defra-fish/dynamics-lib": "1.63.0-rc.7",
     "@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": "313d1d1a8ab65bd841c27cf615f8abeec527f401"
+  "gitHead": "345c44f58b91a92723579e072235aa9607827efb"
 }

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', () => ({
@@ -150,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

@@ -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

@@ -530,6 +530,15 @@ describe('recurring payments service', () => {
       await expect(generateRecurringPaymentRecord(sampleTransaction)).rejects.toThrow('Invalid dates provided for permission')
     })
 
+    it('returns a false flag when recurringPayment is not present', async () => {
+      const sampleTransaction = createFinalisedSampleTransaction()
+      delete sampleTransaction.recurringPayment
+
+      const rpRecord = await generateRecurringPaymentRecord(sampleTransaction)
+
+      expect(rpRecord.payment?.recurring).toBeFalsy()
+    })
+
     it('returns a false flag when agreementId is not present', async () => {
       const sampleTransaction = createFinalisedSampleTransaction(
         null,

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

@@ -37,7 +37,7 @@ const getNextDueDate = (startDate, issueDate, endDate) => {
 }
 
 export const generateRecurringPaymentRecord = async (transactionRecord, permission) => {
-  if (transactionRecord.recurringPayment.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