test(NODE-6834): fix flaky x509 test (#4498)

Author: durran

.evergreen/config.yml

@@ -1345,7 +1345,7 @@ tasks:
         params:
           updates:
             - {key: VERSION, value: latest}
-            - {key: TOPOLOGY, value: sharded_cluster}
+            - {key: TOPOLOGY, value: server}
             - {key: AUTH, value: noauth}
             - {key: SSL, value: ssl}
       - func: install dependencies

.evergreen/generate_evergreen_tasks.js

@@ -120,7 +120,7 @@ BASE_TASKS.push({
   commands: [
     updateExpansions({
       VERSION: 'latest',
-      TOPOLOGY: 'sharded_cluster',
+      TOPOLOGY: 'server',
       AUTH: 'noauth',
       SSL: 'ssl'
     }),

test/manual/x509_auth.test.ts

@@ -54,22 +54,6 @@ describe('x509 Authentication', function () {
     });
   });
 
-  context(
-    'when a valid cert is provided but the certificate does not correspond to a user',
-    function () {
-      it.skip('fails to authenticate', async function () {
-        client = new MongoClient(connectionString.toString(), validOptions);
-        const error = await client.connect().then(
-          () => null,
-          e => e
-        );
-
-        expect(error).to.be.instanceOf(MongoServerError);
-        expect(error.codeName).to.match(/UserNotFound/i);
-      }).skipReason = 'TODO(NODE-6834): fix flaky test';
-    }
-  );
-
   context('when the client connects with an invalid certificate', function () {
     // unlike other authentication mechanisms, x509 authentication 1) requires TLS and
     // 2) the server uses the client certificate to derive a username to authenticate with
@@ -93,6 +77,19 @@ describe('x509 Authentication', function () {
       expect(error).to.be.instanceOf(MongoServerSelectionError);
     });
   });
+
+  context(
+    'when a valid cert is provided but the certificate does not correspond to a user',
+    function () {
+      it('fails to authenticate', async function () {
+        client = new MongoClient(connectionString.toString(), validOptions);
+        const error = await client.connect().catch(error => error);
+
+        expect(error).to.be.instanceOf(MongoServerError);
+        expect(error.codeName).to.match(/UserNotFound/i);
+      });
+    }
+  );
 });
 
 async function createX509User() {
@@ -110,8 +107,7 @@ async function createX509User() {
       roles: [
         { role: 'readWrite', db: 'test' },
         { role: 'userAdminAnyDatabase', db: 'admin' }
-      ],
-      writeConcern: { w: 'majority', wtimeout: 5000 }
+      ]
     });
   } finally {
     await utilClient.close();
@@ -128,8 +124,7 @@ async function dropX509User() {
   try {
     await utilClient.connect();
     await utilClient.db('$external').command({
-      dropUser: process.env.SUBJECT,
-      writeConcern: { w: 'majority', wtimeout: 5000 }
+      dropUser: process.env.SUBJECT
     });
   } finally {
     await utilClient.close();