test: add test coverage for fs.truncate
Add test to check: - for `null` as len parameter - if error is propagated into callback if file doesn't exist - if an error is actually thrown if len is not a number PR-URL: https://github.com/nodejs/node/pull/23620 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Denys Otrishko <shishugi@gmail.com>
This commit is contained in:
parent
1a2cf6696f
commit
74f854e769
@ -239,6 +239,41 @@ function testFtruncate(cb) {
|
|||||||
}));
|
}));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
{
|
||||||
|
const file7 = path.resolve(tmp, 'truncate-file-7.txt');
|
||||||
|
fs.writeFileSync(file7, 'Hi');
|
||||||
|
fs.truncate(file7, undefined, common.mustCall(function(err) {
|
||||||
|
assert.ifError(err);
|
||||||
|
assert(fs.readFileSync(file7).equals(Buffer.from('')));
|
||||||
|
}));
|
||||||
|
}
|
||||||
|
|
||||||
|
{
|
||||||
|
const file8 = path.resolve(tmp, 'non-existent-truncate-file.txt');
|
||||||
|
const validateError = (err) => {
|
||||||
|
assert.strictEqual(file8, err.path);
|
||||||
|
assert.strictEqual(
|
||||||
|
err.message,
|
||||||
|
`ENOENT: no such file or directory, open '${file8}'`);
|
||||||
|
assert.strictEqual(err.code, 'ENOENT');
|
||||||
|
assert.strictEqual(err.syscall, 'open');
|
||||||
|
return true;
|
||||||
|
};
|
||||||
|
fs.truncate(file8, 0, common.mustCall(validateError));
|
||||||
|
}
|
||||||
|
|
||||||
|
['', false, null, {}, []].forEach((input) => {
|
||||||
|
assert.throws(
|
||||||
|
() => fs.truncate('/foo/bar', input),
|
||||||
|
{
|
||||||
|
code: 'ERR_INVALID_ARG_TYPE',
|
||||||
|
name: 'TypeError [ERR_INVALID_ARG_TYPE]',
|
||||||
|
message: 'The "len" argument must be of type number. ' +
|
||||||
|
`Received type ${typeof input}`
|
||||||
|
}
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
['', false, null, undefined, {}, []].forEach((input) => {
|
['', false, null, undefined, {}, []].forEach((input) => {
|
||||||
['ftruncate', 'ftruncateSync'].forEach((fnName) => {
|
['ftruncate', 'ftruncateSync'].forEach((fnName) => {
|
||||||
assert.throws(
|
assert.throws(
|
||||||
|
Loading…
x
Reference in New Issue
Block a user