From 98d56224922bb13b637e7f13eef6f0088c16551d Mon Sep 17 00:00:00 2001 From: Deokjin Kim Date: Thu, 29 Dec 2022 18:48:59 +0900 Subject: [PATCH] trace_events: refactor to use `validateStringArray` --- lib/trace_events.js | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/trace_events.js b/lib/trace_events.js index 5211f8b0b1fc74..37353d7d16ca94 100644 --- a/lib/trace_events.js +++ b/lib/trace_events.js @@ -17,7 +17,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 +28,7 @@ const { customInspectSymbol } = require('internal/util'); const { format } = require('internal/util/inspect'); const { validateObject, + validateStringArray, } = require('internal/validators'); const enabledTracingObjects = new SafeSet(); @@ -84,11 +84,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();