Allow config.transform[regex] to be a function for jest.runCLI(config)

packages/jest-cli/src/__tests__/__fixtures__/run/aFile.js

@@ -0,0 +1,13 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+module.exports = {
+  toReplace: 1,
+};

packages/jest-cli/src/__tests__/__fixtures__/run/aFile_spec.js

@@ -0,0 +1,18 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+const expect = require('expect');
+const aFile = require('./aFile');
+
+describe('aFile test', () => {
+  it('should have transformed aFile', () => {
+    expect(JSON.stringify(aFile)).toEqual(JSON.stringify({runReplaced: 1}));
+  });
+});

packages/jest-cli/src/__tests__/__fixtures__/run/transform-module.js

@@ -0,0 +1,13 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+module.exports = {
+  process: (src, filename) => src.replace('toReplace', 'runReplaced'),
+};

packages/jest-cli/src/__tests__/__fixtures__/runCLI/aFile.js

@@ -0,0 +1,13 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+module.exports = {
+  toReplace: 1,
+};

packages/jest-cli/src/__tests__/__fixtures__/runCLI/aFile_spec.js

@@ -0,0 +1,18 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+const expect = require('expect');
+const aFile = require('./aFile');
+
+describe('aFile test', () => {
+  it('should have transformed aFile', () => {
+    expect(JSON.stringify(aFile)).toEqual(JSON.stringify({runCLIReplaced: 1}));
+  });
+});

packages/jest-cli/src/__tests__/__fixtures__/runCLI/transform-module.js

@@ -0,0 +1,13 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+module.exports = {
+  process: (src, filename) => src.replace('toReplace', 'runCLIReplaced'),
+};

packages/jest-cli/src/__tests__/cli/run.test.js

@@ -0,0 +1,99 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+import path from 'path';
+import {run} from '../../cli';
+
+const runProject = path.join(__dirname, '../__fixtures__/run');
+
+jest.mock('exit');
+
+const runArgvString = [
+  '--config',
+  JSON.stringify({
+    rootDir: runProject,
+    testMatch: ['<rootDir>/*_spec.js'],
+    transform: {
+      '^.+\\.jsx?$': './transform-module',
+    },
+  }),
+];
+
+const runArgvObject = [
+  '--config',
+  {
+    rootDir: runProject,
+    testMatch: ['<rootDir>/*_spec.js'],
+    transform: {
+      '^.+\\.jsx?$': './transform-module',
+    },
+  },
+];
+
+const processOnFn = process.on;
+const processExitFn = process.exit;
+const processErrWriteFn = process.stderr.write;
+const consoleErrorFn = console.error;
+
+const noSubTestLogs = true;
+
+describe('run', () => {
+  beforeEach(() => {
+    process.on = jest.fn();
+    process.on.mockReset();
+    process.exit = jest.fn();
+    process.exit.mockReset();
+    if (noSubTestLogs) {
+      process.stderr.write = jest.fn();
+      process.stderr.write.mockReset();
+      console.error = jest.fn();
+      console.error.mockReset();
+    }
+  });
+
+  afterEach(() => {
+    process.on = processOnFn;
+    process.exit = processExitFn;
+    if (noSubTestLogs) {
+      process.stderr.write = processErrWriteFn;
+      console.error = consoleErrorFn;
+    }
+  });
+
+  describe('config as string', () => {
+    it('passes the test when the config has a transform module path', async () => {
+      let runResult = null;
+      let error = null;
+      try {
+        runResult = await run(runArgvString, runProject);
+      } catch (ex) {
+        error = ex;
+      }
+      const numPassedTests = runResult ? runResult.numPassedTests : -1;
+      expect(error).toBe(null);
+      expect(numPassedTests).toBe(1);
+    });
+  });
+
+  describe('config as object', () => {
+    it('throws running the test when the config is an object', async () => {
+      let runResult = null;
+      let error = null;
+      try {
+        runResult = await run(runArgvObject, runProject);
+      } catch (ex) {
+        error = ex;
+      }
+      const numPassedTests = runResult ? runResult.numPassedTests : -1;
+      expect(error).not.toBe(null);
+      expect(numPassedTests).toBe(-1);
+    });
+  });
+});

packages/jest-cli/src/__tests__/cli/runCLI.test.js

@@ -0,0 +1,84 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+import path from 'path';
+import {runCLI} from '../../cli';
+import transformModule from '../__fixtures__/runCLI/transform-module';
+
+const project = path.join(__dirname, '../__fixtures__/runCLI');
+const projects = [project];
+
+const argvObject = {
+  config: {
+    testMatch: ['<rootDir>/*_spec.js'],
+    transform: {
+      '^.+\\.jsx?$': () => transformModule,
+    },
+  },
+};
+
+const argvString = {
+  config: JSON.stringify({
+    rootDir: project,
+    testMatch: ['<rootDir>/*_spec.js'],
+    transform: {
+      '^.+\\.jsx?$': './transform-module',
+    },
+  }),
+};
+
+const processErrWriteFn = process.stderr.write;
+
+const noSubTestLogs = true;
+
+describe('runCLI', () => {
+  beforeEach(() => {
+    if (noSubTestLogs) {
+      process.stderr.write = jest.fn();
+      process.stderr.write.mockReset();
+    }
+  });
+
+  afterEach(() => {
+    if (noSubTestLogs) {
+      process.stderr.write = processErrWriteFn;
+    }
+  });
+
+  describe('config as object', () => {
+    it('passes the test when the config has a transform function', async () => {
+      let runResult = null;
+      let error = null;
+      try {
+        runResult = await runCLI(argvObject, projects);
+      } catch (ex) {
+        error = ex;
+      }
+      const numPassedTests = runResult ? runResult.results.numPassedTests : -1;
+      expect(error).toBe(null);
+      expect(numPassedTests).toBe(1);
+    });
+  });
+
+  describe('config as string', () => {
+    it('passes the test when the config is a string', async () => {
+      let runResult = null;
+      let error = null;
+      try {
+        runResult = await runCLI(argvString, projects);
+      } catch (ex) {
+        error = ex;
+      }
+      const numPassedTests = runResult ? runResult.results.numPassedTests : -1;
+      expect(error).toBe(null);
+      expect(numPassedTests).toBe(1);
+    });
+  });
+});

packages/jest-cli/src/cli/index.js

@@ -33,18 +33,22 @@ import {sync as realpath} from 'realpath-native';
 import init from '../lib/init';
 import logDebugMessages from '../lib/log_debug_messages';
 
-export async function run(maybeArgv?: Argv, project?: Path) {
+export const run = async (
+  maybeArgv?: Argv,
+  project?: Path,
+): Promise<?AggregatedResult> => {
+  let results, globalConfig;
   try {
     const argv: Argv = buildArgv(maybeArgv, project);
 
     if (argv.init) {
       await init();
-      return;
+      return Promise.resolve(results);
     }
 
     const projects = getProjectListFromCLIArgs(argv, project);
 
-    const {results, globalConfig} = await runCLI(argv, projects);
+    ({results, globalConfig} = await runCLI(argv, projects));
     readResultsAndExit(results, globalConfig);
   } catch (error) {
     clearLine(process.stderr);
@@ -53,7 +57,8 @@ export async function run(maybeArgv?: Argv, project?: Path) {
     exit(1);
     throw error;
   }
-}
+  return Promise.resolve(results);
+};
 
 export const runCLI = async (
   argv: Argv,
@@ -170,19 +175,26 @@ const readResultsAndExit = (
 };
 
 const buildArgv = (maybeArgv: ?Argv, project: ?Path) => {
-  const argv: Argv = yargs(maybeArgv || process.argv.slice(2))
-    .usage(args.usage)
-    .alias('help', 'h')
-    .options(args.options)
-    .epilogue(args.docs)
-    .check(args.check).argv;
-
-  validateCLIOptions(
-    argv,
-    Object.assign({}, args.options, {deprecationEntries}),
-  );
+  try {
+    const argv: Argv = yargs(maybeArgv || process.argv.slice(2))
+      .usage(args.usage)
+      .alias('help', 'h')
+      .options(args.options)
+      .epilogue(args.docs)
+      .check(args.check).argv;
+
+    validateCLIOptions(
+      argv,
+      Object.assign({}, args.options, {deprecationEntries}),
+    );
 
-  return argv;
+    return argv;
+  } catch (err) {
+    if (maybeArgv) {
+      throw new Error('Command line arguments should be an array of strings');
+    }
+    throw err;
+  }
 };
 
 const getProjectListFromCLIArgs = (argv, project: ?Path) => {