lib: use Array#includes instead of Array#indexOf

PR-URL: https://github.com/nodejs/node/pull/26732
Refs: https://github.com/nodejs/node/issues/26568
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
index 4cee2d8b9f..dc97e6dceb 100644
--- a/lib/internal/util/inspect.js
+++ b/lib/internal/util/inspect.js
@@ -288,12 +288,12 @@ function strEscape(str) {
   // instead wrap the text in double quotes. If double quotes exist, check for
   // backticks. If they do not exist, use those as fallback instead of the
   // double quotes.
-  if (str.indexOf("'") !== -1) {
+  if (str.includes("'")) {
     // This invalidates the charCode and therefore can not be matched for
     // anymore.
-    if (str.indexOf('"') === -1) {
+    if (!str.includes('"')) {
       singleQuote = -1;
-    } else if (str.indexOf('`') === -1 && str.indexOf('${') === -1) {
+    } else if (!str.includes('`') && !str.includes('${')) {
       singleQuote = -2;
     }
     if (singleQuote !== 39) {
@@ -557,7 +557,7 @@ function formatValue(ctx, value, recurseTimes, typedArray) {
 
   // Using an array here is actually better for the average case than using
   // a Set. `seen` will only check for the depth and will never grow too large.
-  if (ctx.seen.indexOf(value) !== -1)
+  if (ctx.seen.includes(value))
     return ctx.stylize('[Circular]', 'special');
 
   return formatRaw(ctx, value, recurseTimes, typedArray);