From 13ac5fbc57b9cc101416554e340a6e4e8441d5d6 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 568b37d8f6850f..60b021f69906dd 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) {