fix touse validateAbortSignal to check signal

Author: shisama

lib/internal/event_target.js

@@ -34,7 +34,7 @@ const {
  ERR_INVALID_THIS,
  }
 } = require('internal/errors');
-const { validateObject, validateString, validateInternalField } = require('internal/validators');
+const { validateAbortSignal, validateObject, validateString, validateInternalField } = require('internal/validators');
 
 const {
  customInspectSymbol,
@@ -575,9 +575,7 @@ class EventTarget {
  }
  type = String(type);
 
- if (signal === null) {
- throw new ERR_INVALID_ARG_TYPE('options.signal', 'AbortSignal', signal);
- }
+ validateAbortSignal(signal, 'options.signal');
 
  if (signal) {
  if (signal.aborted) {

test/parallel/test-whatwg-events-add-event-listener-options-signal.js

@@ -160,7 +160,9 @@ const {
 }
 {
  const et = new EventTarget();
- throws(() => et.addEventListener('foo', () => {}, { signal: null }), {
- name: 'TypeError',
+ [1, {}, null, false, 'hi'].forEach((signal) => {
+ throws(() => et.addEventListener('foo', () => {}, { signal }), {
+ name: 'TypeError',
+ });
  });
 }