node-opcua-leak-detector 2.169.0 → 2.172.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "node-opcua-leak-detector",
-    "version": "2.169.0",
+    "version": "2.172.0",
     "description": "pure nodejs OPCUA SDK - module leak-detector",
     "main": "index.js",
     "types": "index.d.ts",
@@ -24,14 +24,14 @@
         "internet of things"
     ],
     "homepage": "http://node-opcua.github.io/",
-    "gitHead": "82d570d3e95bea689cbbe30096279885c5282245",
+    "gitHead": "dfe9993a93b5c3897825e898b5f07b25952c7f45",
     "files": [
         "src"
     ],
     "dependencies": {
         "chalk": "4.1.2",
         "node-opcua-assert": "2.164.0",
-        "node-opcua-object-registry": "2.169.0",
+        "node-opcua-object-registry": "2.172.0",
         "wtfnode": "^0.10.1"
     }
 }

package/src/mem_leak_detector.js

@@ -16,7 +16,9 @@ let noGCexposed;
  */
 function forceGC() {
     if (global.gc) {
-        global.gc();
+        // Major GC: thorough enough that callers don't need to invoke a separate
+        // global.gc(true) themselves before calling takeMemorySnapshot.
+        global.gc(true);
         noGCexposed = false;
     } else {
         if (noGCexposed === undefined) {

package/src/resource_leak_detector.js

@@ -477,10 +477,8 @@ exports.installResourceLeakDetector = function(isGlobal, func) {
 
         });
         beforeEach(() => {
-            // make sure we start with a garbage collected situation
-            if (global.gc) {
-                global.gc(true);
-            }
+            // takeMemorySnapshot now does its own major GC; no need for an
+            // explicit gc(true) here. Saves one ~100ms major-GC stall per test.
             beforeSnapshot = takeMemorySnapshot();
         });
         if (func) {
@@ -506,10 +504,7 @@ exports.installResourceLeakDetector = function(isGlobal, func) {
 
     } else {
         beforeEach(function(/*this: Mocha.Test*/) {
-
-            if (global.gc) {
-                global.gc(true);
-            }
+            // takeMemorySnapshot now does its own major GC; no separate gc(true) here.
             resourceLeakDetector.ctx = this.test.ctx;
             resourceLeakDetector.start();
             beforeSnapshot = takeMemorySnapshot();