test: use global.EventTarget instead of internals

`EventTarget` is exposed on the global scope, there is no need to use
`--expose-internals` flag in the tests.

Refs: https://github.com/nodejs/node/pull/35496

PR-URL: https://github.com/nodejs/node/pull/36002
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
This commit is contained in:
Antoine du Hamel committed 2020-11-09 19:25:15 +01:00
1 parent b589128f6f
commit 36fbbe0b86
5 files changed
+3 -22

No files matched your search

+1 -2
View File
@@ -1,10 +1,9 @@
// Flags: --no-warnings --expose-internals
// Flags: --no-warnings
'use strict';
const common = require('../common');
const { ok, strictEqual } = require('assert');
const { Event } = require('internal/event_target');
{
// Tests that abort is fired with the correct event type on AbortControllers
@@ -1,10 +1,5 @@
// Flags: --expose-internals --no-warnings
'use strict';
const common = require('../common');
const {
Event,
EventTarget,
} = require('internal/event_target');
const { once } = require('events');
const et = new EventTarget();
@@ -1,13 +1,7 @@
// Flags: --expose-internals --no-warnings
'use strict';
const common = require('../common');
const {
Event,
EventTarget,
} = require('internal/event_target');
const {
strictEqual,
} = require('assert');
+1 -5
View File
@@ -2,11 +2,7 @@
'use strict';
const common = require('../common');
const {
Event,
EventTarget,
defineEventHandler
} = require('internal/event_target');
const { defineEventHandler } = require('internal/event_target');
const {
ok,
+1 -4
View File
@@ -2,10 +2,7 @@
'use strict';
const common = require('../common');
const {
Event,
NodeEventTarget,
} = require('internal/event_target');
const { NodeEventTarget } = require('internal/event_target');
const {
deepStrictEqual,