Move jest-runtime CLI into jest-repl

Fixes #10011

Author: ghostd

CHANGELOG.md

@@ -22,6 +22,7 @@
 - `[*]` [**BREAKING**] Add `exports` field to all `package.json`s ([#9921](https://github.com/facebook/jest/pull/9921))
 - `[jest-config]` [**BREAKING**] Remove `enabledTestsMap` config, use `filter` instead ([#10787](https://github.com/facebook/jest/pull/10787))
 - `[jest-resolve]` [**BREAKING**] Migrate to ESM ([#10688](https://github.com/facebook/jest/pull/10688))
+- `[jest-repl, jest-runtime]` [**BREAKING**] Move the `jest-runtime` CLI into `jest-repl` ([#10016](https://github.com/facebook/jest/pull/10016))
 
 ### Performance
 

jest.config.js

@@ -50,6 +50,7 @@ module.exports = {
     '/packages/jest-haste-map/src/__tests__/haste_impl.js',
     '/packages/jest-haste-map/src/__tests__/dependencyExtractor.js',
     '/packages/jest-haste-map/src/__tests__/test_dotfiles_root/',
+    '/packages/jest-repl/src/__tests__/test_root',
     '/packages/jest-resolve-dependencies/src/__tests__/__fixtures__/',
     '/packages/jest-runtime/src/__tests__/defaultResolver.js',
     '/packages/jest-runtime/src/__tests__/module_dir/',

packages/jest-core/src/__tests__/SearchSource.test.ts

@@ -451,14 +451,12 @@ describe('SearchSource', () => {
     it('finds tests that depend directly on the path', () => {
       const filePath = path.join(rootDir, 'RegularModule.js');
       const file2Path = path.join(rootDir, 'RequireRegularModule.js');
-      const loggingDep = path.join(rootDir, 'logging.js');
       const parentDep = path.join(rootDir, 'ModuleWithSideEffects.js');
       const data = searchSource.findRelatedTests(new Set([filePath]), false);
       expect(toPaths(data.tests).sort()).toEqual([
         parentDep,
         filePath,
         file2Path,
-        loggingDep,
         rootPath,
       ]);
     });

packages/jest-runtime/bin/jest-runtime.js renamed to packages/jest-repl/bin/jest-runtime-cli.js

@@ -10,4 +10,4 @@ if (process.env.NODE_ENV == null) {
   process.env.NODE_ENV = 'test';
 }
 
-require('../build/cli').run();
+require('../build/cli/runtime-cli').run();

packages/jest-repl/package.json

@@ -15,17 +15,22 @@
     "./bin/jest-repl": "./bin/jest-repl.js"
   },
   "dependencies": {
+    "@jest/console": "^26.6.2",
+    "@jest/environment": "^26.6.2",
     "@jest/transform": "^26.6.2",
     "@jest/types": "^26.6.2",
+    "chalk": "^4.0.0",
     "jest-config": "^26.6.3",
     "jest-runtime": "^26.6.3",
+    "jest-util": "^26.6.2",
     "jest-validate": "^26.6.2",
     "repl": "^0.1.3",
     "yargs": "^16.0.3"
   },
   "devDependencies": {
     "@jest/test-utils": "^26.6.2",
-    "@types/yargs": "^15.0.0"
+    "@types/yargs": "^15.0.0",
+    "execa": "^4.0.0"
   },
   "bin": "./bin/jest-repl.js",
   "engines": {

packages/jest-repl/src/__tests__/runtime_cli.test.js

@@ -0,0 +1,49 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates. 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.
+ */
+
+import path from 'path';
+import {sync as spawnSync} from 'execa';
+
+const JEST_RUNTIME = path.resolve(__dirname, '../../bin/jest-runtime-cli.js');
+
+const run = args =>
+  spawnSync(JEST_RUNTIME, args, {
+    cwd: process.cwd(),
+    env: process.env,
+    reject: false,
+  });
+
+describe('Runtime CLI', () => {
+  it('fails with no path', () => {
+    const expectedOutput =
+      'Please provide a path to a script. (See --help for details)';
+    expect(run([]).stdout).toBe(expectedOutput);
+  });
+
+  it('displays script output', () => {
+    const scriptPath = path.resolve(__dirname, './test_root/logging.js');
+    expect(run([scriptPath, '--no-cache']).stdout).toMatch('Hello, world!');
+  });
+
+  it('always disables automocking', () => {
+    const scriptPath = path.resolve(__dirname, './test_root/logging.js');
+    const output = run([
+      scriptPath,
+      '--no-cache',
+      '--config=' +
+        JSON.stringify({
+          automock: true,
+        }),
+    ]);
+    expect(output.stdout).toMatch('Hello, world!');
+  });
+
+  it('throws script errors', () => {
+    const scriptPath = path.resolve(__dirname, './test_root/throwing.js');
+    expect(run([scriptPath, '--no-cache']).stderr).toMatch('Error: throwing');
+  });
+});

packages/jest-runtime/src/__tests__/test_root/logging.js renamed to packages/jest-repl/src/__tests__/test_root/logging.js

@@ -7,7 +7,9 @@
 
 'use strict';
 
-if (require('./RegularModule').getModuleStateValue()) {
+if (
+  require('jest-runtime/src/__tests__/test_root/RegularModule').getModuleStateValue()
+) {
   console.log('Hello, world!');
 } else {
   console.log('Automocking is not properly disabled in jest-runtime.');

packages/jest-repl/src/__tests__/test_root/throwing.js

@@ -0,0 +1,10 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates. 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';
+
+throw new Error('throwing');

packages/jest-repl/src/cli/args.ts

@@ -7,12 +7,45 @@
  */
 
 import type {Options} from 'yargs';
-import Runtime from 'jest-runtime';
 
 export const usage = 'Usage: $0 [--config=<pathToConfigFile>]';
 
+const runtimeCLIOptions: Record<
+  'cache' | 'config' | 'debug' | 'version' | 'watchman',
+  Options
+> = {
+  cache: {
+    default: true,
+    description:
+      'Whether to use the preprocessor cache. Disable ' +
+      'the cache using --no-cache.',
+    type: 'boolean',
+  },
+  config: {
+    alias: 'c',
+    description: 'The path to a Jest config file.',
+    type: 'string',
+  },
+  debug: {
+    description: 'Print debugging info about your jest config.',
+    type: 'boolean',
+  },
+  version: {
+    alias: 'v',
+    description: 'Print the version and exit',
+    type: 'boolean',
+  },
+  watchman: {
+    default: true,
+    description:
+      'Whether to use watchman for file crawling. Disable using ' +
+      '--no-watchman.',
+    type: 'boolean',
+  },
+};
+
 export const options: Record<string, Options> = {
-  ...Runtime.getCLIOptions(),
+  ...runtimeCLIOptions,
   replname: {
     alias: 'r',
     description:

packages/jest-repl/src/cli/index.ts

@@ -10,10 +10,10 @@
 import yargs = require('yargs');
 import type {Config} from '@jest/types';
 import {deprecationEntries} from 'jest-config';
-import Runtime from 'jest-runtime';
 import {validateCLIOptions} from 'jest-validate';
 import * as args from './args';
-const {version: VERSION} = require('../../package.json');
+import {run as runtimeCLI} from './runtime-cli';
+import {VERSION} from './version';
 
 const REPL_SCRIPT = require.resolve('./repl.js');
 
@@ -24,5 +24,5 @@ export = function (): void {
 
   argv._ = [REPL_SCRIPT];
 
-  Runtime.runCLI(argv, [`Jest REPL v${VERSION}`]);
+  runtimeCLI(argv, [`Jest REPL v${VERSION}`]);
 };