test(NODE-3711): unskip transaction retry spec tests

mongodb · Nov 18, 2021 · 0620ec0 · 0620ec0
1 parent ac44599
commit 0620ec0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/src/sessions.ts b/src/sessions.ts
@@ -8,6 +8,7 @@ import {
   MongoError,
   MongoInvalidArgumentError,
   isRetryableError,
+  isRetryableWriteError,
   MongoCompatibilityError,
   MongoNetworkError,
   MongoWriteConcernError,
@@ -767,7 +768,11 @@ function endTransaction(session: ClientSession, commandName: string, callback: C
         session.unpin();
       }
 
-      if (err && isRetryableError(err as MongoError)) {
+      /* eslint no-console: 0 */
+      console.log('\n\n\nerr', err);
+      if (err && isRetryableWriteError(err as MongoError)) {
+        /* eslint no-console: 0 */
+        console.log('retrying');
         // SPEC-1185: apply majority write concern when retrying commitTransaction
         if (command.commitTransaction) {
           // per txns spec, must unpin session in this case

diff --git a/test/functional/transactions.test.js b/test/functional/transactions.test.js
@@ -111,16 +111,10 @@ const SKIP_TESTS = [
 
   // Will be implemented as part of NODE-2034
   'Client side error in command starting transaction',
-  'Client side error when transaction is in progress',
-
-  // Will be implemented as part of NODE-2538
-  'abortTransaction only retries once with RetryableWriteError from server',
-  'abortTransaction does not retry without RetryableWriteError label',
-  'commitTransaction does not retry error without RetryableWriteError label',
-  'commitTransaction retries once with RetryableWriteError from server'
+  'Client side error when transaction is in progress'
 ];
 
-describe('Transactions Spec Legacy Tests', function () {
+describe.only('Transactions Spec Legacy Tests', function () {
   const testContext = new TransactionsRunnerContext();
   const suitesToRun = [{ name: 'spec tests', specPath: path.join('transactions', 'legacy') }];
   // Note: convenient-api tests are skipped for serverless