quic: fixup test-fs-chown

PR-URL: nodejs#360 Reviewed-By: Anna Henningsen <[email protected]>
jasnell · Mar 17, 2020 · 8c66e50 · 8c66e50
1 parent 5f0d9ba
commit 8c66e50
Showing 1 changed file with 12 additions and 12 deletions.
diff --git a/lib/fs.js b/lib/fs.js
@@ -1180,26 +1180,26 @@ function chmodSync(path, mode) {
 function lchown(path, uid, gid, callback) {
   callback = makeCallback(callback);
   path = getValidatedPath(path);
-  validateInteger(uid, 'uid', -1, kMaxUserId);
-  validateInteger(gid, 'gid', -1, kMaxUserId);
+  validateInteger(uid, 'uid', { min: -1, max: kMaxUserId });
+  validateInteger(gid, 'gid', { min: -1, max: kMaxUserId });
   const req = new FSReqCallback();
   req.oncomplete = callback;
   binding.lchown(pathModule.toNamespacedPath(path), uid, gid, req);
 }
 
 function lchownSync(path, uid, gid) {
   path = getValidatedPath(path);
-  validateInteger(uid, 'uid', -1, kMaxUserId);
-  validateInteger(gid, 'gid', -1, kMaxUserId);
+  validateInteger(uid, 'uid', { min: -1, max: kMaxUserId });
+  validateInteger(gid, 'gid', { min: -1, max: kMaxUserId });
   const ctx = { path };
   binding.lchown(pathModule.toNamespacedPath(path), uid, gid, undefined, ctx);
   handleErrorFromBinding(ctx);
 }
 
 function fchown(fd, uid, gid, callback) {
   validateInt32(fd, 'fd', 0);
-  validateInteger(uid, 'uid', -1, kMaxUserId);
-  validateInteger(gid, 'gid', -1, kMaxUserId);
+  validateInteger(uid, 'uid', { min: -1, max: kMaxUserId });
+  validateInteger(gid, 'gid', { min: -1, max: kMaxUserId });
 
   const req = new FSReqCallback();
   req.oncomplete = makeCallback(callback);
@@ -1208,8 +1208,8 @@ function fchown(fd, uid, gid, callback) {
 
 function fchownSync(fd, uid, gid) {
   validateInt32(fd, 'fd', 0);
-  validateInteger(uid, 'uid', -1, kMaxUserId);
-  validateInteger(gid, 'gid', -1, kMaxUserId);
+  validateInteger(uid, 'uid', { min: -1, max: kMaxUserId });
+  validateInteger(gid, 'gid', { min: -1, max: kMaxUserId });
 
   const ctx = {};
   binding.fchown(fd, uid, gid, undefined, ctx);
@@ -1219,8 +1219,8 @@ function fchownSync(fd, uid, gid) {
 function chown(path, uid, gid, callback) {
   callback = makeCallback(callback);
   path = getValidatedPath(path);
-  validateInteger(uid, 'uid', -1, kMaxUserId);
-  validateInteger(gid, 'gid', -1, kMaxUserId);
+  validateInteger(uid, 'uid', { min: -1, max: kMaxUserId });
+  validateInteger(gid, 'gid', { min: -1, max: kMaxUserId });
 
   const req = new FSReqCallback();
   req.oncomplete = callback;
@@ -1229,8 +1229,8 @@ function chown(path, uid, gid, callback) {
 
 function chownSync(path, uid, gid) {
   path = getValidatedPath(path);
-  validateInteger(uid, 'uid', -1, kMaxUserId);
-  validateInteger(gid, 'gid', -1, kMaxUserId);
+  validateInteger(uid, 'uid', { min: -1, max: kMaxUserId });
+  validateInteger(gid, 'gid', { min: -1, max: kMaxUserId });
   const ctx = { path };
   binding.chown(pathModule.toNamespacedPath(path), uid, gid, undefined, ctx);
   handleErrorFromBinding(ctx);