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

package/package.json

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

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

@@ -96,27 +96,45 @@ describe('createTransactionSchema', () => {
     await expect(createTransactionSchema.validateAsync(mockPayload)).rejects.toThrow()
   })
 
-  it('validates successfully when an agreementId is supplied', async () => {
+  it('validates successfully when recurring payment detail is supplied', async () => {
     const mockPayload = mockTransactionPayload()
-    mockPayload.agreementId = 't3jl08v2nqqmujrnhs09pmhtjx'
+    mockPayload.recurringPayment = {
+      agreementId: 't3jl08v2nqqmujrnhs09pmhtjx',
+      id: 'fdc73d20-a0bf-4da6-9a49-2f0a24bd3509'
+    }
     await expect(createTransactionSchema.validateAsync(mockPayload)).resolves.not.toThrow()
   })
 
-  it('validates successfully when agreementId is omitted', async () => {
+  it('validates successfully when recurring payment detail is omitted', async () => {
     const mockPayload = mockTransactionPayload()
     await expect(createTransactionSchema.validateAsync(mockPayload)).resolves.not.toThrow()
   })
 
   it.each([
-    ['too short string', 'foo'],
-    ['too long string', 'foobarbazfoobarbazfoobarbaz'],
-    ['string containing invalid characters', '!3j@08v2nqqmujrnhs09_mhtjx'],
-    ['null', null],
-    ['numeric', 4567]
-  ])('fails validation when provided with a %s for agreementId', async (_d, agreementId) => {
+    ['agreement id', { id: 'fdc73d20-a0bf-4da6-9a49-2f0a24bd3509' }],
+    ['id', { agreementId: 'jhy7u8ii87uyhjui87u89ui8ie' }]
+  ])('fails validation if %s is omitted from recurring payment detail', async (_d, recurringPayment) => {
     const mockPayload = mockTransactionPayload()
-    mockPayload.agreementId = agreementId
-    await expect(createTransactionSchema.validateAsync(mockPayload)).rejects.toThrow()
+    mockPayload.recurringPayment = recurringPayment
+    await expect(() => createTransactionSchema.validateAsync(mockPayload)).rejects.toThrow()
+  })
+
+  it.each([
+    ['agreement id is too long', { agreementId: 'thisistoolongtobeanagreementid' }],
+    ['agreement id is too short', { agreementId: 'tooshorttobeanagreementid' }],
+    ['agreement id contains invalid characters', '!3j@08v2nqqmujrnhs09_mhtjx'],
+    ['agreement id is null', { agreementId: null }],
+    ['agreement id is a numeric', { agreementId: 4567 }],
+    ['id is not a guid', { id: 'not-a-guid' }],
+    ['id is null', { id: null }]
+  ])('fails validation if %s', async (_d, recurringPayment) => {
+    const mockPayload = mockTransactionPayload()
+    mockPayload.recurringPayment = {
+      agreementId: 'jhyu78iujhy7u87y6thu87uyj8',
+      id: '7a0660ec-8535-4357-b925-e598a9358119',
+      ...recurringPayment
+    }
+    await expect(() => createTransactionSchema.validateAsync(mockPayload)).rejects.toThrow()
   })
 })
 

package/src/schema/transaction.schema.js

@@ -36,7 +36,10 @@ const createTransactionRequestSchemaContent = {
   createdBy: Joi.string().optional(),
   journalId: Joi.string().optional(),
   transactionId: Joi.string().guid({ version: 'uuidv4' }).optional(),
-  agreementId: Joi.string().alphanum().length(AGREEMENT_ID_LENGTH).optional()
+  recurringPayment: Joi.object({
+    agreementId: Joi.string().alphanum().length(AGREEMENT_ID_LENGTH).required(),
+    id: Joi.string().guid().required()
+  }).optional()
 }
 
 /**