From d03a58bf10f7fde5e13add30cd7f6747541cc95b Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 3 Apr 2021 08:50:52 -0700 Subject: [PATCH] test: fix test-vm-memleak for high baseline platforms test-vm-memleak always fails on AIX in CI because the test checks that total memory consumption is less than 64 Mb, but AIX uses over 100 Mb just as a baseline. So instead, let's compare the memory consumption to the baseline and make sure it is within 32 Mb. --- test/pummel/test-vm-memleak.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/test/pummel/test-vm-memleak.js b/test/pummel/test-vm-memleak.js index 752488e3efc0ed..22aa9f3ead54f6 100644 --- a/test/pummel/test-vm-memleak.js +++ b/test/pummel/test-vm-memleak.js @@ -26,6 +26,8 @@ require('../common'); const assert = require('assert'); const vm = require('vm'); +const baselineRss = process.memoryUsage.rss(); + const start = Date.now(); const interval = setInterval(function() { @@ -36,8 +38,8 @@ const interval = setInterval(function() { global.gc(); const rss = process.memoryUsage.rss(); - assert.ok(rss < 64 * 1024 * 1024, - `memory usage: ${rss} (${Math.round(rss / (1024 * 1024))} MB)`); + assert.ok(rss < baselineRss + 32 * 1024 * 1024, + `memory usage: ${rss} baseline: ${baselineRss}`); // Stop after 5 seconds. if (Date.now() - start > 5 * 1000) {