Skip to content

Commit 93060c5

Browse files
committed
test: remove duplicate event listeners and check known events
1 parent 8ec6dcb commit 93060c5

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

test/tools/unified-spec-runner/entities.ts

+9-8
Original file line numberDiff line numberDiff line change
@@ -119,15 +119,16 @@ export class UnifiedMongoClient extends MongoClient {
119119
'configureFailPoint'
120120
];
121121

122-
this.observedEvents = (description.observeEvents ?? [])
123-
.map(e => UnifiedMongoClient.EVENT_NAME_LOOKUP[e])
124-
.filter(e => !!e);
125-
this.observedEvents = (description.observeEvents ?? [])
126-
.map(e => UnifiedMongoClient.EVENT_NAME_LOOKUP[e])
127-
.filter(e => !!e);
128-
for (const eventName of this.observedEvents) {
129-
this.on(eventName, this.pushEvent);
122+
this.observedEvents = [];
123+
124+
for (const e of description.observeEvents ?? []) {
125+
const eventToObserve = UnifiedMongoClient.EVENT_NAME_LOOKUP[e];
126+
if (!eventToObserve) {
127+
throw new Error(`Unknown expected event: ${e}`);
128+
}
129+
this.observedEvents.push(eventToObserve);
130130
}
131+
131132
for (const eventName of this.observedEvents) {
132133
this.on(eventName, this.pushEvent);
133134
}

0 commit comments

Comments
 (0)