ml-testing-toolkit 18.17.0-snapshot.8 → 18.17.0

package/CHANGELOG.md

@@ -1,4 +1,16 @@
 # Changelog: [mojaloop/thirdparty-api-svc](https://github.com/mojaloop/thirdparty-api-svc)
+## [18.17.0](https://github.com/mojaloop/ml-testing-toolkit/compare/v18.16.6...v18.17.0) (2026-01-26)
+
+
+### Features
+
+* add transfer store for timeout interscheme get test ([#362](https://github.com/mojaloop/ml-testing-toolkit/issues/362)) ([48cb7c6](https://github.com/mojaloop/ml-testing-toolkit/commit/48cb7c6bbf1feb1289cb42f15b490cf06b71f68a))
+
+
+### Bug Fixes
+
+* remove unsupported mongodb options ([#361](https://github.com/mojaloop/ml-testing-toolkit/issues/361)) ([7f7b7bb](https://github.com/mojaloop/ml-testing-toolkit/commit/7f7b7bb12dfd28e21cb7b8d2375df4bf11ff0618))
+
 ### [18.16.6](https://github.com/mojaloop/ml-testing-toolkit/compare/v18.16.5...v18.16.6) (2026-01-08)
 
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ml-testing-toolkit",
   "description": "Testing Toolkit for Mojaloop implementations",
-  "version": "18.17.0-snapshot.8",
+  "version": "18.17.0",
   "license": "Apache-2.0",
   "author": "Vijaya Kumar Guthi, ModusBox Inc. ",
   "contributors": [

package/src/lib/mocking/middleware-functions/ilpModel.js

@@ -30,6 +30,7 @@
 const { ilpFactory, ILP_VERSIONS } = require('@mojaloop/sdk-standard-components').Ilp
 const customLogger = require('../../requestLogger')
 const { logger } = require('../../logger')
+const objectStore = require('../../objectStore')
 
 let ilpObj = null
 let ilpV4Obj = null
@@ -143,27 +144,24 @@ const handleTransferIlp = (context, response) => {
 
   customLogger.logMessage('debug', 'Generated callback body', { additionalData: { context, response } })
   if (context.request.method === 'get' && response.method === 'put' && pathMatch.test(response.path)) {
-    customLogger.logMessage('debug', 'Returning stored fulfilment if exists for transfer GET')
     const transferId = response.path.match(pathMatch)[1]
-    customLogger.logMessage('debug', 'Fetching stored transfer for fulfilment', { additionalData: { transferId } })
-    const storedTransfer = context.storedTransfers?.[transferId]
-    customLogger.logMessage('debug', 'Stored transfer fetched for fulfilment', { additionalData: { storedTransfer } })
-    // Check if stored request exists and is within 30 seconds
+    // Use objectStore to get the stored transfer
+    const storedTransfer = objectStore.get('storedTransfers', transferId)
+    customLogger.logMessage('debug', 'Stored transfer fetched for fulfilment', { additionalData: { transferId } })
     if (storedTransfer) {
-      if (storedTransfer.request.ilpPacket) {
-        customLogger.logMessage('debug', 'Stored transfer has ilpPacket. Generating fulfilment.', { additionalData: { transferId, ilpPacket: storedTransfer.request.ilpPacket } })
-        const generatedFulfilment = ilpObj.calculateFulfil(storedTransfer.request.ilpPacket).replace('"', '')
+      if (storedTransfer.data?.request?.ilpPacket) {
+        customLogger.logMessage('debug', 'Stored transfer has ilpPacket. Generating fulfilment.', { additionalData: { transferId, ilpPacket: storedTransfer.data.request.ilpPacket } })
+        const generatedFulfilment = ilpObj.calculateFulfil(storedTransfer.data.request.ilpPacket).replace('"', '')
         response.body.fulfilment = generatedFulfilment
-        customLogger.logMessage('debug', 'Fulfilment set in response body', { additionalData: { transferId, fulfilment: generatedFulfilment } })
-      } else if (storedTransfer.request.CdtTrfTxInf?.VrfctnOfTerms?.IlpV4PrepPacket) {
-        customLogger.logMessage('debug', 'Stored transfer has IlpV4PrepPacket. Generating fulfilment.', { additionalData: { transferId, IlpV4PrepPacket: storedTransfer.request.CdtTrfTxInf.VrfctnOfTerms.IlpV4PrepPacket } })
-        const generatedFulfilment = ilpV4Obj.calculateFulfil(storedTransfer.request.CdtTrfTxInf.VrfctnOfTerms.IlpV4PrepPacket).replace('"', '')
+      } else if (storedTransfer.data?.request?.CdtTrfTxInf?.VrfctnOfTerms?.IlpV4PrepPacket) {
+        customLogger.logMessage('debug', 'Stored transfer has IlpV4PrepPacket. Generating fulfilment.', { additionalData: { transferId } })
+        const generatedFulfilment = ilpV4Obj.calculateFulfil(storedTransfer.data.request.CdtTrfTxInf.VrfctnOfTerms.IlpV4PrepPacket).replace('"', '')
         response.body.TxInfAndSts.ExctnConf = generatedFulfilment
-        customLogger.logMessage('debug', 'ExctnConf set in response body', { additionalData: { transferId, ExctnConf: generatedFulfilment } })
       } else {
-        customLogger.logMessage('warn', 'No ILP packet or IlpV4PrepPacket found in stored transfer request', { additionalData: { transferId, storedTransfer } })
+        customLogger.logMessage('warn', 'No ILP packet or IlpV4PrepPacket found in stored transfer request', { additionalData: { transferId } })
       }
-      delete context.storedTransfers[transferId]
+      // Remove the stored transfer from objectStore regardless of success or failure
+      objectStore.deleteObject('storedTransfers', transferId)
     } else {
       delete response.body.fulfilment
     }

package/src/lib/mocking/openApiMockHandler.js

@@ -323,9 +323,6 @@ const generateAsyncCallback = async (item, context, req) => {
     const fxTransferPathMatch = /\/fxTransfers(?:\/([^/]+))?$/
     customLogger.logMessage('debug', 'Processing transfer request', { additionalData: { method: req.method, path: req.path }, request: req })
     if (req.method === 'post' && (transferPathMatch.test(req.path) || fxTransferPathMatch.test(req.path))) {
-      if (!context.storedTransfers) {
-        context.storedTransfers = {}
-      }
       const isFxTransfer = fxTransferPathMatch.test(req.path)
       let transferId = (req.payload && req.payload.transferId) || (req.payload && req.payload.commitRequestId)
       if (!transferId && req.payload && req.payload.CdtTrfTxInf && req.payload.CdtTrfTxInf.PmtId && req.payload.CdtTrfTxInf.PmtId.TxId) {
@@ -333,9 +330,17 @@ const generateAsyncCallback = async (item, context, req) => {
       }
       customLogger.logMessage('debug', 'Storing transfer for validation', { additionalData: { transferId }, request: req })
 
-      context.storedTransfers[transferId] = {
-        request: req.payload,
-        type: isFxTransfer ? 'fxTransfer' : 'transfer'
+      // Use objectStore to store the transfer only when a valid transferId is resolved
+      if (transferId) {
+        objectStore.push('storedTransfers', transferId, {
+          request: req.payload,
+          type: isFxTransfer ? 'fxTransfer' : 'transfer'
+        })
+      } else {
+        customLogger.logMessage('warn', 'Skipping storing transfer: transferId could not be resolved from payload', {
+          additionalData: { method: req.method, path: req.path },
+          request: req
+        })
       }
     }
 

package/src/lib/objectStore.js

@@ -29,13 +29,15 @@
  ******/
 
 const _ = require('lodash')
+const { logger } = require('./logger')
 
 const storedObject = {
   data: {
     transactions: {},
     inboundEnvironment: {},
     requests: {},
-    callbacks: {}
+    callbacks: {},
+    storedTransfers: {}
   }
 }
 
@@ -76,13 +78,17 @@ const push = (key, item, value, user) => {
 }
 
 const clear = (object, interval) => {
-  for (const context in storedObject) {
-    for (const item in storedObject[context][object]) {
-      const timeDiff = Date.now() - storedObject[context][object][item].insertedDate
-      if (timeDiff > interval) {
-        delete storedObject[context][object][item]
+  try {
+    for (const context in storedObject) {
+      for (const item in storedObject[context][object]) {
+        const timeDiff = Date.now() - storedObject[context][object][item].insertedDate
+        if (timeDiff > interval) {
+          delete storedObject[context][object][item]
+        }
       }
     }
+  } catch (err) {
+    logger.error('Error clearing old objects', err)
   }
 }
 
@@ -96,6 +102,15 @@ const popObject = (key, item, user) => {
   return null
 }
 
+const deleteObject = (key, item, user) => {
+  const context = init(key, user)
+  if (Object.prototype.hasOwnProperty.call(storedObject[context][key], item)) {
+    delete storedObject[context][key][item]
+    return true
+  }
+  return false
+}
+
 const clearOldObjects = () => {
   const interval = 10 * 60 * 1000
   clear('transactions', interval)
@@ -103,6 +118,7 @@ const clearOldObjects = () => {
   clear('callbacks', interval)
   clear('requestsHistory', interval)
   clear('callbacksHistory', interval)
+  clear('storedTransfers', interval)
 }
 
 const initObjectStore = (initConfig = null) => {
@@ -116,7 +132,9 @@ module.exports = {
   set,
   get,
   initObjectStore,
+  init,
   push,
   clear,
-  popObject
+  popObject,
+  deleteObject
 }