FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: skip instead of fail when mem constrained · nodejs/node@9d16729 · GitHub

/ node Public

Commit 9d16729

Browse files
mcornac
authored and
rvagg
committed
test: skip instead of fail when mem constrained
The current implementation of tests for strings with length at or exceeding kStringMaxLength allocate a temporary buffer inside a try block to skip the test if there is insufficient memory. This commit adds an invocation of the garbage collector after the temporary buffer is allocated so that memory is freed for later allocations. Change the corresponding catch block to rethrow the original exception instead of asserting the exception message to provide more information about the exception. Add an additional check before trying to allocate memory to immediately skip the test on machines with insufficient total memory. PR-URL: #3697 Reviewed-By: Trevor Norris <trev.norris@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Michaël Zasso <mic.besace@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
1 parent d969c09 commit 9d16729

9 files changed

Lines changed: 140 additions & 58 deletions

‎test/common.js‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ exports.isLinuxPPCBE = (process.platform === 'linux') &&
2222
exports.isSunOS = process.platform === 'sunos';
2323
exports.isFreeBSD = process.platform === 'freebsd';
2424

25+
exports.enoughTestMem = os.totalmem() > 0x20000000; /* 512MB */
26+
2527
function rimrafSync(p) {
2628
try {
2729
var st = fs.lstatSync(p);

‎test/parallel/test-stringbytes-external-at-max.js‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,32 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

67
// v8 fails silently if string length > v8::String::kMaxLength
78
// v8::String::kMaxLength defined in v8.h
89
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
910

11+
const skipMessage =
12+
'1..0 # Skipped: intensive toString tests due to memory confinements';
13+
if (!common.enoughTestMem) {
14+
console.log(skipMessage);
15+
return;
16+
}
17+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
18+
1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf = new Buffer(kStringMaxLength);
20-
2131
const maxString = buf.toString('binary');
2232
assert.equal(maxString.length, kStringMaxLength);

‎test/parallel/test-stringbytes-external-exceed-max-by-1-ascii.js‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,33 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength + 1);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf = new Buffer(kStringMaxLength + 1);
20-
2131
assert.throws(function() {
2232
buf.toString('ascii');
2333
}, /toString failed/);

‎test/parallel/test-stringbytes-external-exceed-max-by-1-base64.js‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,33 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength + 1);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf = new Buffer(kStringMaxLength + 1);
20-
2131
assert.throws(function() {
2232
buf.toString('base64');
2333
}, /toString failed/);

‎test/parallel/test-stringbytes-external-exceed-max-by-1-binary.js‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,33 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength + 1);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf = new Buffer(kStringMaxLength + 1);
20-
2131
assert.throws(function() {
2232
buf.toString('binary');
2333
}, /toString failed/);

‎test/parallel/test-stringbytes-external-exceed-max-by-1-hex.js‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,33 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength + 1);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf = new Buffer(kStringMaxLength + 1);
20-
2131
assert.throws(function() {
2232
buf.toString('hex');
2333
}, /toString failed/);

‎test/parallel/test-stringbytes-external-exceed-max-by-1-utf8.js‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,33 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength + 1);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf = new Buffer(kStringMaxLength + 1);
20-
2131
assert.throws(function() {
2232
buf.toString();
2333
}, /toString failed|Invalid array buffer length/);

‎test/parallel/test-stringbytes-external-exceed-max-by-2.js‎

Lines changed: 18 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,32 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength + 2);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf2 = new Buffer(kStringMaxLength + 2);
20-
21-
const maxString = buf2.toString('utf16le');
31+
const maxString = buf.toString('utf16le');
2232
assert.equal(maxString.length, (kStringMaxLength + 2) / 2);

‎test/parallel/test-stringbytes-external-exceed-max.js‎

Lines changed: 18 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,33 @@
11
'use strict';
2+
// Flags: --expose-gc
23

3-
require('../common');
4+
const common = require('../common');
45
const assert = require('assert');
56

7+
const skipMessage =
8+
'1..0 # Skipped: intensive toString tests due to memory confinements';
9+
if (!common.enoughTestMem) {
10+
console.log(skipMessage);
11+
return;
12+
}
13+
assert(typeof gc === 'function', 'Run this test with --expose-gc');
14+
615
// v8 fails silently if string length > v8::String::kMaxLength
716
// v8::String::kMaxLength defined in v8.h
817
const kStringMaxLength = process.binding('buffer').kStringMaxLength;
918

1019
try {
11-
new Buffer(kStringMaxLength * 3);
20+
var buf = new Buffer(kStringMaxLength * 2 + 2);
21+
// Try to allocate memory first then force gc so future allocations succeed.
22+
new Buffer(2 * kStringMaxLength);
23+
gc();
1224
} catch(e) {
13-
assert.equal(e.message, 'Invalid array buffer length');
14-
console.log(
15-
'1..0 # Skipped: intensive toString tests due to memory confinements');
25+
// If the exception is not due to memory confinement then rethrow it.
26+
if (e.message !== 'Invalid array buffer length') throw (e);
27+
console.log(skipMessage);
1628
return;
1729
}
1830

19-
const buf0 = new Buffer(kStringMaxLength * 2 + 2);
20-
2131
assert.throws(function() {
22-
buf0.toString('utf16le');
32+
buf.toString('utf16le');
2333
}, /toString failed/);

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL