Skip to content

Commit

Permalink
test: skip some pummel tests on slower machines
Browse files Browse the repository at this point in the history
Skipped the longest-running pummel tests on the Raspberry Pi devices in
CI.

Refs: #34289 (comment)

PR-URL: #38394
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
  • Loading branch information
Trott authored and targos committed Jun 11, 2021
1 parent 490bc58 commit 296b969
Show file tree
Hide file tree
Showing 6 changed files with 41 additions and 5 deletions.
9 changes: 8 additions & 1 deletion test/pummel/test-crypto-dh-hash-modp18.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,15 @@

'use strict';
const common = require('../common');
if (!common.hasCrypto)

if (!common.hasCrypto) {
common.skip('node compiled without OpenSSL.');
}

if ((process.config.variables.arm_version === '6') ||
(process.config.variables.arm_version === '7')) {
common.skip('Too slow for armv6 and armv7 bots');
}

const assert = require('assert');
const crypto = require('crypto');
Expand Down
9 changes: 8 additions & 1 deletion test/pummel/test-crypto-dh-hash.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,15 @@

'use strict';
const common = require('../common');
if (!common.hasCrypto)

if (!common.hasCrypto) {
common.skip('node compiled without OpenSSL.');
}

if ((process.config.variables.arm_version === '6') ||
(process.config.variables.arm_version === '7')) {
common.skip('Too slow for armv6 and armv7 bots');
}

const assert = require('assert');
const crypto = require('crypto');
Expand Down
1 change: 1 addition & 0 deletions test/pummel/test-crypto-dh-keys.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@

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

if (!common.hasCrypto) {
common.skip('node compiled without OpenSSL.');
}
Expand Down
9 changes: 8 additions & 1 deletion test/pummel/test-dh-regr.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,15 @@

'use strict';
const common = require('../common');
if (!common.hasCrypto)

if (!common.hasCrypto) {
common.skip('missing crypto');
}

if ((process.config.variables.arm_version === '6') ||
(process.config.variables.arm_version === '7')) {
common.skip('Too slow for armv6 and armv7 bots');
}

const assert = require('assert');
const crypto = require('crypto');
Expand Down
7 changes: 6 additions & 1 deletion test/pummel/test-next-tick-infinite-calls.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,12 @@
// USE OR OTHER DEALINGS IN THE SOFTWARE.

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

if ((process.config.variables.arm_version === '6') ||
(process.config.variables.arm_version === '7')) {
common.skip('Too slow for armv6 and armv7 bots');
}

let complete = 0;

Expand Down
11 changes: 10 additions & 1 deletion test/pummel/test-policy-integrity.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,16 @@
'use strict';

const common = require('../common');
if (!common.hasCrypto) common.skip('missing crypto');

if (!common.hasCrypto) {
common.skip('missing crypto');
}

if ((process.config.variables.arm_version === '6') ||
(process.config.variables.arm_version === '7')) {
common.skip('Too slow for armv6 and armv7 bots');
}

common.requireNoPackageJSONAbove();

const { debuglog } = require('util');
Expand Down

0 comments on commit 296b969

Please sign in to comment.