chore: create @jest/test-results package (#8034)

Author: SimenB

CHANGELOG.md

@@ -90,6 +90,7 @@
 - `[jest-cli]`: Migrate to TypeScript ([#8024](https://github.com/facebook/jest/pull/8024))
 - `[jest]`: Migrate to TypeScript ([#8024](https://github.com/facebook/jest/pull/8024))
 - `[docs]` Update automock configuration, add note related to manual mocks ([#8051](https://github.com/facebook/jest/pull/8051))
+- `[@jest/test-result]`: Extract TestResult types and helpers into a new separate package ([#8034](https://github.com/facebook/jest/pull/8034))
 
 ### Performance
 

packages/jest-circus/package.json

@@ -12,6 +12,7 @@
   "dependencies": {
     "@babel/traverse": "^7.1.0",
     "@jest/environment": "^24.1.0",
+    "@jest/test-result": "^24.1.0",
     "@jest/types": "^24.1.0",
     "@types/node": "*",
     "chalk": "^2.0.1",

packages/jest-circus/src/legacy-code-todo-rewrite/jestAdapter.ts

@@ -6,11 +6,12 @@
  */
 
 import path from 'path';
-import {Config, TestResult} from '@jest/types';
+import {Config} from '@jest/types';
 import {JestEnvironment} from '@jest/environment';
+import {TestResult} from '@jest/test-result';
 // eslint-disable-next-line import/no-extraneous-dependencies
 import Runtime from 'jest-runtime';
-import {SnapshotState} from 'jest-snapshot';
+import {SnapshotStateType} from 'jest-snapshot';
 
 const FRAMEWORK_INITIALIZER = require.resolve('./jestAdapterInit');
 
@@ -20,7 +21,7 @@ const jestAdapter = async (
   environment: JestEnvironment,
   runtime: Runtime,
   testPath: string,
-): Promise<TestResult.TestResult> => {
+): Promise<TestResult> => {
   const {
     initialize,
     runAndTransformResultsToJestFormat,
@@ -86,9 +87,8 @@ const jestAdapter = async (
 };
 
 const _addSnapshotData = (
-  results: TestResult.TestResult,
-  // TODO: make just snapshotState: SnapshotState when `jest-snapshot` is ESM
-  snapshotState: typeof SnapshotState.prototype,
+  results: TestResult,
+  snapshotState: SnapshotStateType,
 ) => {
   results.testResults.forEach(({fullName, status}) => {
     if (status === 'pending' || status === 'failed') {

packages/jest-circus/src/legacy-code-todo-rewrite/jestAdapterInit.ts

@@ -5,8 +5,8 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-import {Config, TestResult} from '@jest/types';
-
+import {Config} from '@jest/types';
+import {AssertionResult, Status, TestResult} from '@jest/test-result';
 import {extractExpectedAssertionsErrors, getState, setState} from 'expect';
 import {formatExecError, formatResultsErrors} from 'jest-message-util';
 import {
@@ -127,51 +127,51 @@ export const runAndTransformResultsToJestFormat = async ({
   config: Config.ProjectConfig;
   globalConfig: Config.GlobalConfig;
   testPath: string;
-}): Promise<TestResult.TestResult> => {
+}): Promise<TestResult> => {
   const runResult: RunResult = await run();
 
   let numFailingTests = 0;
   let numPassingTests = 0;
   let numPendingTests = 0;
   let numTodoTests = 0;
 
-  const assertionResults: Array<
-    TestResult.AssertionResult
-  > = runResult.testResults.map(testResult => {
-    let status: TestResult.Status;
-    if (testResult.status === 'skip') {
-      status = 'pending';
-      numPendingTests += 1;
-    } else if (testResult.status === 'todo') {
-      status = 'todo';
-      numTodoTests += 1;
-    } else if (testResult.errors.length) {
-      status = 'failed';
-      numFailingTests += 1;
-    } else {
-      status = 'passed';
-      numPassingTests += 1;
-    }
+  const assertionResults: Array<AssertionResult> = runResult.testResults.map(
+    testResult => {
+      let status: Status;
+      if (testResult.status === 'skip') {
+        status = 'pending';
+        numPendingTests += 1;
+      } else if (testResult.status === 'todo') {
+        status = 'todo';
+        numTodoTests += 1;
+      } else if (testResult.errors.length) {
+        status = 'failed';
+        numFailingTests += 1;
+      } else {
+        status = 'passed';
+        numPassingTests += 1;
+      }
 
-    const ancestorTitles = testResult.testPath.filter(
-      name => name !== ROOT_DESCRIBE_BLOCK_NAME,
-    );
-    const title = ancestorTitles.pop();
+      const ancestorTitles = testResult.testPath.filter(
+        name => name !== ROOT_DESCRIBE_BLOCK_NAME,
+      );
+      const title = ancestorTitles.pop();
 
-    return {
-      ancestorTitles,
-      duration: testResult.duration,
-      failureMessages: testResult.errors,
-      fullName: title
-        ? ancestorTitles.concat(title).join(' ')
-        : ancestorTitles.join(' '),
-      invocations: testResult.invocations,
-      location: testResult.location,
-      numPassingAsserts: 0,
-      status,
-      title: testResult.testPath[testResult.testPath.length - 1],
-    };
-  });
+      return {
+        ancestorTitles,
+        duration: testResult.duration,
+        failureMessages: testResult.errors,
+        fullName: title
+          ? ancestorTitles.concat(title).join(' ')
+          : ancestorTitles.join(' '),
+        invocations: testResult.invocations,
+        location: testResult.location,
+        numPassingAsserts: 0,
+        status,
+        title: testResult.testPath[testResult.testPath.length - 1],
+      };
+    },
+  );
 
   let failureMessage = formatResultsErrors(
     assertionResults,

packages/jest-circus/tsconfig.json

@@ -11,6 +11,7 @@
     {"path": "../jest-message-util"},
     {"path": "../jest-runtime"},
     {"path": "../jest-snapshot"},
+    {"path": "../jest-test-result"},
     {"path": "../jest-types"},
     {"path": "../jest-util"},
     {"path": "../pretty-format"}

packages/jest-cli/package.json

@@ -6,6 +6,7 @@
   "types": "build/index.d.ts",
   "dependencies": {
     "@jest/core": "^24.1.0",
+    "@jest/test-result": "^24.1.0",
     "@jest/types": "^24.1.0",
     "chalk": "^2.0.1",
     "exit": "^0.1.2",

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

@@ -6,7 +6,8 @@
  */
 
 import path from 'path';
-import {Config, TestResult} from '@jest/types';
+import {Config} from '@jest/types';
+import {AggregatedResult} from '@jest/test-result';
 import {clearLine} from 'jest-util';
 import {validateCLIOptions} from 'jest-validate';
 import {deprecationEntries} from 'jest-config';
@@ -104,7 +105,7 @@ const getProjectListFromCLIArgs = (
 };
 
 const readResultsAndExit = (
-  result: TestResult.AggregatedResult | null,
+  result: AggregatedResult | null,
   globalConfig: Config.GlobalConfig,
 ) => {
   const code = !result || result.success ? 0 : globalConfig.testFailureExitCode;

packages/jest-cli/tsconfig.json

@@ -7,6 +7,7 @@
   "references": [
     {"path": "../jest-core"},
     {"path": "../jest-config"},
+    {"path": "../jest-test-result"},
     {"path": "../jest-types"},
     {"path": "../jest-util"},
     {"path": "../jest-validate"}

packages/jest-core/package.json

@@ -8,6 +8,7 @@
     "@jest/console": "^24.1.0",
     "@jest/types": "^24.1.0",
     "@jest/reporters": "^24.1.0",
+    "@jest/test-result": "^24.1.0",
     "@jest/transform": "^24.1.0",
     "ansi-escapes": "^3.0.0",
     "chalk": "^2.0.1",

packages/jest-core/src/FailedTestsCache.ts

@@ -6,7 +6,8 @@
  */
 
 import {Test} from 'jest-runner';
-import {Config, TestResult} from '@jest/types';
+import {Config} from '@jest/types';
+import {TestResult} from '@jest/test-result';
 
 type TestMap = {[key: string]: {[key: string]: boolean}};
 
@@ -22,7 +23,7 @@ export default class FailedTestsCache {
     return tests.filter(testResult => enabledTestsMap[testResult.path]);
   }
 
-  setTestResults(testResults: Array<TestResult.TestResult>) {
+  setTestResults(testResults: Array<TestResult>) {
     this._enabledTestsMap = (testResults || [])
       .filter(testResult => testResult.numFailingTests)
       .reduce<TestMap>((suiteMap, testResult) => {