test: skip explicitly

Author: durran

test/integration/transactions-convenient-api/transactions-convenient-api.spec.test.ts

@@ -6,7 +6,9 @@ import { runUnifiedSuite } from '../../tools/unified-spec-runner/runner';
 const SKIPPED_TESTS = [
   'callback succeeds after multiple connection errors',
   'callback is not retried after non-transient error',
-  'callback is not retried after non-transient error (DuplicateKeyError)'
+  'callback is not retried after non-transient error (DuplicateKeyError)',
+  'withTransaction succeeds if callback aborts',
+  'unpin after transient error within a transaction'
 ];
 
 describe('Transactions Convenient API Spec Unified Tests', function () {
@@ -22,7 +24,7 @@ describe('Transactions Convenient API Spec Unified Tests', function () {
 
   runUnifiedSuite(loadSpecTests(path.join('transactions-convenient-api', 'unified')), test => {
     return SKIPPED_TESTS.includes(test.description)
-      ? 'TODO(DRIVERS-5855): Skipping failing transaction tests'
+      ? 'TODO(NODE-5855/DRIVERS-2816): Skipping failing transaction tests'
       : false;
   });
 });

test/integration/unified-test-format/unified_test_format.spec.test.ts

@@ -23,6 +23,17 @@ const filter: TestFilter = ({ description }) => {
     return 'TODO(NODE-3308): failures due unnecessary getMore and killCursors calls in 5.0';
   }
 
+  if (
+    [
+      'withTransaction and no transaction options set',
+      'withTransaction inherits transaction options from client',
+      'withTransaction inherits transaction options from defaultTransactionOptions',
+      'withTransaction explicit transaction options'
+    ].includes(description)
+  ) {
+    return 'TODO(DRIVERS-2816): fix migration conflict in transaction tests';
+  }
+
   if (
     process.env.AUTH === 'auth' &&
     [