Fix and enable migrate options tests

jest.config.ts

@@ -9,11 +9,12 @@ const config: Config = {
     preset: 'ts-jest',
     testEnvironment: 'node',
     transformIgnorePatterns: ['/dist/.+\\.js'],
+    // We need to explicity ignore the out directory for modules - otherwise we'll get duplicate vscode module,
+    // the TS version from the __mocks__ directory and the compiled js version from the out directory.
     modulePathIgnorePatterns: ['out'],
-    /* Specify jest to only run tests in jestUnitTests.
-     * We also have to include the __mocks__ folder.  That folder must be next to node_modules so we can't move it,
-     * but if we specify roots, jest won't automatically pick it up.  So we have to specify it here.
-     */
+    // Specify jest to only run tests in jest folders.
+    // We also have to include the __mocks__ folder.  That folder must be next to node_modules so we can't move it,
+    // but if we specify roots, jest won't automatically pick it up.  So we have to specify it here.
     roots: ['<rootDir>/test/unitTests', '<rootDir>/omnisharptest/omnisharpJestTests', '<rootDir>/__mocks__'],
 };
 

test/unitTests/migrateOptionsTest.ts → test/unitTests/migrateOptions.test.ts

@@ -10,7 +10,8 @@ import * as jestLib from '@jest/globals';
 
 jestLib.describe('Migrate configuration should in package.json', () => {
     const packageJson = JSON.parse(readFileSync('package.json').toString());
-    const configurations = Object.keys(packageJson['contributes']['configuration']['properties']);
+    const properties = packageJson.contributes.configuration[1].properties;
+    const configurations = Object.keys(properties);
 
     migrateOptions.forEach((data) => {
         jestLib.test(`Should have ${data.roslynOption} in package.json`, () => {