Skip to content

Commit d7b13ab

Browse files
addaleaxtargos
authored andcommitted
test: skip memory usage tests when ASAN is enabled
Running tests with an ASAN build leads to increased memory usage, rendering the memory usage assumptions in the test invalid. Refs: #32776 (comment) PR-URL: #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>
1 parent 2383538 commit d7b13ab

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

test/parallel/test-crypto-dh-leak.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
const common = require('../common');
55
if (!common.hasCrypto)
66
common.skip('missing crypto');
7+
if (process.config.variables.asan)
8+
common.skip('ASAN messes with memory measurements');
79

810
const assert = require('assert');
911
const crypto = require('crypto');

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@
22
'use strict';
33

44
const common = require('../common');
5+
if (process.config.variables.asan)
6+
common.skip('ASAN messes with memory measurements');
7+
58
const assert = require('assert');
69
const net = require('net');
710

0 commit comments

Comments
 (0)