test(NODE-3713): adding todo comments for skipped tests

Author: durran

test/functional/connections_stepdown.test.js

@@ -26,6 +26,7 @@ function expectPoolWasNotCleared(initialCount) {
   return count => expect(count).to.equal(initialCount);
 }
 
+// TODO: NODE-3819: Unskip flaky MacOS tests.
 describe.skip('Connections survive primary step down', function () {
   let client;
   let checkClient;

test/functional/cursorstream.test.js

@@ -207,7 +207,7 @@ describe('Cursor Streams', function () {
     }
   });
 
-  // Fails on MacOS on Evergreen (NODE-3713)
+  // TODO: NODE-3819: Unskip flaky MacOS tests.
   it.skip('should stream documents across getMore command and count correctly', {
     metadata: {
       requires: { topology: ['single', 'replicaset', 'sharded', 'ssl', 'heap', 'wiredtiger'] }

test/functional/examples/change_streams.test.js

@@ -4,7 +4,7 @@
 const setupDatabase = require('../shared').setupDatabase;
 const expect = require('chai').expect;
 
-// Fails on MacOS on Evergreen (NODE-3713)
+// TODO: NODE-3819: Unskip flaky MacOS tests.
 describe.skip('examples(change-stream):', function () {
   let client;
   let db;

test/functional/find.test.js

@@ -1655,6 +1655,7 @@ describe('Find', function () {
     }
   });
 
+  // TODO: NODE-3819: Unskip flaky MacOS tests.
   it.skip('shouldCorrectlyIterateOverCollection', {
     metadata: {
       requires: { topology: ['single', 'replicaset', 'sharded', 'ssl', 'heap', 'wiredtiger'] }

test/functional/unit-sdam/monitoring.test.js

@@ -22,7 +22,7 @@ describe('monitoring', function () {
     return mock.createServer().then(server => (mockServer = server));
   });
 
-  // Fails on MacOS on Evergreen (NODE-3713)
+  // TODO: NODE-3819: Unskip flaky MacOS tests.
   it.skip('should record roundTripTime', function (done) {
     mockServer.setMessageHandler(request => {
       const doc = request.document;

test/functional/write_concern.test.js

@@ -97,6 +97,7 @@ describe('Write Concern', function () {
 
   after(() => mock.cleanup());
 
+  // TODO: NODE-3816
   it.skip('should pipe writeConcern from client down to API call', function () {
     server.setMessageHandler(request => {
       if (request.document && request.document.ismaster) {