1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fs: complete error message for validate function

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/19909
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
This commit is contained in:
buji 2018-04-10 15:31:35 +08:00 committed by Ruben Bridgewater
parent 50e9f8df62
commit 61e232b9b8
No known key found for this signature in database
GPG Key ID: F07496B3EB3C1762
2 changed files with 30 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/internal/fs.js
View File

@ -350,11 +350,13 @@ function validateLen(len) {
let err; let err;
if (!isInt32(len)) { if (!isInt32(len)) {
if (typeof value !== 'number') { if (typeof len !== 'number') {
err = new ERR_INVALID_ARG_TYPE('len', 'number', len); err = new ERR_INVALID_ARG_TYPE('len', 'number', len);
} else { } else if (!Number.isInteger(len)) {
// TODO(BridgeAR): Improve this error message.
err = new ERR_OUT_OF_RANGE('len', 'an integer', len); err = new ERR_OUT_OF_RANGE('len', 'an integer', len);
} else {
// 2 ** 31 === 2147483648
err = new ERR_OUT_OF_RANGE('len', '> -2147483649 && < 2147483648', len);
} }
} }

25
test/parallel/test-fs-truncate.js
View File

@ -190,6 +190,31 @@ function testFtruncate(cb) {
); );
}); });
[-1.5, 1.5].forEach((input) => {
assert.throws(
() => fs.ftruncate(fd, input),
{
code: 'ERR_OUT_OF_RANGE',
name: 'RangeError [ERR_OUT_OF_RANGE]',
message: 'The value of "len" is out of range. It must be ' +
`an integer. Received ${input}`
}
);
});
// 2 ** 31 = 2147483648
[2147483648, -2147483649].forEach((input) => {
assert.throws(
() => fs.ftruncate(fd, input),
{
code: 'ERR_OUT_OF_RANGE',
name: 'RangeError [ERR_OUT_OF_RANGE]',
message: 'The value of "len" is out of range. It must be ' +
`> -2147483649 && < 2147483648. Received ${input}`
}
);
});
fs.ftruncate(fd, undefined, common.mustCall(function(err) { fs.ftruncate(fd, undefined, common.mustCall(function(err) {
assert.ifError(err); assert.ifError(err);
assert(fs.readFileSync(file5).equals(Buffer.from(''))); assert(fs.readFileSync(file5).equals(Buffer.from('')));