From ec7ee61af00f4c9abc221fc4f5a93b6c71bbaeae Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sat, 23 Jan 2021 03:55:38 -0800
Subject: [PATCH] test: improve assertion message for test-vm-memleak

The test is unreliable in CI. It might be helpful to get an exact read
of `rss` so this adds that to the assertion message.

PR-URL: https://github.com/nodejs/node/pull/37034
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
---
 test/pummel/test-vm-memleak.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/pummel/test-vm-memleak.js b/test/pummel/test-vm-memleak.js
index a433ab6565e147..d9e2c753565792 100644
--- a/test/pummel/test-vm-memleak.js
+++ b/test/pummel/test-vm-memleak.js
@@ -36,7 +36,7 @@ const interval = setInterval(function() {
 
   const rss = process.memoryUsage().rss;
   assert.ok(rss < 64 * 1024 * 1024,
-            `memory usage: ${Math.round(rss / (1024 * 1024))}Mb`);
+            `memory usage: ${rss} (${Math.round(rss / (1024 * 1024))} MB)`);
 
   // Stop after 5 seconds.
   if (Date.now() - start > 5 * 1000) {