test(NODE-3713): skip/fix failing evergreen tests (#3069)

mongodb · Dec 17, 2021 · 8a3bab7 · 8a3bab7
1 parent 55d1c5f
commit 8a3bab7
Show file tree

Hide file tree

Showing 11 changed files with 27 additions and 14 deletions.
diff --git a/.evergreen/config.yml b/.evergreen/config.yml
@@ -118,7 +118,7 @@ functions:
       type: test
       params:
         working_dir: src
-        timeout_secs: 60
+        timeout_secs: 300
         script: |
           ${PREPARE_SHELL}
 
@@ -186,7 +186,7 @@ functions:
       type: test
       params:
         working_dir: src
-        timeout_secs: 60
+        timeout_secs: 300
         script: |
           ${PREPARE_SHELL}
 

diff --git a/.evergreen/config.yml.in b/.evergreen/config.yml.in
@@ -138,7 +138,7 @@ functions:
       type: test
       params:
         working_dir: "src"
-        timeout_secs: 60
+        timeout_secs: 300
         script: |
           ${PREPARE_SHELL}
 
@@ -210,7 +210,7 @@ functions:
       type: test
       params:
         working_dir: src
-        timeout_secs: 60
+        timeout_secs: 300
         script: |
           ${PREPARE_SHELL}
 

diff --git a/test/functional/connections_stepdown.test.js b/test/functional/connections_stepdown.test.js
@@ -26,7 +26,9 @@ function expectPoolWasNotCleared(initialCount) {
   return count => expect(count).to.equal(initialCount);
 }
 
-describe('Connections survive primary step down', function () {
+// TODO: NODE-3819: Unskip flaky MacOS tests.
+const maybeDescribe = process.platform === 'darwin' ? describe.skip : describe;
+maybeDescribe('Connections survive primary step down', function () {
   let client;
   let checkClient;
   let db;

diff --git a/test/functional/cursorstream.test.js b/test/functional/cursorstream.test.js
@@ -207,7 +207,9 @@ describe('Cursor Streams', function () {
     }
   });
 
-  it('should stream documents across getMore command and count correctly', {
+  // TODO: NODE-3819: Unskip flaky MacOS tests.
+  const maybeIt = process.platform === 'darwin' ? it.skip : it;
+  maybeIt('should stream documents across getMore command and count correctly', {
     metadata: {
       requires: { topology: ['single', 'replicaset', 'sharded', 'ssl', 'heap', 'wiredtiger'] }
     },

diff --git a/test/functional/examples/change_streams.test.js b/test/functional/examples/change_streams.test.js
@@ -4,7 +4,9 @@
 const setupDatabase = require('../shared').setupDatabase;
 const expect = require('chai').expect;
 
-describe('examples(change-stream):', function () {
+// TODO: NODE-3819: Unskip flaky MacOS/Windows tests.
+const maybeDescribe = process.platform !== 'linux' ? describe.skip : describe;
+maybeDescribe('examples(change-stream):', function () {
   let client;
   let db;
 

diff --git a/test/functional/find.test.js b/test/functional/find.test.js
@@ -1655,7 +1655,8 @@ describe('Find', function () {
     }
   });
 
-  it('shouldCorrectlyIterateOverCollection', {
+  // TODO: NODE-3819: Unskip flaky tests.
+  it.skip('shouldCorrectlyIterateOverCollection', {
     metadata: {
       requires: { topology: ['single', 'replicaset', 'sharded', 'ssl', 'heap', 'wiredtiger'] }
     },

diff --git a/test/functional/unit-sdam/monitoring.test.js b/test/functional/unit-sdam/monitoring.test.js
@@ -24,7 +24,8 @@ describe('monitoring', function () {
     return mock.createServer().then(server => (mockServer = server));
   });
 
-  it('should record roundTripTime', function (done) {
+  // TODO: NODE-3819: Unskip flaky tests.
+  it.skip('should record roundTripTime', function (done) {
     mockServer.setMessageHandler(request => {
       const doc = request.document;
       if (isHello(doc)) {

diff --git a/test/integration/change-streams/change_stream.test.js b/test/integration/change-streams/change_stream.test.js
@@ -509,7 +509,9 @@ describe('Change Streams', function () {
     }
   });
 
-  it('should cache the change stream resume token using promises', {
+  // TODO: NODE-3819: Unskip flaky MacOS tests.
+  const maybeIt = process.platform === 'darwin' ? it.skip : it;
+  maybeIt('should cache the change stream resume token using promises', {
     metadata: { requires: { topology: 'replicaset', mongodb: '>=3.6' } },
     test: function () {
       const configuration = this.configuration;

diff --git a/test/integration/change-streams/change_streams.spec.test.js b/test/integration/change-streams/change_streams.spec.test.js
@@ -12,7 +12,9 @@ describe('Change Streams Spec - Unified', function () {
   runUnifiedSuite(loadSpecTests(path.join('change-streams', 'unified')));
 });
 
-describe('Change Stream Spec - v1', function () {
+// TODO: NODE-3819: Unskip flaky MacOS tests.
+const maybeDescribe = process.platform === 'darwin' ? describe.skip : describe;
+maybeDescribe('Change Stream Spec - v1', function () {
   let globalClient;
   let ctx;
   let events;

diff --git a/test/integration/read-write-concern/write_concern.test.js b/test/integration/read-write-concern/write_concern.test.js
@@ -82,7 +82,8 @@ describe('Write Concern', function () {
 
     after(() => mock.cleanup());
 
-    it('should pipe writeConcern from client down to API call', function () {
+    // TODO: NODE-3816
+    it.skip('should pipe writeConcern from client down to API call', function () {
       server.setMessageHandler(request => {
         if (request.document && request.document[LEGACY_HELLO_COMMAND]) {
           return request.reply(mock.HELLO);

diff --git a/test/manual/kerberos.test.js b/test/manual/kerberos.test.js
@@ -72,7 +72,7 @@ describe('Kerberos', function () {
       client.connect(function (err) {
         expect(err).to.exist;
         expect(err.message).to.match(
-          /(Error from KDC: LOOKING_UP_SERVER)|(not found in Kerberos database)/
+          /(Error from KDC: LOOKING_UP_SERVER)|(not found in Kerberos database)|(UNKNOWN_SERVER)/
         );
         done();
       });
@@ -85,7 +85,7 @@ describe('Kerberos', function () {
       client.connect(function (err) {
         expect(err).to.exist;
         expect(err.message).to.match(
-          /(Error from KDC: LOOKING_UP_SERVER)|(not found in Kerberos database)/
+          /(Error from KDC: LOOKING_UP_SERVER)|(not found in Kerberos database)|(UNKNOWN_SERVER)/
         );
         done();
       });