Commit d48e00e
test: fix test-memory-usage.js for IBMi
Newly added process.memoryUsage.rss() will presumably return 0 on IBMi
the same way process.memoryUsage().rss does. Allow IBMi to skip the new
assertion.
The test was using a mix of `assert()` and `assert.ok()`. This change
makes it consistently use `assert.ok()`.
PR-URL: #36758
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>1 parent b2c1aeb commit d48e00eCopy full SHA for d48e00e
File tree
Expand file treeCollapse file tree
1 file changed
+6
-5
lines changedOpen diff view settings
Filter options
- test/parallel
Expand file treeCollapse file tree
1 file changed
+6
-5
lines changedOpen diff view settings
Collapse file
test/parallel/test-memory-usage.js
Copy file name to clipboardExpand all lines: test/parallel/test-memory-usage.js+6-5Lines changed: 6 additions & 5 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
26 | 26 | |
27 | 27 | |
28 | 28 | |
29 | | - |
| 29 | + |
30 | 30 | |
| 31 | + |
| 32 | + |
| 33 | + |
31 | 34 | |
32 | 35 | |
33 | 36 | |
| ||
39 | 42 | |
40 | 43 | |
41 | 44 | |
42 | | - |
43 | | - |
| 45 | + |
| 46 | + |
44 | 47 | |
45 | 48 | |
46 | 49 | |
47 | | - |
48 | | - |
|
0 commit comments