Skip to content

Commit ec7ee61

Browse files
Trotttargos
authored andcommitted
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: #37034 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent 49f1374 commit ec7ee61

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/pummel/test-vm-memleak.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ const interval = setInterval(function() {
3636

3737
const rss = process.memoryUsage().rss;
3838
assert.ok(rss < 64 * 1024 * 1024,
39-
`memory usage: ${Math.round(rss / (1024 * 1024))}Mb`);
39+
`memory usage: ${rss} (${Math.round(rss / (1024 * 1024))} MB)`);
4040

4141
// Stop after 5 seconds.
4242
if (Date.now() - start > 5 * 1000) {

0 commit comments

Comments
 (0)