ml-testing-toolkit 18.16.0-snapshot.6 → 18.16.0

package/.ncurc.yaml

@@ -6,6 +6,5 @@ reject: [
   "jest",
   "@types/jest",
   "uuid",
-  "axios"
 ]
 

package/CHANGELOG.md

@@ -1,4 +1,11 @@
 # Changelog: [mojaloop/thirdparty-api-svc](https://github.com/mojaloop/thirdparty-api-svc)
+## [18.16.0](https://github.com/mojaloop/ml-testing-toolkit/compare/v18.15.1...v18.16.0) (2025-09-16)
+
+
+### Bug Fixes
+
+* **csi-1782:** fix MyEmitter memory leak ([#342](https://github.com/mojaloop/ml-testing-toolkit/issues/342)) ([c141048](https://github.com/mojaloop/ml-testing-toolkit/commit/c1410487a3c17824b0869b6920c2132d491a6007))
+
 ### [18.15.1](https://github.com/mojaloop/ml-testing-toolkit/compare/v18.15.0...v18.15.1) (2025-09-01)
 
 

package/audit-ci.jsonc

@@ -3,6 +3,5 @@
   // audit-ci supports reading JSON, JSONC, and JSON5 config files.
   // Only use one of ["low": true, "moderate": true, "high": true, "critical": true]
   "moderate": true,
-  "allowlist": [
-  ]
+  "allowlist": []
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ml-testing-toolkit",
   "description": "Testing Toolkit for Mojaloop implementations",
-  "version": "18.16.0-snapshot.6",
+  "version": "18.16.0",
   "license": "Apache-2.0",
   "author": "Vijaya Kumar Guthi, ModusBox Inc. ",
   "contributors": [
@@ -40,7 +40,7 @@
   ],
   "main": "src/index.js",
   "scripts": {
-    "start": "node --trace-warnings src/index.js",
+    "start": "node src/index.js",
     "standard": "standard",
     "standard:fix": "standard --fix",
     "dev": "nodemon src/index.js",
@@ -80,15 +80,15 @@
     "@hapi/vision": "7.0.3",
     "@mojaloop/central-services-logger": "11.9.3",
     "@mojaloop/central-services-metrics": "12.7.1",
-    "@mojaloop/ml-schema-transformer-lib": "2.7.7",
+    "@mojaloop/ml-schema-transformer-lib": "2.7.8",
     "@mojaloop/ml-testing-toolkit-shared-lib": "14.2.0",
-    "@mojaloop/sdk-standard-components": "19.16.7",
+    "@mojaloop/sdk-standard-components": "19.17.0",
     "@now-ims/hapi-now-auth": "2.1.0",
     "@types/socket.io": "3.0.2",
     "adm-zip": "0.5.16",
     "ajv-formats": "3.0.1",
     "atob": "2.1.2",
-    "axios": "1.12.0",
+    "axios": "1.12.2",
     "chai": "4.4.1",
     "connection-string": "^5.0.0",
     "cookie-parser": "1.4.7",
@@ -194,12 +194,6 @@
     "yargs-parser": "21.1.1",
     "oas-validator": {
       "ajv": "6.12.3"
-    },
-    "@mojaloop/central-services-shared": {
-      "axios": "1.12.0"
-    },
-    "@mojaloop/sdk-standard-components": {
-      "axios": "1.12.0"
     }
   }
 }

package/src/lib/eventListenerClient/inboundEventListener.js

@@ -27,9 +27,6 @@
  --------------
  ******/
 
-// Ignore temporarily to get the snapshot release done
-/* istanbul ignore file */
-
 const MyEventEmitter = require('../MyEventEmitter')
 const EventEmitter = require('events')
 class MyEmitter extends EventEmitter {}
@@ -46,17 +43,17 @@ class InboundEventListener {
       this.consoleFn = console
     }
     this.emitter = MyEventEmitter.getEmitter('inboundRequest')
-    this.newInboundHandler = null
   }
 
   async init () {
     this.userConfig = await Config.getStoredUserConfig()
 
-    if (this.newInboundHandler) {
-      this.emitter.removeListener('newInbound', this.newInboundHandler)
+    // Fixes the MyEmitter listener leak issue
+    if (this.emitter.listenerCount('newInbound') > 0) {
+      this.emitter.removeAllListeners('newInbound')
     }
 
-    this.newInboundHandler = (data) => {
+    this.emitter.on('newInbound', (data) => {
       for (const [, eventListener] of Object.entries(this.eventListeners)) {
         // Match method, path and condition for each inbound request
         if (eventListener.method === data.method && eventListener.path === data.path) {
@@ -70,9 +67,7 @@ class InboundEventListener {
           }
         }
       }
-    }
-
-    this.emitter.on('newInbound', this.newInboundHandler)
+    })
   }
 
   customLog (logMessage) {
@@ -195,20 +190,6 @@ class InboundEventListener {
     }
     return true
   }
-
-  // Add a new cleanup method
-  cleanup () {
-    // Remove the newInbound listener
-    if (this.newInboundHandler) {
-      this.emitter.removeListener('newInbound', this.newInboundHandler)
-      this.newInboundHandler = null
-    }
-
-    // Clean up all event listeners
-    Object.keys(this.eventListeners).forEach(clientName => {
-      this.destroy(clientName)
-    })
-  }
 }
 
 module.exports = {

package/src/lib/scripting-engines/vm-javascript-sandbox.js

@@ -265,9 +265,6 @@ const executeAsync = async (script, data, contextObj) => {
     }
     consoleLog.push([{ execution: 0 }, 'executionError', err.toString()])
   }
-  // finally {
-  //   contextObj.inboundEvent.cleanup()
-  // }
   const result = {
     consoleLog,
     environment: { ...contextObj.environment }

package/src/lib/webSocketClient/WebSocketClientManager.js

@@ -28,9 +28,6 @@
  --------------
  ******/
 
-// Ignore temporarily to get the snapshot release done
-/* istanbul ignore file */
-
 const WebSocket = require('ws')
 const EventEmitter = require('events')
 class MyEmitter extends EventEmitter {}
@@ -56,6 +53,8 @@ class WebSocketClientManager {
   }
 
   connect (url, clientName, timeout = 15000) {
+    this.customLog(`Connecting to WebSocket URL: ${url} with client name: ${clientName} and timeout: ${timeout}ms`)
+
     const tlsOptions = { rejectUnauthorized: false }
     const urlObject = new URL(url)
     if (this.userConfig.CLIENT_MUTUAL_TLS_ENABLED) {
@@ -165,7 +164,10 @@ class WebSocketClientManager {
 
   disconnect (clientName) {
     if (this.ws[clientName]) {
-      this.ws[clientName].client.close()
+      if (this.ws[clientName].client) {
+        this.ws[clientName].client.removeAllListeners()
+        this.ws[clientName].client.close()
+      }
       if (this.ws[clientName].eventEmitter) {
         this.ws[clientName].eventEmitter.removeAllListeners()
       }
@@ -180,7 +182,10 @@ class WebSocketClientManager {
 
   disconnectAll () {
     for (const clientName in this.ws) {
-      this.ws[clientName].client.close()
+      if (this.ws[clientName].client) {
+        this.ws[clientName].client.removeAllListeners()
+        this.ws[clientName].client.close()
+      }
       if (this.ws[clientName].eventEmitter) {
         this.ws[clientName].eventEmitter.removeAllListeners()
       }
@@ -193,13 +198,6 @@ class WebSocketClientManager {
     }
     return true
   }
-
-  clearAllTimers () {
-    Object.keys(this.timers).forEach((clientName) => {
-      clearTimeout(this.timers[clientName])
-      delete this.timers[clientName]
-    })
-  }
 }
 
 module.exports = {