Commit ec7ee61
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
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -36,7 +36,7 @@ const interval = setInterval(function() { | |||
| 36 | 36 | ||
| 37 | 37 | const rss = process.memoryUsage().rss; | |
| 38 | 38 | 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)`); | ||
| 40 | 40 | ||
| 41 | 41 | // Stop after 5 seconds. | |
| 42 | 42 | if (Date.now() - start > 5 * 1000) { | |
0 commit comments