Nodejs/node
1
0
Fork 0
mirror of https://github.com/nodejs/node.git synced 2024-11-21 10:59:27 +00:00
Code Issues Actions 25 Packages Projects Releases Wiki Activity

test: skip memory usage tests when ASAN is enabled

Browse Source 
Running tests with an ASAN build leads to increased memory usage,
rendering the memory usage assumptions in the test invalid.

Refs: https://github.com/nodejs/node/pull/32776#issuecomment-620407014

PR-URL: https://github.com/nodejs/node/pull/33129
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matheus Marchini <mat@mmarchini.me>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
Anna Henningsen 2020-04-28 18:12:43 +02:00
parent 4b2d95804b
commit d5e1795f53
No known key found for this signature in database
GPG Key ID: A94130F0BFC8EBE9
2 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/parallel/test-crypto-dh-leak.js
View File

@ -4,6 +4,8 @@
const common = require('../common');
if (!common.hasCrypto)
common.skip('missing crypto');
if (process.config.variables.asan)
common.skip('ASAN messes with memory measurements');
const assert = require('assert');
const crypto = require('crypto');

3
test/sequential/test-net-bytes-per-incoming-chunk-overhead.js
View File

@ -2,6 +2,9 @@
'use strict';
const common = require('../common');
if (process.config.variables.asan)
common.skip('ASAN messes with memory measurements');
const assert = require('assert');
const net = require('net');