chore(NODE-4358): move legacy tests to legacy folder (#3307)

Author: baileympearson

test/integration/client-side-encryption/client_side_encryption.spec.test.js renamed to test/integration/client-side-encryption/client_side_encryption.spec.test.ts

@@ -1,11 +1,12 @@
-'use strict';
+import * as path from 'path';
 
-const path = require('path');
-const {
-  TestRunnerContext,
+import { loadSpecTests } from '../../spec';
+import {
   gatherTestSuites,
-  generateTopologyTests
-} = require('../../tools/spec-runner');
+  generateTopologyTests,
+  TestRunnerContext
+} from '../../tools/spec-runner';
+import { runUnifiedSuite } from '../../tools/unified-spec-runner/runner';
 
 const isAuthEnabled = process.env.AUTH === 'auth';
 
@@ -62,11 +63,10 @@ const SKIPPED_TESTS = new Set([
   ...(isAuthEnabled ? skippedAuthTests.concat(skippedNoAuthTests) : skippedNoAuthTests)
 ]);
 
-describe('Client Side Encryption', function () {
-  const testContext = new TestRunnerContext();
-  testContext.requiresCSFLE = true;
+describe('Client Side Encryption (Legacy)', function () {
+  const testContext = new TestRunnerContext({ requiresCSFLE: true });
   const testSuites = gatherTestSuites(
-    path.join(__dirname, '../../spec/client-side-encryption/tests'),
+    path.join(__dirname, '../../spec/client-side-encryption/tests/legacy'),
     testContext
   );
   after(() => testContext.teardown());
@@ -78,3 +78,10 @@ describe('Client Side Encryption', function () {
     return !SKIPPED_TESTS.has(spec.description);
   });
 });
+
+describe('Client Side Encryption (Unified)', function () {
+  runUnifiedSuite(
+    loadSpecTests(path.join('client-side-encryption', 'tests', 'unified')),
+    () => 'NODE-4330 - implement the key management API'
+  );
+});

test/integration/unified-test-format/unified_test_format.spec.test.ts

@@ -37,6 +37,14 @@ const filter: TestFilter = ({ description }) => {
     return 'TODO(NODE-3891): fix tests broken when AUTH enabled';
   }
 
+  if (description.length === 0) {
+    // This may seem weird, but the kmsProvider valid pass tests really test that the new
+    // client encryption entity is constructed correctly so the "test" section of each
+    // unified test is empty (save the required properties) and the test description
+    // is just the empty string
+    return 'TODO(NODE-4363): add support for client encryption entity to unified runner';
+  }
+
   return false;
 };