All non-lb or TLS tests

Author: baileympearson

test/integration/causal-consistency/causal_consistency.prose.test.js

@@ -4,32 +4,22 @@ const { LEGACY_HELLO_COMMAND } = require('../../mongodb');
 
 const { setupDatabase } = require('../shared');
 const { expect } = require('chai');
-const { skipBrokenAuthTestBeforeEachHook } = require('../../tools/runner/hooks/configuration');
 
 const ignoredCommands = [LEGACY_HELLO_COMMAND, 'endSessions'];
 const test = { commands: { started: [], succeeded: [] } };
 
 // TODO(NODE-3882) - properly implement all prose tests and add missing cases 1, 8, 9, 11, 12
-describe('Causal Consistency - prose tests', function () {
+describe.only('Causal Consistency - prose tests', function () {
   before(function () {
     return setupDatabase(this.configuration);
   });
 
-  beforeEach(
-    skipBrokenAuthTestBeforeEachHook({
-      skippedTests: [
-        '2. The first read in a causally consistent session must not send afterClusterTime to the server',
-        'case: successful read with causal consistency',
-        'case: second operation is findOne',
-        'case: successful insert',
-        '6. A read operation in a ClientSession that is not causally consistent should not include the afterClusterTime parameter in the command sent to the server'
-      ]
-    })
-  );
-
   beforeEach(function () {
     test.commands = { started: [], succeeded: [] };
-    test.client = this.configuration.newClient({ w: 1 }, { maxPoolSize: 1, monitorCommands: true });
+    test.client = this.configuration.newClient(
+      { w: 1 },
+      { maxPoolSize: 1, monitorCommands: true, __skipPingOnConnect: true }
+    );
     test.client.on('commandStarted', event => {
       if (ignoredCommands.indexOf(event.commandName) === -1) test.commands.started.push(event);
     });

test/integration/connection-monitoring-and-pooling/connection.test.ts

@@ -21,7 +21,6 @@ import {
   Topology
 } from '../../mongodb';
 import * as mock from '../../tools/mongodb-mock/index';
-import { skipBrokenAuthTestBeforeEachHook } from '../../tools/runner/hooks/configuration';
 import { processTick, sleep } from '../../tools/utils';
 import { assert as test, setupDatabase } from '../shared';
 
@@ -37,15 +36,6 @@ const commonConnectOptions = {
 };
 
 describe('Connection', function () {
-  beforeEach(
-    skipBrokenAuthTestBeforeEachHook({
-      skippedTests: [
-        'should support calling back multiple times on exhaust commands',
-        'should correctly connect to server using domain socket'
-      ]
-    })
-  );
-
   before(function () {
     return setupDatabase(this.configuration);
   });
@@ -187,7 +177,10 @@ describe('Connection', function () {
         const configuration = this.configuration;
         client = configuration.newClient(
           `mongodb://${encodeURIComponent('/tmp/mongodb-27017.sock')}?w=1`,
-          { maxPoolSize: 1 }
+          {
+            maxPoolSize: 1,
+            auth: { ...this.configuration.options.auth }
+          }
         );
 
         const db = client.db(configuration.db);

test/integration/node-specific/operation_examples.test.ts

@@ -1,5 +1,4 @@
 import { expect } from 'chai';
-import { format as f } from 'util';
 
 import {
   Code,
@@ -8,7 +7,6 @@ import {
   ProfilingLevel,
   ReturnDocument
 } from '../../mongodb';
-import { skipBrokenAuthTestBeforeEachHook } from '../../tools/runner/hooks/configuration';
 import { sleep as delay } from '../../tools/utils';
 import { setupDatabase } from '../shared';
 
@@ -27,16 +25,6 @@ describe('Operations', function () {
     return setupDatabase(this.configuration, ['integration_tests_2', 'hr', 'reporting']);
   });
 
-  beforeEach(
-    skipBrokenAuthTestBeforeEachHook({
-      skippedTests: [
-        'Should correctly connect to a replicaset',
-        'Should connect to mongos proxies using connectiong string With Promises',
-        'Should correctly connect to a replicaset With Promises'
-      ]
-    })
-  );
-
   /**************************************************************************
    *
    * COLLECTION TESTS
@@ -3244,14 +3232,7 @@ describe('Operations', function () {
 
     test: function () {
       const configuration = this.configuration;
-      const url = f(
-        'mongodb://%s,%s/%s?replicaSet=%s&readPreference=%s',
-        f('%s:%s', configuration.host, configuration.port),
-        f('%s:%s', configuration.host, configuration.port + 1),
-        'integration_test_',
-        configuration.replicasetName,
-        'primary'
-      );
+      const url = configuration.url();
 
       const client = configuration.newClient(url);
       return client.connect().then(function (client) {
@@ -3291,13 +3272,7 @@ describe('Operations', function () {
 
     test: function () {
       const configuration = this.configuration;
-      const url = f(
-        'mongodb://%s:%s,%s:%s/sharded_test_db?w=1',
-        configuration.host,
-        configuration.port,
-        configuration.host,
-        configuration.port + 1
-      );
+      const url = configuration.url();
 
       const client = configuration.newClient(url);
       return client.connect().then(function (client) {

test/tools/runner/hooks/configuration.ts

@@ -107,22 +107,6 @@ const testSkipBeforeEachHook = async function () {
   }
 };
 
-/**
- * TODO: NODE-3891 - fix tests that are broken with auth enabled and remove this hook
- * @param skippedTests - define list of tests to skip
- * @returns
- */
-export const skipBrokenAuthTestBeforeEachHook = function (
-  { skippedTests }: { skippedTests: string[] } = { skippedTests: [] }
-) {
-  return function () {
-    if (process.env.AUTH === 'auth' && skippedTests.includes(this.currentTest.title)) {
-      this.currentTest.skipReason = 'TODO: NODE-3891 - fix tests broken when AUTH enabled';
-      this.skip();
-    }
-  };
-};
-
 const testConfigBeforeHook = async function () {
   if (process.env.DRIVERS_ATLAS_TESTING_URI) {
     this.configuration = new AstrolabeTestConfiguration(process.env.DRIVERS_ATLAS_TESTING_URI, {});