test: Fix test config

Our Mocha and Jest configuration has been messy because we're in the
middle of a migration from Mocha to Jest. Each file had to be
explicitly included in the Jest configuration or ignored in the Mocha
configuration. This was inconvenient and error-prone, and resulted in
some tests not being run at all.

Both test configurations have been updated to use a shared list of
Mocha test files. There are only a few of these files left, and this
list should only get shorter as we migrate more tests to Jest. No
further configuration changes will be needed to add Jest tests.

.mocharc.js

@@ -1,23 +1,6 @@
 module.exports = {
   // TODO: Remove the `exit` setting, it can hide broken tests.
   exit: true,
-  ignore: [
-    './app/scripts/lib/**/*.test.js',
-    './app/scripts/migrations/*.test.js',
-    './app/scripts/platforms/*.test.js',
-    './app/scripts/controllers/app-state.test.js',
-    './app/scripts/controllers/permissions/**/*.test.js',
-    './app/scripts/controllers/mmi-controller.test.ts',
-    './app/scripts/metamask-controller.actions.test.js',
-    './app/scripts/detect-multiple-instances.test.js',
-    './app/scripts/controllers/swaps.test.js',
-    './app/scripts/controllers/metametrics.test.js',
-    './app/scripts/controllers/preferences.test.js',
-    './app/scripts/constants/error-utils.test.js',
-    './app/scripts/metamask-controller.test.js',
-    './development/fitness-functions/**/*.test.ts',
-    './test/e2e/helpers.test.js',
-  ],
   recursive: true,
   require: ['test/env.js', 'test/setup.js'],
 };

jest.config.js

@@ -1,26 +1,10 @@
+const { legacyMochaTests } = require('./test/mocha/legacy-mocha-tests');
+
 module.exports = {
   collectCoverageFrom: [
-    '<rootDir>/app/scripts/constants/error-utils.js',
-    '<rootDir>/app/scripts/controllers/app-metadata.ts',
-    '<rootDir>/app/scripts/controllers/permissions/**/*.js',
-    '<rootDir>/app/scripts/controllers/sign.ts',
-    '<rootDir>/app/scripts/controllers/decrypt-message.ts',
-    '<rootDir>/app/scripts/controllers/encryption-public-key.ts',
-    '<rootDir>/app/scripts/controllers/transactions/etherscan.ts',
-    '<rootDir>/app/scripts/controllers/transactions/EtherscanRemoteTransactionSource.ts',
-    '<rootDir>/app/scripts/controllers/transactions/IncomingTransactionHelper.ts',
-    '<rootDir>/app/scripts/controllers/preferences.js',
-    '<rootDir>/app/scripts/flask/**/*.js',
-    '<rootDir>/app/scripts/lib/**/*.(js|ts)',
-    '<rootDir>/app/scripts/metamask-controller.js',
-    '<rootDir>/app/scripts/migrations/*.js',
-    '<rootDir>/app/scripts/migrations/*.ts',
-    '!<rootDir>/app/scripts/migrations/*.test.(js|ts)',
-    '<rootDir>/app/scripts/platforms/*.js',
+    '<rootDir>/app/scripts/**/*.(js|ts|tsx)',
     '<rootDir>/shared/**/*.(js|ts|tsx)',
     '<rootDir>/ui/**/*.(js|ts|tsx)',
-    '<rootDir>/development/fitness-functions/**/*.test.(js|ts|tsx)',
-    '<rootDir>/test/e2e/helpers.test.js',
   ],
   coverageDirectory: './coverage',
   coveragePathIgnorePatterns: ['.stories.*', '.snap'],
@@ -41,39 +25,15 @@ module.exports = {
   setupFiles: ['<rootDir>/test/setup.js', '<rootDir>/test/env.js'],
   setupFilesAfterEnv: ['<rootDir>/test/jest/setup.js'],
   testMatch: [
-    '<rootDir>/app/scripts/constants/error-utils.test.js',
-    '<rootDir>/app/scripts/controllers/app-metadata.test.ts',
-    '<rootDir>/app/scripts/controllers/app-state.test.js',
-    '<rootDir>/app/scripts/controllers/encryption-public-key.test.ts',
-    '<rootDir>/app/scripts/controllers/transactions/etherscan.test.ts',
-    '<rootDir>/app/scripts/controllers/transactions/EtherscanRemoteTransactionSource.test.ts',
-    '<rootDir>/app/scripts/controllers/transactions/IncomingTransactionHelper.test.ts',
-    '<rootDir>/app/scripts/controllers/onboarding.test.ts',
-    '<rootDir>/app/scripts/controllers/mmi-controller.test.ts',
-    '<rootDir>/app/scripts/controllers/permissions/**/*.test.js',
-    '<rootDir>/app/scripts/controllers/preferences.test.js',
-    '<rootDir>/app/scripts/controllers/sign.test.ts',
-    '<rootDir>/app/scripts/controllers/decrypt-message.test.ts',
-    '<rootDir>/app/scripts/controllers/authentication/**/*.test.ts',
-    '<rootDir>/app/scripts/controllers/push-platform-notifications/**/*.test.ts',
-    '<rootDir>/app/scripts/controllers/user-storage/**/*.test.ts',
-    '<rootDir>/app/scripts/controllers/metamask-notifications/**/*.test.ts',
-    '<rootDir>/app/scripts/metamask-controller.actions.test.js',
-    '<rootDir>/app/scripts/detect-multiple-instances.test.js',
-    '<rootDir>/app/scripts/controllers/swaps.test.js',
-    '<rootDir>/app/scripts/controllers/metametrics.test.js',
-    '<rootDir>/app/scripts/flask/**/*.test.js',
-    '<rootDir>/app/scripts/lib/**/*.test.(js|ts)',
-    '<rootDir>/app/scripts/lib/createRPCMethodTrackingMiddleware.test.js',
-    '<rootDir>/app/scripts/metamask-controller.test.js',
-    '<rootDir>/app/scripts/migrations/*.test.(js|ts)',
-    '<rootDir>/app/scripts/platforms/*.test.js',
-    '<rootDir>/app/scripts/translate.test.ts',
-    '<rootDir>/shared/**/*.test.(js|ts)',
+    '<rootDir>/app/scripts/**/*.test.(js|ts|tsx)',
+    '<rootDir>/shared/**/*.test.(js|ts|tsx)',
     '<rootDir>/ui/**/*.test.(js|ts|tsx)',
     '<rootDir>/development/fitness-functions/**/*.test.(js|ts|tsx)',
     '<rootDir>/test/e2e/helpers.test.js',
   ],
+  testPathIgnorePatterns: legacyMochaTests.map((filePath) => {
+    return `<rootDir>/${filePath}`;
+  }),
   testTimeout: 5500,
   // We have to specify the environment we are running in, which is jsdom. The
   // default is 'node'. This can be modified *per file* using a comment at the

test/mocha/legacy-mocha-tests.js

@@ -0,0 +1,10 @@
+// This is a list of remaining unit tests written for Mocha.
+// We are in the middle of a migration from Mocha to Jest. Please do not add any additional test
+// files to this list. We should only be removing files.
+const legacyMochaTests = [
+  'app/scripts/metamask-controller.actions.test.js',
+]
+
+module.exports = {
+  legacyMochaTests,
+}

test/run-unit-tests.js

@@ -1,6 +1,7 @@
 const { hideBin } = require('yargs/helpers');
 const yargs = require('yargs/yargs');
 const { runCommand, runInShell } = require('../development/lib/run-command');
+const { legacyMochaTests } = require('./mocha/legacy-mocha-tests');
 
 const { CIRCLE_NODE_INDEX, CIRCLE_NODE_TOTAL } = process.env;
 
@@ -71,7 +72,7 @@ async function runJest(
  * @param {boolean} coverage - Use nyc to collect coverage
  */
 async function runMocha({ coverage }) {
-  const options = ['mocha', './app/**/*.test.js'];
+  const options = ['mocha', ...legacyMochaTests];
   // If coverage is true, then we need to run nyc as the first command
   // and mocha after, so we use unshift to add three options to the beginning
   // of the options array.