test: temporarily skip sharded transactions tests

Author: mbroadst

test/functional/spec-runner/index.js

@@ -123,14 +123,16 @@ function generateTopologyTests(testSuites, testContext, filter) {
           afterEach(() => testContext.cleanupAfterSuite());
 
           testSuite.tests.forEach(spec => {
-            const maybeSkipIt =
-              spec.skipReason || (filter && typeof filter === 'function' && !filter(spec))
-                ? it.skip
-                : it;
+            it(spec.description, function() {
+              if (
+                spec.skipReason ||
+                (filter && typeof filter === 'function' && !filter(spec, this.configuration))
+              ) {
+                this.skip();
+                return;
+              }
 
-            maybeSkipIt(spec.description, function() {
               let testPromise = Promise.resolve();
-
               if (spec.failPoint) {
                 testPromise = testPromise.then(() => testContext.enableFailPoint(spec.failPoint));
               }

test/functional/transactions.test.js

@@ -8,6 +8,7 @@ const TestRunnerContext = require('./spec-runner').TestRunnerContext;
 const loadSpecTests = require('../spec').loadSpecTests;
 const generateTopologyTests = require('./spec-runner').generateTopologyTests;
 const MongoNetworkError = require('../../lib/core').MongoNetworkError;
+const semver = require('semver');
 
 describe('Transactions', function() {
   const testContext = new TestRunnerContext();
@@ -26,7 +27,12 @@ describe('Transactions', function() {
         return testContext.setup(this.configuration);
       });
 
-      function testFilter(spec) {
+      function testFilter(spec, config) {
+        // NODE-2574: remove this when HELP-15010 is resolved
+        if (config.topologyType === 'Sharded' && semver.satisfies(config.version, '>=4.4')) {
+          return false;
+        }
+
         const SKIP_TESTS = [
           // commitTransaction retry seems to be swallowed by mongos in these three cases
           'commitTransaction retry succeeds on new mongos',

test/tools/runner/config.js

@@ -10,6 +10,7 @@ const core = require('../../../lib/core');
 class NativeConfiguration {
   constructor(parsedURI, context) {
     this.topologyType = context.topologyType;
+    this.version = context.version;
     this.clientSideEncryption = context.clientSideEncryption;
     this.options = Object.assign(
       {