test: increase test coverage for fs/promises.js

PR-URL: #19811
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

Author: humphd

test/parallel/test-fs-promises.js

@@ -87,24 +87,60 @@ function verifyStatObject(stat) {
 stats=awaitstat(dest);
 verifyStatObject(stats);
 
+stats=awaithandle.stat();
+verifyStatObject(stats);
+
 awaitfdatasync(handle);
+awaithandle.datasync();
 awaitfsync(handle);
+awaithandle.sync();
 
-constbuf=Buffer.from('hello world');
-
+constbuf=Buffer.from('hello fsPromises');
+constbufLen=buf.length;
 awaitwrite(handle,buf);
-
-constret=awaitread(handle,Buffer.alloc(11),0,11,0);
-assert.strictEqual(ret.bytesRead,11);
+constret=awaitread(handle,Buffer.alloc(bufLen),0,bufLen,0);
+assert.strictEqual(ret.bytesRead,bufLen);
 assert.deepStrictEqual(ret.buffer,buf);
 
+constbuf2=Buffer.from('hello FileHandle');
+constbuf2Len=buf2.length;
+awaithandle.write(buf2,0,buf2Len,0);
+constret2=awaithandle.read(Buffer.alloc(buf2Len),0,buf2Len,0);
+assert.strictEqual(ret2.bytesRead,buf2Len);
+assert.deepStrictEqual(ret2.buffer,buf2);
+
 awaitchmod(dest,0o666);
 awaitfchmod(handle,0o666);
+awaithandle.chmod(0o666);
+
+// `mode` can't be > 0o777
+assert.rejects(
+async()=>chmod(handle,(0o777+1)),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+name: 'TypeError [ERR_INVALID_ARG_TYPE]'
+}
+);
+assert.rejects(
+async()=>fchmod(handle,(0o777+1)),
+{
+code: 'ERR_OUT_OF_RANGE',
+name: 'RangeError [ERR_OUT_OF_RANGE]'
+}
+);
+assert.rejects(
+async()=>handle.chmod(handle,(0o777+1)),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+name: 'TypeError [ERR_INVALID_ARG_TYPE]'
+}
+);
 
 awaitutimes(dest,newDate(),newDate());
 
 try{
 awaitfutimes(handle,newDate(),newDate());
+awaithandle.utimes(newDate(),newDate());
 }catch(err){
 // Some systems do not have futimes. If there is an error,
 // expect it to be ENOSYS
@@ -152,6 +188,15 @@ function verifyStatObject(stat) {
 awaitrmdir(newdir);
 
 awaitmkdtemp(path.resolve(tmpDir,'FOO'));
+assert.rejects(
+// mkdtemp() expects to get a string prefix.
+async()=>mkdtemp(1),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+name: 'TypeError [ERR_INVALID_ARG_TYPE]'
+}
+);
+
 }
 
 doTest().then(common.mustCall());