@mojaloop/sdk-scheme-adapter 12.3.0 → 13.0.2

package/test/unit/api/transfers/utils.js

@@ -35,7 +35,7 @@ function createGetTransfersTester({ reqInbound, reqOutbound, apiSpecsOutbound })
 
             return reqInbound.put(putUrl)
                 .send(putBody)
-                .set('content-type', 'application/vnd.interoperability.transfers+json;version=1.0')
+                .set('content-type', 'application/vnd.interoperability.transfers+json;version=1.1')
                 .set('Date', new Date().toISOString())
                 .set('fspiop-source', 'mojaloop-sdk')
                 .expect(200);
@@ -50,9 +50,49 @@ function createGetTransfersTester({ reqInbound, reqOutbound, apiSpecsOutbound })
         const res = await reqOutbound.get(`/transfers/${TRANSFER_ID}`);
         const {body} = res;
         expect(res.statusCode).toEqual(responseCode);
+
+        // remove elements of the response we do not want/need to compare for correctness.
+        // timestamps on requests/responses for example will be set by the HTTP framework
+        // and we dont want to compare against static values.
         delete body.initiatedTimestamp;
         if (body.transferState) {
             delete body.transferState.initiatedTimestamp;
+            if(body.transferState.quoteResponse) {
+                delete body.transferState.quoteResponse.headers;
+            }
+            if(body.transferState.getPartiesResponse) {
+                delete body.transferState.getPartiesResponse.headers;
+            }
+            if(body.transferState.fulfil) {
+                delete body.transferState.fulfil.headers;
+            }
+            if(body.transferState.quoteRequest) {
+                delete body.transferState.quoteRequest;
+            }
+            if(body.transferState.getPartiesRequest) {
+                delete body.transferState.getPartiesRequest;
+            }
+            if(body.transferState.prepare) {
+                delete body.transferState.prepare;
+            }
+        }
+        if(body.quoteResponse) {
+            delete body.quoteResponse.headers;
+        }
+        if(body.getPartiesResponse) {
+            delete body.getPartiesResponse.headers;
+        }
+        if(body.fulfil) {
+            delete body.fulfil.headers;
+        }
+        if(body.quoteRequest) {
+            delete body.quoteRequest;
+        }
+        if(body.getPartiesRequest) {
+            delete body.getPartiesRequest;
+        }
+        if(body.prepare) {
+            delete body.prepare;
         }
         expect(body).toEqual(responseBody);
         const responseValidator = new OpenAPIResponseValidator(apiSpecsOutbound.paths['/transfers/{transferId}'].get);
@@ -108,17 +148,17 @@ function createPostTransfersTester(
             if (urlPath.startsWith('/parties/')) {
                 putBody = await Promise.resolve(bodyFn.parties.put());
                 putUrl = urlPath;
-                contentType = 'application/vnd.interoperability.parties+json;version=1.0';
+                contentType = 'application/vnd.interoperability.parties+json;version=1.1';
             } else if (urlPath === '/quotes') {
                 expect(body).toEqual(bodyFn.quotes.post(body));
                 putBody = await Promise.resolve(bodyFn.quotes.put(body));
                 putUrl = `/quotes/${body.quoteId}`;
-                contentType = 'application/vnd.interoperability.quotes+json;version=1.0';
+                contentType = 'application/vnd.interoperability.quotes+json;version=1.1';
             } else if (urlPath === '/transfers') {
                 expect(body).toEqual(bodyFn.transfers.post(body));
                 putBody = await Promise.resolve(bodyFn.transfers.put(body));
                 putUrl = `/transfers/${body.transferId}`;
-                contentType = 'application/vnd.interoperability.transfers+json;version=1.0';
+                contentType = 'application/vnd.interoperability.transfers+json;version=1.1';
             } else {
                 throw new Error(`Unexpected url ${urlPath}`);
             }
@@ -162,10 +202,51 @@ function createPostTransfersTester(
 
         const res = await reqOutbound.post('/transfers').send(postTransfersSimpleBody);
         const {body} = res;
+
         expect(res.statusCode).toEqual(responseCode);
+
+        // remove elements of the response we do not want/need to compare for correctness.
+        // timestamps on requests/responses for example will be set by the HTTP framework
+        // and we dont want to compare against static values.
         delete body.initiatedTimestamp;
         if (body.transferState) {
             delete body.transferState.initiatedTimestamp;
+            if(body.transferState.quoteResponse) {
+                delete body.transferState.quoteResponse.headers;
+            }
+            if(body.transferState.getPartiesResponse) {
+                delete body.transferState.getPartiesResponse.headers;
+            }
+            if(body.transferState.fulfil) {
+                delete body.transferState.fulfil.headers;
+            }
+            if(body.transferState.quoteRequest) {
+                delete body.transferState.quoteRequest;
+            }
+            if(body.transferState.getPartiesRequest) {
+                delete body.transferState.getPartiesRequest;
+            }
+            if(body.transferState.prepare) {
+                delete body.transferState.prepare;
+            }
+        }
+        if(body.quoteResponse) {
+            delete body.quoteResponse.headers;
+        }
+        if(body.getPartiesResponse) {
+            delete body.getPartiesResponse.headers;
+        }
+        if(body.fulfil) {
+            delete body.fulfil.headers;
+        }
+        if(body.quoteRequest) {
+            delete body.quoteRequest;
+        }
+        if(body.getPartiesRequest) {
+            delete body.getPartiesRequest;
+        }
+        if(body.prepare) {
+            delete body.prepare;
         }
         expect(body).toEqual(responseBody);
         const responseValidator = new OpenAPIResponseValidator(apiSpecsOutbound.paths['/transfers'].post);

package/test/unit/data/commonHttpHeaders.json

@@ -1,4 +1,5 @@
 {
+  "content-type": "application/vnd.interoperability.parties+json;version=1.1",
   "fspiop-source": "other-dfsp",
   "fspiop-destination": "mojaloop-sdk",
   "fspiop-signature": "{\"signature\":\"aTTa1TTCBJA1K1VoEFgpSicWYU0q1VYXV-bjkk7uoeNicog7QSp9_AbwtYm4u8NJ1HFM_3mekE8wioAs5YNugnTlJ1k-q4Ouvp5Jo3ZnozoPVtnLaqdhxRMUBOHfDp0X8eCHEo7lETjKcCcH4r5_KT_9Vwx5TMytoG_y9Be8PpviJFkOqOV5jCeIl7XzL_pZQoY0pRJdkXDzYpXDu-HTYKr8ckxWQzx4HO-viJQd2ByQkbqPfQom9IQaAX1t4yztCCpOQn1LY9j9sbfEX9RPXG3UbY6UyDsNjUKYP9BAhXwI9pFWlgv2i9FvEtay2QYdwbW7XEpIiGZ_vi5d6yc12w\",\"protectedHeader\":\"eyJhbGciOiJSUzI1NiIsIkZTUElPUC1VUkkiOiIvcGFydGllcy9NU0lTRE4vMTIzNDU2Nzg5IiwiRlNQSU9QLUhUVFAtTWV0aG9kIjoiUFVUIiwiRlNQSU9QLVNvdXJjZSI6InNpbSIsIkZTUElPUC1EZXN0aW5hdGlvbiI6ImRmc3AiLCJEYXRlIjoiVGh1LCAzMSBPY3QgMjAxOSAxMTo0MTo0MyBHTVQifQ\"}",

package/test/unit/inboundApi/handlers.test.js

@@ -58,7 +58,7 @@ describe('Inbound API handlers:', () => {
             await expect(handlers['/quotes'].post(mockContext)).resolves.toBe(undefined);
 
             expect(quoteRequestSpy).toHaveBeenCalledTimes(1);
-            expect(quoteRequestSpy.mock.calls[0][0]).toBe(mockContext.request.body);
+            expect(quoteRequestSpy.mock.calls[0][0]).toStrictEqual(mockContext.request);
             expect(quoteRequestSpy.mock.calls[0][1]).toBe(mockContext.request.headers['fspiop-source']);
         });
 
@@ -138,7 +138,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     quoteId: mockContext.state.path.params.ID
                 }
@@ -215,8 +218,10 @@ describe('Inbound API handlers:', () => {
             expect(bulkQuotesSpy).toHaveBeenCalledTimes(1);
             expect(bulkQuotesSpy.mock.calls[0][1]).toMatchObject({
                 type: 'bulkQuoteResponse',
-                data: mockContext.request.body,
-                headers: mockContext.request.headers
+                data: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                }
             });
         });
     });
@@ -263,7 +268,10 @@ describe('Inbound API handlers:', () => {
             expect(bulkQuotesSpy).toHaveBeenCalledTimes(1);
             expect(bulkQuotesSpy.mock.calls[0][1]).toMatchObject({
                 type: 'bulkQuoteResponseError',
-                data: mockContext.request.body
+                data: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                }
             });
         });
     });
@@ -372,8 +380,10 @@ describe('Inbound API handlers:', () => {
             expect(bulkTransfersSpy).toHaveBeenCalledTimes(1);
             expect(bulkTransfersSpy.mock.calls[0][1]).toMatchObject({
                 type: 'bulkTransferResponse',
-                data: mockContext.request.body,
-                headers: mockContext.request.headers
+                data: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                }
             });
         });
     });
@@ -420,7 +430,10 @@ describe('Inbound API handlers:', () => {
             expect(bulkTransfersSpy).toHaveBeenCalledTimes(1);
             expect(bulkTransfersSpy.mock.calls[0][1]).toMatchObject({
                 type: 'bulkTransferResponseError',
-                data: mockContext.request.body
+                data: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                }
             });
         });
     });
@@ -601,7 +614,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     type: mockContext.state.path.params.Type,
                     id: mockContext.state.path.params.ID
@@ -620,7 +636,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     type: mockContext.state.path.params.Type,
                     id: mockContext.state.path.params.ID,
@@ -667,7 +686,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     transferId: mockContext.state.path.params.ID
                 }
@@ -682,7 +704,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     transferId: mockContext.state.path.params.ID,
                 }
@@ -728,7 +753,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     transferId: mockContext.state.path.params.ID
                 }
@@ -743,7 +771,10 @@ describe('Inbound API handlers:', () => {
             expect(triggerDeferredJobSpy).toHaveBeenCalledTimes(1);
             expect(triggerDeferredJobSpy).toBeCalledWith({
                 cache: mockContext.state.cache,
-                message: mockContext.request.body,
+                message: {
+                    body: mockContext.request.body,
+                    headers: mockContext.request.headers,
+                },
                 args: {
                     transferId: mockContext.state.path.params.ID,
                 }

package/test/unit/lib/model/AccountsModel.test.js

@@ -51,14 +51,17 @@ describe('AccountsModel', () => {
             const response = {
                 type: 'accountsCreationSuccessfulResponse',
                 data: {
-                    partyList: request.partyList.map(party => ({
-                        partyId: party,
-                        // errorInformation: null
-                    })),
-                    currency: request.currency,
+                    body: {
+                        partyList: request.partyList.map(party => ({
+                            partyId: party,
+                            // errorInformation: null
+                        })),
+                        currency: request.currency,
+                    },
+                    headers: {}
                 },
             };
-            cache.publish(`ac_${request.requestId}`, JSON.stringify(response));
+            cache.publish(`ac_${request.requestId}`, response);
             return Promise.resolve();
         });
 

package/test/unit/lib/model/InboundTransfersModel.test.js

@@ -55,6 +55,12 @@ describe('inboundModel', () => {
         beforeEach(async () => {
             expectedQuoteResponseILP = Ilp.__response;
             BackendRequests.__postQuoteRequests = jest.fn().mockReturnValue(Promise.resolve(mockArgs.internalQuoteResponse));
+            MojaloopRequests.__putQuotes = jest.fn().mockReturnValue(Promise.resolve({
+                originalRequest: {
+                    headers: {},
+                    body: {},
+                }
+            }));
 
             cache = new Cache({
                 host: 'dummycachehost',
@@ -78,11 +84,18 @@ describe('inboundModel', () => {
         test('calls `mojaloopRequests.putQuotes` with the expected arguments.', async () => {
             await model.quoteRequest(mockArgs.quoteRequest, mockArgs.fspId);
 
+            expect(BackendRequests.__postQuoteRequests).toHaveBeenCalledTimes(1);
+            expect(BackendRequests.__postQuoteRequests.mock.calls[0][0]).toEqual(mockArgs.internalQuoteRequest);
+
             expect(MojaloopRequests.__putQuotes).toHaveBeenCalledTimes(1);
             expect(MojaloopRequests.__putQuotes.mock.calls[0][1].expiration).toBe(mockArgs.internalQuoteResponse.expiration);
             expect(MojaloopRequests.__putQuotes.mock.calls[0][1].ilpPacket).toBe(expectedQuoteResponseILP.ilpPacket);
             expect(MojaloopRequests.__putQuotes.mock.calls[0][1].condition).toBe(expectedQuoteResponseILP.condition);
             expect(MojaloopRequests.__putQuotes.mock.calls[0][2]).toBe(mockArgs.fspId);
+
+            // check the extension list gets translated correctly to the mojaloop form
+            expect(MojaloopRequests.__putQuotes.mock.calls[0][1].extensionList)
+                .toStrictEqual(mockArgs.internalQuoteResponse.extensionList);
         });
 
         test('adds a custom `expiration` property in case it is not defined.', async() => {
@@ -247,7 +260,12 @@ describe('inboundModel', () => {
         beforeEach(async () => {
             MojaloopRequests.__putTransfersError.mockClear();
             BackendRequests.__postTransfers = jest.fn().mockReturnValue(Promise.resolve({}));
-            MojaloopRequests.__putTransfers = jest.fn().mockReturnValue(Promise.resolve({}));
+            MojaloopRequests.__putTransfers = jest.fn().mockReturnValue(Promise.resolve({
+                originalRequest: {
+                    headers: {},
+                    body: {},
+                }
+            }));
 
             cache = new Cache({
                 host: 'dummycachehost',
@@ -269,13 +287,17 @@ describe('inboundModel', () => {
                 logger,
                 rejectTransfersOnExpiredQuotes: true,
             });
-            cache.set(`quote_${TRANSFER_ID}`, {
-                mojaloopResponse: {
-                    expiration: new Date(new Date().getTime() - 1000).toISOString(),
+            cache.set(`transferModel_in_${TRANSFER_ID}`, {
+                quote: {
+                    mojaloopResponse: {
+                        expiration: new Date(new Date().getTime() - 1000).toISOString(),
+                    }
                 }
             });
             const args = {
-                transferId: TRANSFER_ID,
+                body: {
+                    transferId: TRANSFER_ID,
+                }
             };
 
             await model.prepareTransfer(args, mockArgs.fspId);
@@ -358,13 +380,15 @@ describe('inboundModel', () => {
         test('fail on transfer without quote.', async () => {
             const TRANSFER_ID = 'without_quote-transfer-id';
             const args = {
-                transferId: TRANSFER_ID,
-                amount: {
-                    currency: 'USD',
-                    amount: 20.13
-                },
-                ilpPacket: 'mockBase64encodedIlpPacket',
-                condition: 'mockGeneratedCondition'
+                body: {
+                    transferId: TRANSFER_ID,
+                    amount: {
+                        currency: 'USD',
+                        amount: 20.13
+                    },
+                    ilpPacket: 'mockBase64encodedIlpPacket',
+                    condition: 'mockGeneratedCondition'
+                }
             };
 
             const model = new Model({
@@ -382,16 +406,76 @@ describe('inboundModel', () => {
             expect(call[1].errorInformation.errorCode).toEqual('2001');
         });
 
-        test('pass on transfer without quote.', async () => {
-            const TRANSFER_ID = 'without_quote-transfer-id';
+        test('stores homeTransactionId in cache when received by dfsp acting as payee', async () => {
+            const TRANSFER_ID = 'transfer-id';
+            const HOME_TRANSACTION_ID = 'mockHomeTransactionId';
+            shared.mojaloopPrepareToInternalTransfer = jest.fn().mockReturnValueOnce({});
+
+            // mock response from dfsp acting as payee
+            BackendRequests.__postTransfers = jest.fn().mockReturnValueOnce(Promise.resolve({
+                homeTransactionId: HOME_TRANSACTION_ID,
+                transferId: TRANSFER_ID
+            }));
+
             const args = {
+                body: {
+                    transferId: TRANSFER_ID,
+                    amount: {
+                        currency: 'USD',
+                        amount: 20.13
+                    },
+                    ilpPacket: 'mockBase64encodedIlpPacket',
+                    condition: 'mockGeneratedCondition'
+                }
+            };
+
+            const model = new Model({
+                ...config,
+                cache,
+                logger,
+                checkIlp: false,
+                rejectTransfersOnExpiredQuotes: false
+            });
+
+            cache.set(`transferModel_in_${TRANSFER_ID}`, {
                 transferId: TRANSFER_ID,
-                amount: {
-                    currency: 'USD',
-                    amount: 20.13
+                quote: {
+                    fulfilment: 'mockFulfilment',
+                    mojaloopResponse: {
+                        condition: 'mockCondition',
+                    }
+                }
+            });
+
+            await model.prepareTransfer(args, mockArgs.fspId);
+
+            expect(MojaloopRequests.__putTransfersError).toHaveBeenCalledTimes(0);
+            expect(BackendRequests.__postTransfers).toHaveBeenCalledTimes(1);
+            expect(MojaloopRequests.__putTransfers).toHaveBeenCalledTimes(1);
+            expect((await cache.get(`transferModel_in_${TRANSFER_ID}`)).homeTransactionId)
+                .toEqual(HOME_TRANSACTION_ID);
+        });
+
+        test('pass on transfer without quote.', async () => {
+            const TRANSFER_ID = 'without_quote-transfer-id';
+            cache.set(`transferModel_in_${TRANSFER_ID}`, {
+                fulfilment: '',
+                mojaloopResponse: {
+                    response: ''
                 },
-                ilpPacket: 'mockBase64encodedIlpPacket',
-                condition: 'mockGeneratedCondition'
+                quote: null
+            });
+
+            const args = {
+                body: {
+                    transferId: TRANSFER_ID,
+                    amount: {
+                        currency: 'USD',
+                        amount: 20.13
+                    },
+                    ilpPacket: 'mockBase64encodedIlpPacket',
+                    condition: 'mockGeneratedCondition'
+                }
             };
 
             const model = new Model({
@@ -413,21 +497,29 @@ describe('inboundModel', () => {
             const TRANSFER_ID = 'transfer-id';
             shared.mojaloopPrepareToInternalTransfer = jest.fn().mockReturnValueOnce({});
 
-            cache.set(`quote_${transactionId}`, {
+            cache.set(`transferModel_in_${transactionId}`, {
                 fulfilment: '',
                 mojaloopResponse: {
                     response: ''
+                },
+                quote: {
+                    fulfilment: 'mockFulfilment',
+                    mojaloopResponse: {
+                        condition: 'mockCondition',
+                    }
                 }
             });
 
             const args = {
-                transferId: TRANSFER_ID,
-                amount: {
-                    currency: 'USD',
-                    amount: 20.13
-                },
-                ilpPacket: 'mockIlpPacket',
-                condition: 'mockGeneratedCondition'
+                body: {
+                    transferId: TRANSFER_ID,
+                    amount: {
+                        currency: 'USD',
+                        amount: 20.13
+                    },
+                    ilpPacket: 'mockIlpPacket',
+                    condition: 'mockGeneratedCondition'
+                }
             };
 
             const model = new Model({
@@ -648,7 +740,12 @@ describe('inboundModel', () => {
 
         test('sends notification to fsp backend', async () => {
             BackendRequests.__putTransfersNotification = jest.fn().mockReturnValue(Promise.resolve({}));
-            const backendResponse = JSON.parse(JSON.stringify(notificationToPayee));
+            const notif = JSON.parse(JSON.stringify(notificationToPayee));
+
+            const expectedRequest = {
+                currentState: 'COMPLETED',
+                finalNotification: notif.data,
+            };
 
             const model = new Model({
                 ...config,
@@ -656,11 +753,94 @@ describe('inboundModel', () => {
                 logger,
             });
 
-            await model.sendNotificationToPayee(backendResponse.data, transferId);
+            await model.sendNotificationToPayee(notif.data, transferId);
             expect(BackendRequests.__putTransfersNotification).toHaveBeenCalledTimes(1);
             const call = BackendRequests.__putTransfersNotification.mock.calls[0];
-            expect(call[0]).toEqual(backendResponse.data);
+            expect(call[0]).toEqual(expectedRequest);
             expect(call[1]).toEqual(transferId);
         });
     });
+
+    describe('error handling:', () => {
+        let cache;
+        beforeEach(async () => {
+            cache = new Cache({
+                host: 'dummycachehost',
+                port: 1234,
+                logger,
+            });
+            await cache.connect();
+        });
+        afterEach(async () => {
+            await cache.disconnect();
+        });
+        test('creates mojaloop spec error body when backend returns standard error code', async () => {
+            const model = new Model({
+                ...config,
+                cache,
+                logger,
+            });
+            const testErr = new HTTPResponseError({
+                msg: 'Request returned non-success status code 500',
+                res: {
+                    data: {
+                        statusCode: '3200',
+                    },
+                }
+            });
+            const err = await model._handleError(testErr);
+            expect(err).toBeDefined();
+            expect(err.errorInformation).toBeDefined();
+            expect(err.errorInformation.errorCode).toEqual('3200');
+            // error message should be the default one, not custom.
+            // it is debatibale whether this is truly correct, to overwrite
+            // and custom error message; but it is the case for now.
+            expect(err.errorInformation.errorDescription).toEqual('Generic ID not found');
+        });
+        test('creates custom error body when backend returns custom error code', async () => {
+            const model = new Model({
+                ...config,
+                cache,
+                logger,
+            });
+            const customMessage = 'some custom message';
+            const testErr = new HTTPResponseError({
+                msg: 'Request returned non-success status code 500',
+                res: {
+                    data: {
+                        statusCode: '3299',
+                        message: customMessage,
+                    },
+                }
+            });
+            const err = await model._handleError(testErr);
+            expect(err).toBeDefined();
+            expect(err.errorInformation).toBeDefined();
+            expect(err.errorInformation.errorCode).toEqual('3299');
+            expect(err.errorInformation.errorDescription).toEqual(customMessage);
+        });
+        test('creates custom error message when backend returns standard error code and message', async () => {
+            const model = new Model({
+                ...config,
+                cache,
+                logger,
+            });
+            const customMessage = 'some custom message';
+            const testErr = new HTTPResponseError({
+                msg: 'Request returned non-success status code 500',
+                res: {
+                    data: {
+                        statusCode: '3200',
+                        message: customMessage,
+                    },
+                }
+            });
+            const err = await model._handleError(testErr);
+            expect(err).toBeDefined();
+            expect(err.errorInformation).toBeDefined();
+            expect(err.errorInformation.errorCode).toEqual('3200');
+            // error message should be custom
+            expect(err.errorInformation.errorDescription).toEqual(customMessage);
+        });
+    });
 });

package/test/unit/lib/model/OutboundRequestToPayModel.test.js

@@ -27,7 +27,7 @@ const payeeParty = require('./data/payeeParty');
 const transactionRequestResponseTemplate = require('./data/transactionRequestResponse');
 
 const genPartyId = (party) => {
-    const { partyIdType, partyIdentifier, partySubIdOrType } = party.party.partyIdInfo;
+    const { partyIdType, partyIdentifier, partySubIdOrType } = party.body.party.partyIdInfo;
     return PartiesModel.channelName({
         type: partyIdType,
         id: partyIdentifier,

package/test/unit/lib/model/OutboundRequestToPayTransferModel.test.js

@@ -121,7 +121,7 @@ describe('outboundRequestToPayTransferModel', () => {
         MojaloopRequests.__postTransfers = jest.fn((postTransfersBody, destFspId) => {
             //ensure that the `MojaloopRequests.postTransfers` method has been called with the correct arguments
             // set as the destination FSPID, picked up from the header's value `fspiop-source`
-            expect(model.data.quoteResponseSource).toBe(quoteResponse.headers['fspiop-source']);
+            expect(model.data.quoteResponseSource).toBe(quoteResponse.data.headers['fspiop-source']);
 
             const extensionList = postTransfersBody.extensionList.extension;
             expect(extensionList).toBeTruthy();
@@ -129,8 +129,8 @@ describe('outboundRequestToPayTransferModel', () => {
             expect(extensionList[0]).toEqual({ key: 'tkey1', value: 'tvalue1' });
             expect(extensionList[1]).toEqual({ key: 'tkey2', value: 'tvalue2' });
 
-            expect(destFspId).toBe(quoteResponse.headers['fspiop-source']);
-            expect(quoteResponse.headers['fspiop-source']).not.toBe(model.data.to.fspId);
+            expect(destFspId).toBe(quoteResponse.data.headers['fspiop-source']);
+            expect(quoteResponse.data.headers['fspiop-source']).not.toBe(model.data.to.fspId);
 
             // simulate a callback with the transfer fulfilment
             emitTransferFulfilCacheMessage(cache, postTransfersBody.transferId, transferFulfil);