Skip to content

build: add pummel tests to ci runs #34289

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Trott merged 5 commits into from
Apr 10, 2021
Merged

build: add pummel tests to ci runs #34289

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
5020891
test: skip test-hash-seed on armv6 and armv7
Trott Apr 4, 2021
213ae4f
test: skip test-vm-memleak in ASAN
Trott Apr 6, 2021
8cee284
test: fix test-dh-regr for OpenSSL 3
Trott Apr 7, 2021
24426cd
build: prepare Windows coverage GitHub Action for pummel tests
Trott Apr 10, 2021
2853b76
build: add pummel tests to ci runs
Trott Jul 10, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: skip test-vm-memleak in ASAN 
ASAN increases memory usage, which in turn messes up the memory leak
test. Skip the test on ASAN.

PR-URL: #34289
Reviewed-By: Richard Lau <rlau@redhat.com>
  • Loading branch information
@Trott
Trott committed Apr 10, 2021
commit 213ae4f4c6ce8bc5907c9c1c6a6eaec59391aaa5
7 changes: 6 additions & 1 deletion test/pummel/test-vm-memleak.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,12 @@
'use strict';
// Flags: --max_old_space_size=32 --expose_gc

require('../common');
const common = require('../common');

if (process.config.variables.asan) {
common.skip('ASAN messes with memory measurements');
}

const assert = require('assert');
const vm = require('vm');

Expand Down