Skip to content

Commit e33f2b2

Browse files
committed
test(NODE-3688): remove extra cmap event from ping
1 parent fecb104 commit e33f2b2

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

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

-11
Original file line numberDiff line numberDiff line change
@@ -163,14 +163,8 @@ export async function runUnifiedTest(
163163
}
164164
}
165165

166-
let testPing = false;
167166
for (const operation of test.operations) {
168167
trace(operation.name);
169-
// TODO: NODE-2149: Making connect optional should get rid of the initial ping in
170-
// the driver so this block can then be removed.
171-
if (operation.name === 'runCommand' && operation.arguments.commandName === 'ping') {
172-
testPing = true;
173-
}
174168
try {
175169
await executeOperationAndCheck(operation, entities, utilClient);
176170
} catch (e) {
@@ -198,11 +192,6 @@ export async function runUnifiedTest(
198192
const actualEvents =
199193
eventType === 'cmap' ? clientCmapEvents.get(clientId) : clientCommandEvents.get(clientId);
200194

201-
// TODO: NODE-2149: Making connect optional should get rid of the initial ping in
202-
// the driver so this block can then be removed.
203-
if (eventType === 'cmap' && testPing) {
204-
expectedEventList.events.push({ connectionCheckOutStartedEvent: {} });
205-
}
206195
expect(actualEvents, `No client entity found with id ${clientId}`).to.exist;
207196
matchesEvents(expectedEventList.events, actualEvents, entities);
208197
}

0 commit comments

Comments
 (0)