wip

Author: nbbeeken

src/operations/execute_operation.ts

@@ -125,7 +125,7 @@ export function executeOperation<
 
 function executeWithServerSelection<TResult>(
   topology: Topology,
-  session: ClientSession | undefined,
+  session: ClientSession,
   operation: AbstractOperation,
   callback: Callback<TResult>
 ) {
@@ -140,12 +140,7 @@ function executeWithServerSelection<TResult>(
     );
   }
 
-  if (
-    session &&
-    session.isPinned &&
-    session.transaction.isCommitted &&
-    !operation.bypassPinningCheck
-  ) {
+  if (session.isPinned && session.transaction.isCommitted && !operation.bypassPinningCheck) {
     session.unpin();
   }
 
@@ -189,7 +184,6 @@ function executeWithServerSelection<TResult>(
 
     if (
       originalError instanceof MongoNetworkError &&
-      session &&
       session.isPinned &&
       !session.inTransaction() &&
       operation.hasAspect(Aspect.CURSOR_CREATING)

test/tools/spec-runner/index.js

@@ -459,19 +459,19 @@ function validateExpectations(commandEvents, spec, savedSessionData) {
   const rawExpectedEvents = spec.expectations.map(x => x.command_started_event);
   const expectedEvents = normalizeCommandShapes(rawExpectedEvents);
 
-  expect(actualEvents.map(({ commandName }) => commandName)).to.deep.equal(
-    expectedEvents.map(({ commandName }) => commandName)
-  );
+  for (const [idx, expectedEvent] of expectedEvents.entries()) {
+    const actualEvent = actualEvents[idx];
 
-  expect(actualEvents.map(({ databaseName }) => databaseName)).to.deep.equal(
-    expectedEvents.map(({ databaseName }) => databaseName)
-  );
+    if (typeof expectedEvent.commandName === 'string') {
+      expect(actualEvent).to.have.property('commandName', expectedEvent.commandName);
+    }
 
-  for (const [idx, expected] of expectedEvents.entries()) {
-    const actual = actualEvents[idx];
+    if (typeof expectedEvent.databaseName === 'string') {
+      expect(actualEvent).to.have.property('databaseName', expectedEvent.databaseName);
+    }
 
-    const actualCommand = actual.command;
-    const expectedCommand = expected.command;
+    const actualCommand = actualEvent.command;
+    const expectedCommand = expectedEvent.command;
     if (expectedCommand.sort) {
       // TODO(NODE-3235): This is a workaround that works because all sorts in the specs
       // are objects with one key; ideally we'd want to adjust the spec definitions