From 3bac969655db25ec68f9bfefb0284dd5fa3dcbc1 Mon Sep 17 00:00:00 2001
From: mawaregetsuka <33221990+mawaregetsuka@users.noreply.github.com>
Date: Sun, 3 Apr 2022 06:57:44 +0800
Subject: [PATCH] lib: improve the coverage of the validator

PR-URL: https://github.com/nodejs/node/pull/42443
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 lib/internal/validators.js       | 12 ++++++------
 test/parallel/test-validators.js | 30 ++++++++++++++++++++++++++++++
 2 files changed, 36 insertions(+), 6 deletions(-)

diff --git a/lib/internal/validators.js b/lib/internal/validators.js
index 9fed5b363db9cb..514db01071bfaf 100644
--- a/lib/internal/validators.js
+++ b/lib/internal/validators.js
@@ -83,10 +83,10 @@ const validateInteger = hideStackFrames(
 const validateInt32 = hideStackFrames(
   (value, name, min = -2147483648, max = 2147483647) => {
     // The defaults for min and max correspond to the limits of 32-bit integers.
+    if (typeof value !== 'number') {
+      throw new ERR_INVALID_ARG_TYPE(name, 'number', value);
+    }
     if (!isInt32(value)) {
-      if (typeof value !== 'number') {
-        throw new ERR_INVALID_ARG_TYPE(name, 'number', value);
-      }
       if (!NumberIsInteger(value)) {
         throw new ERR_OUT_OF_RANGE(name, 'an integer', value);
       }
@@ -99,10 +99,10 @@ const validateInt32 = hideStackFrames(
 );
 
 const validateUint32 = hideStackFrames((value, name, positive) => {
+  if (typeof value !== 'number') {
+    throw new ERR_INVALID_ARG_TYPE(name, 'number', value);
+  }
   if (!isUint32(value)) {
-    if (typeof value !== 'number') {
-      throw new ERR_INVALID_ARG_TYPE(name, 'number', value);
-    }
     if (!NumberIsInteger(value)) {
       throw new ERR_OUT_OF_RANGE(name, 'an integer', value);
     }
diff --git a/test/parallel/test-validators.js b/test/parallel/test-validators.js
index 6b0d49c6997a65..0bba9d13b20bf0 100644
--- a/test/parallel/test-validators.js
+++ b/test/parallel/test-validators.js
@@ -10,6 +10,8 @@ const {
   validateNumber,
   validateObject,
   validateString,
+  validateInt32,
+  validateUint32,
 } = require('internal/validators');
 const { MAX_SAFE_INTEGER, MIN_SAFE_INTEGER } = Number;
 const outOfRangeError = {
@@ -41,6 +43,34 @@ const invalidArgValueError = {
   // validateInteger() works with unsafe integers.
   validateInteger(MAX_SAFE_INTEGER + 1, 'foo', 0, MAX_SAFE_INTEGER + 1);
   validateInteger(MIN_SAFE_INTEGER - 1, 'foo', MIN_SAFE_INTEGER - 1);
+
+  // validateInt32() and validateUint32()
+  [
+    Symbol(), 1n, {}, [], false, true, undefined, null, () => {}, '', '1',
+  ].forEach((val) => assert.throws(() => validateInt32(val, 'name'), {
+    code: 'ERR_INVALID_ARG_TYPE'
+  }));
+  [
+    2147483647 + 1, -2147483648 - 1, NaN,
+  ].forEach((val) => assert.throws(() => validateInt32(val, 'name'), {
+    code: 'ERR_OUT_OF_RANGE'
+  }));
+  [
+    0, 1, -1,
+  ].forEach((val) => validateInt32(val, 'name'));
+  [
+    Symbol(), 1n, {}, [], false, true, undefined, null, () => {}, '', '1',
+  ].forEach((val) => assert.throws(() => validateUint32(val, 'name'), {
+    code: 'ERR_INVALID_ARG_TYPE'
+  }));
+  [
+    4294967296, -1, NaN,
+  ].forEach((val) => assert.throws(() => validateUint32(val, 'name'), {
+    code: 'ERR_OUT_OF_RANGE'
+  }));
+  [
+    0, 1,
+  ].forEach((val) => validateUint32(val, 'name'));
 }
 
 {