test: increase RAM requirement for intensive tests
test-fs-read-buffer-tostring-fail and test-fs-readfile-tostring-fail have been timing out on Raspberry Pi 3 devices on the continuous integration server. These devices have 1 Gb of RAM and the tests are memory intensive. Previous checks for memory intensive tests used a 512 Mb cut-off, but that was probably instituted when we only had Pi 1 devices. Consequently, this change increases the threshold for memory-intensive tests to 1 Gb and adds that threshold to test-fs-readfile-tostring-fail. PR-URL: https://github.com/nodejs/node/pull/7772 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
This commit is contained in:
parent
74f0943284
commit
135a863f80
@ -27,7 +27,7 @@ exports.isLinuxPPCBE = (process.platform === 'linux') &&
|
|||||||
exports.isSunOS = process.platform === 'sunos';
|
exports.isSunOS = process.platform === 'sunos';
|
||||||
exports.isFreeBSD = process.platform === 'freebsd';
|
exports.isFreeBSD = process.platform === 'freebsd';
|
||||||
|
|
||||||
exports.enoughTestMem = os.totalmem() > 0x20000000; /* 512MB */
|
exports.enoughTestMem = os.totalmem() > 0x40000000; /* 1 Gb */
|
||||||
exports.rootDir = exports.isWindows ? 'c:\\' : '/';
|
exports.rootDir = exports.isWindows ? 'c:\\' : '/';
|
||||||
|
|
||||||
function rimrafSync(p) {
|
function rimrafSync(p) {
|
||||||
|
@ -1,6 +1,13 @@
|
|||||||
'use strict';
|
'use strict';
|
||||||
|
|
||||||
const common = require('../common');
|
const common = require('../common');
|
||||||
|
|
||||||
|
if (!common.enoughTestMem) {
|
||||||
|
const skipMessage = 'intensive toString tests due to memory confinements';
|
||||||
|
common.skip(skipMessage);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
const assert = require('assert');
|
const assert = require('assert');
|
||||||
const fs = require('fs');
|
const fs = require('fs');
|
||||||
const path = require('path');
|
const path = require('path');
|
||||||
|
Loading…
x
Reference in New Issue
Block a user