refactor(NODE-3717): test reorganization part 6 (#3090)

Author: baileympearson

docs/editor.md

@@ -49,7 +49,7 @@ Here's a quick description of each:
         "files.trimFinalNewlines": true,
         "files.insertFinalNewline": true,
         // Testing settings
-        "mochaExplorer.files": "test/unit/**/*.test.js",
+        "mochaExplorer.files": "test/unit/**/*.test.*",
         "mochaExplorer.ui": "test/tools/runner/metadata_ui.js",
         "mochaExplorer.envPath": null, // Useful for more advanced tests
         // Typescript settings

test/functional/aggregation.test.js

@@ -1120,8 +1120,7 @@ describe('Aggregation', function () {
     metadata: {
       requires: {
         mongodb: '>=3.5.0',
-        topology: 'single',
-        node: '>=4.8.5'
+        topology: 'single'
       }
     },
     test: function (done) {

test/functional/cursor_async_iterator.test.js

@@ -41,19 +41,16 @@ describe('Cursor Async Iterator Tests', function () {
       expect(counter).to.equal(1000);
     });
 
-    it('should be able to use a for-await loop on an aggregation cursor', {
-      metadata: { requires: { node: '>=10.5.0' } },
-      test: async function () {
-        const cursor = collection.aggregate([{ $match: { bar: 1 } }]);
-
-        let counter = 0;
-        for await (const doc of cursor) {
-          expect(doc).to.have.property('bar', 1);
-          counter += 1;
-        }
+    it('should be able to use a for-await loop on an aggregation cursor', async function () {
+      const cursor = collection.aggregate([{ $match: { bar: 1 } }]);
 
-        expect(counter).to.equal(1000);
+      let counter = 0;
+      for await (const doc of cursor) {
+        expect(doc).to.have.property('bar', 1);
+        counter += 1;
       }
+
+      expect(counter).to.equal(1000);
     });
 
     it('should be able to use a for-await loop on a command cursor', {

test/functional/deprecate_warning.test.js

@@ -1,5 +1,4 @@
 'use strict';
-const exec = require('child_process').exec;
 const chai = require('chai');
 
 const expect = chai.expect;
@@ -21,71 +20,6 @@ describe('Deprecation Warnings - functional', function () {
     this.sinon.stub(console, 'error');
   });
 
-  const defaultMessage = ' is deprecated and will be removed in a later version.';
-  it.skip('node --no-deprecation flag should suppress all deprecation warnings', {
-    metadata: { requires: { node: '>=6.0.0' } },
-    test: function (done) {
-      exec(
-        'node --no-deprecation ./test/tools/deprecate_warning_test_program.js',
-        (err, stdout, stderr) => {
-          expect(err).to.not.exist;
-          expect(stdout).to.be.empty;
-          expect(stderr).to.be.empty;
-          done();
-        }
-      );
-    }
-  });
-
-  it.skip('node --trace-deprecation flag should print stack trace to stderr', {
-    metadata: { requires: { node: '>=6.0.0' } },
-    test: function (done) {
-      exec(
-        'node --trace-deprecation ./test/tools/deprecate_warning_test_program.js',
-        (err, stdout, stderr) => {
-          expect(err).to.not.exist;
-          expect(stdout).to.be.empty;
-          expect(stderr).to.not.be.empty;
-
-          // split stderr into separate lines, trimming the first line to just the warning message
-          const split = stderr.split('\n');
-          const warning = split.shift().split(')')[1].trim();
-
-          // ensure warning message matches expected
-          expect(warning).to.equal(
-            'DeprecationWarning: testDeprecationFlags option [maxScan]' + defaultMessage
-          );
-
-          // ensure each following line is from the stack trace, i.e. 'at config.deprecatedOptions.forEach.deprecatedOption'
-          split.pop();
-          split.forEach(s => {
-            expect(s.trim()).to.match(/^at/);
-          });
-
-          done();
-        }
-      );
-    }
-  });
-
-  it.skip('node --throw-deprecation flag should throw error when deprecated function is called', {
-    metadata: { requires: { node: '>=6.0.0' } },
-    test: function (done) {
-      exec(
-        'node --throw-deprecation ./test/tools/deprecate_warning_test_program.js this_arg_should_never_print',
-        (err, stdout, stderr) => {
-          expect(stderr).to.not.be.empty;
-          expect(err).to.not.exist;
-          expect(err).to.have.own.property('code').that.equals(1);
-
-          // ensure stdout is empty, i.e. that the program threw an error before reaching the console.log statement
-          expect(stdout).to.be.empty;
-          done();
-        }
-      );
-    }
-  });
-
   it('test behavior for classes with an associated logger', function () {
     const fakeClass = new ClassWithLogger();
     const logger = fakeClass.getLogger();

test/functional/saslprep.test.js

@@ -70,8 +70,7 @@ describe('SASLPrep', function () {
     it(`should be able to login with username "${username}" and password "${password}"`, {
       metadata: {
         requires: {
-          mongodb: '>=3.7.3',
-          node: '>=6'
+          mongodb: '>=3.7.3'
         }
       },
       test: function () {