trace_events: refactor to use validateStringArray

`options.categories` is string[]. So used `validateStringArray` Refs: https://nodejs.org/dist/latest-v19.x/docs/api/tracing.html#trace_eventscreatetracingoptions PR-URL: #46012 Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Jan 31, 2023 · 0defe4e · 0defe4e
1 parent 297773c
commit 0defe4e
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/lib/trace_events.js b/lib/trace_events.js
@@ -1,7 +1,6 @@
 'use strict';
 
 const {
-  ArrayIsArray,
   ArrayPrototypeJoin,
   SafeSet,
   Symbol,
@@ -17,7 +16,6 @@ const kMaxTracingCount = 10;
 const {
   ERR_TRACE_EVENTS_CATEGORY_REQUIRED,
   ERR_TRACE_EVENTS_UNAVAILABLE,
-  ERR_INVALID_ARG_TYPE
 } = require('internal/errors').codes;
 
 const { ownsProcessState } = require('internal/worker');
@@ -29,6 +27,7 @@ const { customInspectSymbol } = require('internal/util');
 const { format } = require('internal/util/inspect');
 const {
   validateObject,
+  validateStringArray,
 } = require('internal/validators');
 
 const enabledTracingObjects = new SafeSet();
@@ -84,11 +83,7 @@ class Tracing {
 
 function createTracing(options) {
   validateObject(options, 'options');
-
-  if (!ArrayIsArray(options.categories)) {
-    throw new ERR_INVALID_ARG_TYPE('options.categories', 'string[]',
-                                   options.categories);
-  }
+  validateStringArray(options.categories, 'options.categories');
 
   if (options.categories.length <= 0)
     throw new ERR_TRACE_EVENTS_CATEGORY_REQUIRED();