passing tests

Author: baileympearson

src/cmap/connection.ts

@@ -582,7 +582,7 @@ export class Connection extends TypedEventEmitter<ConnectionEvents> {
         this.throwIfAborted();
       }
     } catch (error) {
-      if (options.session != null) {
+      if (options.session != null && !(error instanceof MongoServerError)) {
         updateSessionFromResponse(options.session, MongoDBResponse.empty);
       }
       if (this.shouldEmitAndLogCommand) {

src/operations/execute_operation.ts

@@ -259,12 +259,12 @@ async function executeOperationWithRetries<
     new RetryContext(willRetry ? (timeoutContext.csotEnabled() ? Infinity : 2) : 1);
   for (
     let attempt = 0;
-    attempt <= retryContext.maxAttempts;
+    attempt < retryContext.maxAttempts;
     attempt++,
       retryContext.maxAttempts = previousOperationError?.hasErrorLabel(
         MongoErrorLabel.SystemOverloadedError
       )
-        ? 5
+        ? 6
         : retryContext.maxAttempts
   ) {
     if (previousOperationError) {

test/integration/client-backpressure/client-backpressure.spec.test.ts

@@ -11,6 +11,6 @@ function shouldSkip({ description }: Test) {
   return skippedTests[description] ?? false;
 }
 
-describe.only('Client Backpressure (spec)', function () {
+describe('Client Backpressure (spec)', function () {
   runUnifiedSuite(loadSpecTests('client-backpressure'), shouldSkip);
 });

test/integration/transactions/transactions.spec.test.ts

@@ -13,7 +13,7 @@ const SKIPPED_TESTS = [
   'client bulkWrite with writeConcern in a transaction causes a transaction error'
 ];
 
-describe.only('Transactions Spec Unified Tests', function () {
+describe('Transactions Spec Unified Tests', function () {
   runUnifiedSuite(loadSpecTests(path.join('transactions', 'unified')), test => {
     return SKIPPED_TESTS.includes(test.description)
       ? 'TODO(NODE-5924): Skipping failing transaction tests'