account-lookup-service 17.8.0-snapshot.11 → 17.8.0-snapshot.12

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "account-lookup-service",
   "description": "Account Lookup Service is used to validate Party and Participant lookups.",
-  "version": "17.8.0-snapshot.11",
+  "version": "17.8.0-snapshot.12",
   "license": "Apache-2.0",
   "author": "ModusBox",
   "contributors": [
@@ -92,9 +92,9 @@
     "@hapi/vision": "7.0.3",
     "@mojaloop/central-services-error-handling": "13.0.7",
     "@mojaloop/central-services-health": "15.0.4",
-    "@mojaloop/central-services-logger": "11.8.0",
+    "@mojaloop/central-services-logger": "11.8.1",
     "@mojaloop/central-services-metrics": "12.5.0",
-    "@mojaloop/central-services-shared": "18.23.1",
+    "@mojaloop/central-services-shared": "18.23.2",
     "@mojaloop/central-services-stream": "11.5.2",
     "@mojaloop/database-lib": "11.1.4",
     "@mojaloop/event-sdk": "14.4.0",

package/src/domain/parties/services/BasePartiesService.js

@@ -100,16 +100,17 @@ class BasePartiesService {
   get state () { return this.#state }
 
   async handleError (error) {
-    const { headers, params } = this.inputs
+    const { params } = this.inputs
     const log = this.log.child({ method: 'handleError' })
     try {
       log.error('error in processing parties request: ', error)
       const fspiopError = ErrorHandler.Factory.reformatFSPIOPError(error)
-      const errorInfo = await this.deps.partiesUtils.makePutPartiesErrorPayload(this.deps.config, fspiopError, headers, params)
+      const callbackHeaders = BasePartiesService.createErrorCallbackHeaders(this.inputs.headers, params)
+      const errorInfo = await this.deps.partiesUtils.makePutPartiesErrorPayload(this.deps.config, fspiopError, callbackHeaders, params)
 
       await this.sendErrorCallback({
         errorInfo,
-        headers: BasePartiesService.createErrorCallbackHeaders(headers, params),
+        headers: callbackHeaders,
         params
       })
       log.info('handleError in done')

package/test/unit/domain/parties/services/BasePartiesService.test.js

@@ -54,7 +54,8 @@ describe('BasePartiesService Tests -->', () => {
     expect(sentTo).toBe(source)
     expect(payload.Rpt.Rsn.Cd).toBe('2001')
     expect(payload.Rpt.OrgnlId).toBe(`${params.Type}/${params.ID}`)
-    expect(payload.Assgnmt.Assgnr.Agt.FinInstnId.Othr.Id).toBe(source)
+    expect(payload.Assgnmt.Assgne.Agt.FinInstnId.Othr.Id).toBe(source)
+    expect(payload.Assgnmt.Assgnr.Agt.FinInstnId.Othr.Id).toBe(config.HUB_NAME)
   })
 
   test('should remove proxy getParties timeout cache key', async () => {

package/test/unit/domain/parties/services/GetPartiesService.test.js

@@ -199,15 +199,18 @@ describe('GetPartiesService Tests -->', () => {
       expect(participantMock.sendRequest.mock.lastCall[1]).toBe(proxyOk)
     })
 
-    test('should throw an error if proxyRequest failed after delay, and other proxies have already replied', async () => {
-      expect.assertions(1)
+    const throwDelayedErrorOnNthCall = (N, delay = 1000, error = new Error('Nth call Delayed Error')) => {
       let count = 0
-      participantMock.sendRequest = jest.fn(async () => {
+      return async () => {
         count++
-        if (count !== 2) return {}
-        await sleep(1000) // throw delayed error for 2nd proxy call
-        throw new Error('Proxy delayed error')
-      })
+        if (count !== N) return {}
+        await sleep(1000)
+        throw error
+      }
+    }
+
+    const prepareGetPartiesServiceForDelayedProxyError = () => {
+      participantMock.sendRequest = jest.fn(throwDelayedErrorOnNthCall(2)) // throw error on 2nd proxy call
       const proxies = createProxiesUtilMock({
         getAllProxiesNames: jest.fn().mockResolvedValue(['proxy1', 'proxy2'])
       })
@@ -215,14 +218,31 @@ describe('GetPartiesService Tests -->', () => {
         receivedErrorResponse: jest.fn().mockResolvedValue(true) // failed proxy request is last in inter-scheme discovery flow
       })
       const deps = createMockDeps({ proxies, proxyCache })
-
       const headers = fixtures.partiesCallHeadersDto({ destination: '' })
       const params = fixtures.partiesParamsDto()
-      const service = new GetPartiesService(deps, { headers, params })
+
+      return new GetPartiesService(deps, { headers, params })
+    }
+
+    test('should throw an error if proxyRequest failed after delay, and other proxies have already replied', async () => {
+      expect.assertions(1)
+      const service = prepareGetPartiesServiceForDelayedProxyError()
+      const { headers } = service.inputs
 
       await expect(service.triggerInterSchemeDiscoveryFlow(headers))
         .rejects.toThrow(ERROR_MESSAGES.noSuccessfulProxyDiscoveryResponses)
     })
+
+    test('should send error callback in ISO format if proxyRequest failed after delay, and other proxies have already replied', async () => {
+      const service = prepareGetPartiesServiceForDelayedProxyError()
+      const { headers } = service.inputs
+      service.deps.config.API_TYPE = API_TYPES.iso20022
+
+      await service.triggerInterSchemeDiscoveryFlow(headers)
+        .catch(err => service.handleError(err))
+      expect(participantMock.sendErrorToParticipant).toHaveBeenCalledTimes(1)
+      expect(participantMock.sendErrorToParticipant.mock.lastCall[2].Rpt.Rsn.Cd).toBe('3200')
+    })
   })
 
   describe('setProxyGetPartiesTimeout Tests', () => {