@defra-fish/recurring-payments-job 1.64.0-rc.5 → 1.64.0-rc.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/recurring-payments-job",
-  "version": "1.64.0-rc.5",
+  "version": "1.64.0-rc.6",
   "description": "Rod Licensing Recurring Payments Job",
   "type": "module",
   "engines": {
@@ -36,11 +36,11 @@
     "test": "echo \"Error: run tests from root\" && exit 1"
   },
   "dependencies": {
-    "@defra-fish/business-rules-lib": "1.64.0-rc.5",
-    "@defra-fish/connectors-lib": "1.64.0-rc.5",
+    "@defra-fish/business-rules-lib": "1.64.0-rc.6",
+    "@defra-fish/connectors-lib": "1.64.0-rc.6",
     "commander": "7.2.0",
     "debug": "4.3.3",
     "moment-timezone": "0.5.34"
   },
-  "gitHead": "5085479bba611556749c49c4a87c29181bec4592"
+  "gitHead": "4ca80d63c58c37122b6fc8903840681202983468"
 }

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

@@ -70,6 +70,7 @@ const getMockDueRecurringPayment = ({ agreementId = 'test-agreement-id', id = 'a
   expanded: { activePermission: { entity: { referenceNumber } } }
 })
 
+// eslint-disable-next-line camelcase
 const getMockSendPaymentResponse = ({ payment_id = 'pay-1', agreementId = 'agr-1', created_date = '2025-01-01T00:00:00.000Z' } = {}) => ({
   payment_id,
   agreementId,
@@ -215,7 +216,8 @@ describe('recurring-payments-processor', () => {
   })
 
   describe('When payment request throws an error...', () => {
-    it('debug is called with error message', async () => {
+    it('console.error is called with error message', async () => {
+      jest.spyOn(console, 'error')
       salesApi.getDueRecurringPayments.mockReturnValueOnce(getMockPaymentRequestResponse())
       const oopsie = new Error('payment gate down')
       sendPayment.mockRejectedValueOnce(oopsie)
@@ -224,7 +226,7 @@ describe('recurring-payments-processor', () => {
         await execute()
       } catch {}
 
-      expect(debugLogger).toHaveBeenCalledWith(expect.any(String), oopsie)
+      expect(console.error).toHaveBeenCalledWith(expect.any(String), oopsie)
     })
 
     it('prepares and sends all payment requests, even if some fail', async () => {
@@ -283,6 +285,7 @@ describe('recurring-payments-processor', () => {
     })
 
     it('logs an error for every failure', async () => {
+      jest.spyOn(console, 'error')
       const errors = [new Error('error 1'), new Error('error 2'), new Error('error 3')]
       salesApi.getDueRecurringPayments.mockReturnValueOnce([
         getMockDueRecurringPayment({ referenceNumber: 'fee', agreementId: 'a1' }),
@@ -299,7 +302,7 @@ describe('recurring-payments-processor', () => {
 
       await execute()
 
-      expect(debugLogger).toHaveBeenCalledWith(expect.any(String), ...errors)
+      expect(console.error).toHaveBeenCalledWith(expect.any(String), ...errors)
     })
   })
 
@@ -583,6 +586,7 @@ describe('recurring-payments-processor', () => {
   })
 
   it('logs an error if createTransaction fails', async () => {
+    jest.spyOn(console, 'error')
     salesApi.getDueRecurringPayments.mockReturnValueOnce([getMockDueRecurringPayment()])
     const error = new Error('Wuh-oh!')
     salesApi.createTransaction.mockImplementationOnce(() => {
@@ -591,7 +595,7 @@ describe('recurring-payments-processor', () => {
 
     await execute()
 
-    expect(debugLogger).toHaveBeenCalledWith(expect.any(String), error)
+    expect(console.error).toHaveBeenCalledWith(expect.any(String), error)
   })
 
   // --- //
@@ -735,6 +739,7 @@ describe('recurring-payments-processor', () => {
     [512, 'Payment status API error for test-payment-id, error 512'],
     [599, 'Payment status API error for test-payment-id, error 599']
   ])('logs the correct message when getPaymentStatus rejects with HTTP %i', async (statusCode, expectedMessage) => {
+    jest.spyOn(console, 'error')
     const mockPaymentId = 'test-payment-id'
     const mockResponse = [
       { entity: { agreementId: 'agreement-1' }, expanded: { activePermission: { entity: { referenceNumber: 'ref-1' } } } }
@@ -752,10 +757,11 @@ describe('recurring-payments-processor', () => {
 
     await execute()
 
-    expect(debugLogger).toHaveBeenCalledWith(expectedMessage)
+    expect(console.error).toHaveBeenCalledWith(expectedMessage)
   })
 
   it('logs the generic unexpected-error message and still rejects', async () => {
+    jest.spyOn(console, 'error')
     const mockPaymentId = 'test-payment-id'
     salesApi.getDueRecurringPayments.mockResolvedValueOnce(getMockPaymentRequestResponse())
     salesApi.createTransaction.mockResolvedValueOnce({ id: mockPaymentId })
@@ -770,7 +776,7 @@ describe('recurring-payments-processor', () => {
 
     await execute()
 
-    expect(debugLogger).toHaveBeenCalledWith(`Unexpected error fetching payment status for ${mockPaymentId}.`)
+    expect(console.error).toHaveBeenCalledWith(`Unexpected error fetching payment status for ${mockPaymentId}.`)
   })
 
   it('should call setTimeout with correct delay when there are recurring payments', async () => {

package/src/recurring-payments-processor.js

@@ -70,7 +70,7 @@ const requestPayments = async dueRCPayments => {
   const payments = paymentRequestResults.filter(prr => prr.status === 'fulfilled').map(p => p.value)
   const failures = paymentRequestResults.filter(prr => prr.status === 'rejected').map(f => f.reason)
   if (failures.length) {
-    debug('Error requesting payments:', ...failures)
+    console.error('Error requesting payments:', ...failures)
   }
   return payments
 }
@@ -178,11 +178,11 @@ const processRecurringPaymentStatus = async payment => {
     const status = error.response?.status
 
     if (isClientError(status)) {
-      debug(`Failed to fetch status for payment ${payment.paymentId}, error ${status}`)
+      console.error(`Failed to fetch status for payment ${payment.paymentId}, error ${status}`)
     } else if (isServerError(status)) {
-      debug(`Payment status API error for ${payment.paymentId}, error ${status}`)
+      console.error(`Payment status API error for ${payment.paymentId}, error ${status}`)
     } else {
-      debug(`Unexpected error fetching payment status for ${payment.paymentId}.`)
+      console.error(`Unexpected error fetching payment status for ${payment.paymentId}.`)
     }
   }
 }

package/src/services/__tests__/govuk-pay-service.spec.js

@@ -57,19 +57,6 @@ describe('govuk-pay-service', () => {
       }
     })
 
-    it('should log error message when the GOV.UK Pay API raises an error', async () => {
-      const consoleSpy = jest.spyOn(console, 'error').mockImplementation(() => {})
-      govUkPayApi.createPayment.mockImplementationOnce(() => {
-        throw new Error()
-      })
-
-      try {
-        await sendPayment(preparedPayment)
-      } catch (error) {
-        expect(consoleSpy).toHaveBeenCalledWith('Error creating payment', preparedPayment.id)
-      }
-    })
-
     it('should throw an error when response is not ok', async () => {
       const mockFetchResponse = {
         ok: false,
@@ -92,43 +79,6 @@ describe('govuk-pay-service', () => {
         })
       ).rejects.toThrow('Unexpected response from GOV.UK Pay API')
     })
-
-    it('should log details when response is not ok', async () => {
-      const status = 400
-      const serviceResponseBody = {
-        code: 'P0102',
-        field: 'agreement_id',
-        description: 'Invalid attribute value: agreement_id. Agreement does not exist'
-      }
-      const transactionId = 'a50f0d51-295f-42b3-98f8-97c0641ede5a'
-      const preparedPayment = {
-        amount: 100,
-        description: 'The recurring card payment for your rod fishing licence',
-        id: transactionId,
-        authorisation_mode: 'agreement',
-        agreement_id: 'does_not_exist'
-      }
-      govUkPayApi.createPayment.mockResolvedValueOnce({
-        ok: false,
-        status,
-        json: jest.fn().mockResolvedValue(serviceResponseBody)
-      })
-      jest.spyOn(console, 'error')
-
-      try {
-        await sendPayment(preparedPayment)
-      } catch {}
-
-      expect(console.error).toHaveBeenCalledWith(
-        expect.objectContaining({
-          method: 'POST',
-          status,
-          response: serviceResponseBody,
-          transactionId,
-          payload: preparedPayment
-        })
-      )
-    })
   })
 
   describe('getPaymentStatus', () => {

package/src/services/govuk-pay-service.js

@@ -3,24 +3,15 @@ import db from 'debug'
 const debug = db('recurring-payments:gov.uk-pay-service')
 
 export const sendPayment = async preparedPayment => {
-  const createPayment = async () => {
-    try {
-      return await govUkPayApi.createPayment(preparedPayment, true)
-    } catch (e) {
-      console.error('Error creating payment', preparedPayment.id)
-      throw e
-    }
-  }
+  const createPayment = () => govUkPayApi.createPayment(preparedPayment, true)
   const response = await createPayment()
   if (!response.ok) {
-    console.error({
-      method: 'POST',
-      status: response.status,
-      response: await response.json(),
-      transactionId: preparedPayment.id,
-      payload: preparedPayment
-    })
-    throw new Error('Unexpected response from GOV.UK Pay API')
+    throw new Error(`Unexpected response from GOV.UK Pay API. 
+      Status: ${response.status}, 
+      Response: ${JSON.stringify(await response.json())}
+      Transaction ID: ${preparedPayment.id}
+      Payload: ${JSON.stringify(preparedPayment)}
+    `)
   }
   return response.json()
 }