TS migration jest-circus

packages/jest-circus/package.json

@@ -8,8 +8,10 @@
   },
   "license": "MIT",
   "main": "build/index.js",
+  "types": "build/index.d.ts",
   "dependencies": {
     "@babel/traverse": "^7.1.0",
+    "@jest/types": "^24.1.0",
     "chalk": "^2.0.1",
     "co": "^4.6.0",
     "expect": "^24.1.0",

packages/jest-circus/src/__mocks__/testEventHandler.js → packages/jest-circus/src/__mocks__/testEventHandler.ts

@@ -4,13 +4,9 @@
  * This source code is licensed under the BSD-style license found in the
  * LICENSE file in the root directory of this source tree. An additional grant
  * of patent rights can be found in the PATENTS file in the same directory.
- *
- * @flow scrict-local
  */
 
-'use strict';
-
-import type {EventHandler} from 'types/Circus';
+import {EventHandler} from '../types';
 
 const testEventHandler: EventHandler = (event, state) => {
   switch (event.name) {

packages/jest-circus/src/__mocks__/testUtils.js → packages/jest-circus/src/__mocks__/testUtils.ts

@@ -4,17 +4,14 @@
  * This source code is licensed under the BSD-style license found in the
  * LICENSE file in the root directory of this source tree. An additional grant
  * of patent rights can be found in the PATENTS file in the same directory.
- *
- * @flow strict-local
  */
 
-'use strict';
-// $FlowFixMe - execa is untyped
-import {sync as spawnSync} from 'execa';
 import fs from 'fs';
 import os from 'os';
 import path from 'path';
 import crypto from 'crypto';
+import {sync as spawnSync, ExecaReturns} from 'execa';
+// @ts-ignore
 import {skipSuiteOnWindows} from '../../../../scripts/ConditionalTest';
 
 const CIRCUS_PATH = require.resolve('../../build/index');
@@ -25,6 +22,11 @@ const BABEL_REGISTER_PATH = require.resolve('@babel/register');
 
 skipSuiteOnWindows();
 
+interface Result extends ExecaReturns {
+  status: number;
+  error: string;
+}
+
 export const runTest = (source: string) => {
   const filename = crypto
     .createHash('md5')
@@ -54,7 +56,9 @@ export const runTest = (source: string) => {
   `;
 
   fs.writeFileSync(tmpFilename, content);
-  const result = spawnSync('node', [tmpFilename], {cwd: process.cwd()});
+  const result = spawnSync('node', [tmpFilename], {
+    cwd: process.cwd(),
+  }) as Result;
 
   // For compat with cross-spawn
   result.status = result.code;

packages/jest-circus/src/__tests__/afterAll.test.js → packages/jest-circus/src/__tests__/afterAll.test.ts

@@ -4,12 +4,8 @@
  * This source code is licensed under the BSD-style license found in the
  * LICENSE file in the root directory of this source tree. An additional grant
  * of patent rights can be found in the PATENTS file in the same directory.
- *
- * @flow strict-local
  */
 
-'use strict';
-
 import {runTest} from '../__mocks__/testUtils';
 
 test('tests are not marked done until their parent afterAll runs', () => {

packages/jest-circus/src/__tests__/baseTest.test.js → packages/jest-circus/src/__tests__/baseTest.test.ts

@@ -4,12 +4,8 @@
  * This source code is licensed under the BSD-style license found in the
  * LICENSE file in the root directory of this source tree. An additional grant
  * of patent rights can be found in the PATENTS file in the same directory.
- *
- * @flow strict-local
  */
 
-'use strict';
-
 import {runTest} from '../__mocks__/testUtils';
 
 test('simple test', () => {

packages/jest-circus/src/__tests__/circusItTestError.test.js → packages/jest-circus/src/__tests__/circusItTestError.test.ts

@@ -3,20 +3,15 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- * @flow strict-local
  */
-
-'use strict';
-
 let circusIt;
 let circusTest;
 
 // using jest-jasmine2's 'it' to test jest-circus's 'it'. Had to differentiate
 // the two with this alias.
 
 const aliasCircusIt = () => {
-  const {it, test} = require('../index.js');
+  const {it, test} = require('../index.ts');
   circusIt = it;
   circusTest = test;
 };
@@ -35,21 +30,18 @@ describe('test/it error throwing', () => {
   });
   it(`it throws error with missing callback function`, () => {
     expect(() => {
-      // $FlowFixMe: Easy, we're testing runtime errors here
       circusIt('test2');
     }).toThrowError(
       'Missing second argument. It must be a callback function. Perhaps you want to use `test.todo` for a test placeholder.',
     );
   });
   it(`it throws an error when first argument isn't a string`, () => {
     expect(() => {
-      // $FlowFixMe: Easy, we're testing runtime errors here
       circusIt(() => {});
     }).toThrowError('Invalid first argument, () => {}. It must be a string.');
   });
   it('it throws an error when callback function is not a function', () => {
     expect(() => {
-      // $FlowFixMe: Easy, we're testing runtime errors here
       circusIt('test4', 'test4b');
     }).toThrowError(
       'Invalid second argument, test4b. It must be a callback function.',
@@ -62,21 +54,18 @@ describe('test/it error throwing', () => {
   });
   it(`test throws error with missing callback function`, () => {
     expect(() => {
-      // $FlowFixMe: Easy, we're testing runtime errors here
       circusTest('test6');
     }).toThrowError(
       'Missing second argument. It must be a callback function. Perhaps you want to use `test.todo` for a test placeholder.',
     );
   });
   it(`test throws an error when first argument isn't a string`, () => {
     expect(() => {
-      // $FlowFixMe: Easy, we're testing runtime errors here
       circusTest(() => {});
     }).toThrowError('Invalid first argument, () => {}. It must be a string.');
   });
   it('test throws an error when callback function is not a function', () => {
     expect(() => {
-      // $FlowFixMe: Easy, we're testing runtime errors here
       circusTest('test8', 'test8b');
     }).toThrowError(
       'Invalid second argument, test8b. It must be a callback function.',

packages/jest-circus/src/__tests__/circusItTodoTestError.test.js → packages/jest-circus/src/__tests__/circusItTodoTestError.test.ts

@@ -3,19 +3,15 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- * @flow strict-local
  */
 
-'use strict';
-
 let circusIt;
 
 // using jest-jasmine2's 'it' to test jest-circus's 'it'. Had to differentiate
 // the two with this alias.
 
 const aliasCircusIt = () => {
-  const {it} = require('../index.js');
+  const {it} = require('../index.ts');
   circusIt = it;
 };
 
@@ -24,7 +20,6 @@ aliasCircusIt();
 describe('test/it.todo error throwing', () => {
   it('todo throws error when given no arguments', () => {
     expect(() => {
-      // $FlowFixMe: Testing runitme errors here
       circusIt.todo();
     }).toThrowError('Todo must be called with only a description.');
   });
@@ -35,7 +30,6 @@ describe('test/it.todo error throwing', () => {
   });
   it('todo throws error when given none string description', () => {
     expect(() => {
-      // $FlowFixMe: Testing runitme errors here
       circusIt.todo(() => {});
     }).toThrowError('Todo must be called with only a description.');
   });

packages/jest-circus/src/__tests__/hooks.test.js → packages/jest-circus/src/__tests__/hooks.test.ts

@@ -4,12 +4,8 @@
  * This source code is licensed under the BSD-style license found in the
  * LICENSE file in the root directory of this source tree. An additional grant
  * of patent rights can be found in the PATENTS file in the same directory.
- *
- * @flow strict-local
  */
 
-'use strict';
-
 import {runTest} from '../__mocks__/testUtils';
 
 test('beforeEach is executed before each test in current/child describe blocks', () => {

packages/jest-circus/src/__tests__/hooksError.test.js → packages/jest-circus/src/__tests__/hooksError.test.ts

@@ -3,17 +3,13 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- * @flow
  */
 
-'use strict';
-
-const circus = require('../index.js');
+import circus from '../';
 
 describe.each([['beforeEach'], ['beforeAll'], ['afterEach'], ['afterAll']])(
   '%s hooks error throwing',
-  fn => {
+  (fn: 'beforeEach' | 'beforeAll' | 'afterEach' | 'afterAll') => {
     test.each([
       ['String'],
       [1],

packages/jest-circus/src/dts/co.d.ts

@@ -0,0 +1,20 @@
+// TODO replace with @types/co when it is merged https://github.com/DefinitelyTyped/DefinitelyTyped/pull/33120
+declare module 'co' {
+  type ExtractType<T> = T extends IterableIterator<infer R> ? R : never;
+
+  interface Co {
+    <F extends (...args: any[]) => Generator>(
+      fn: F,
+      ...args: Parameters<F>
+    ): Promise<ExtractType<ReturnType<F>>>;
+    default: Co;
+    co: Co;
+    wrap: <F extends (...args: any[]) => Generator>(
+      fn: F,
+    ) => (...args: Parameters<F>) => Promise<ExtractType<ReturnType<F>>>;
+  }
+
+  const co: Co;
+
+  export = co;
+}

packages/jest-circus/src/eventHandler.js → packages/jest-circus/src/eventHandler.ts

@@ -3,11 +3,9 @@
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
- *
- * @flow strict-local
  */
 
-import type {EventHandler, Exception} from 'types/Circus';
+import {EventHandler, Exception, TEST_TIMEOUT_SYMBOL} from './types';
 
 import {
   addErrorToEachTestUnderDescribe,
@@ -22,9 +20,6 @@ import {
   restoreGlobalErrorHandlers,
 } from './globalErrorHandlers';
 
-// To pass this value from Runtime object to state we need to use global[sym]
-const TEST_TIMEOUT_SYMBOL = Symbol.for('TEST_TIMEOUT_SYMBOL');
-
 const eventHandler: EventHandler = (event, state): void => {
   switch (event.name) {
     case 'include_test_location_in_result': {
@@ -112,14 +107,14 @@ const eventHandler: EventHandler = (event, state): void => {
 
       if (type === 'beforeAll') {
         invariant(describeBlock, 'always present for `*All` hooks');
-        addErrorToEachTestUnderDescribe(describeBlock, error, asyncError);
+        addErrorToEachTestUnderDescribe(describeBlock!, error, asyncError);
       } else if (type === 'afterAll') {
         // Attaching `afterAll` errors to each test makes execution flow
         // too complicated, so we'll consider them to be global.
         state.unhandledErrors.push([error, asyncError]);
       } else {
         invariant(test, 'always present for `*Each` hooks');
-        test.errors.push([error, asyncError]);
+        test!.errors.push([error, asyncError]);
       }
       break;
     }
@@ -152,7 +147,7 @@ const eventHandler: EventHandler = (event, state): void => {
       break;
     }
     case 'test_retry': {
-      const errors: Array<[?Exception, Exception]> = [];
+      const errors: Array<[Exception | undefined | null, Exception]> = [];
       event.test.errors = errors;
       break;
     }
@@ -172,7 +167,7 @@ const eventHandler: EventHandler = (event, state): void => {
       // i'm not sure if this is works. For now i just replicated whatever
       // jasmine was doing -- dabramov
       state.parentProcess = event.parentProcess;
-      invariant(state.parentProcess);
+      invariant(state.parentProcess, '');
       state.originalGlobalErrorHandlers = injectGlobalErrorHandlers(
         state.parentProcess,
       );
@@ -182,11 +177,11 @@ const eventHandler: EventHandler = (event, state): void => {
       break;
     }
     case 'teardown': {
-      invariant(state.originalGlobalErrorHandlers);
-      invariant(state.parentProcess);
+      invariant(state.originalGlobalErrorHandlers, '');
+      invariant(state.parentProcess, '');
       restoreGlobalErrorHandlers(
-        state.parentProcess,
-        state.originalGlobalErrorHandlers,
+        state.parentProcess!,
+        state.originalGlobalErrorHandlers!,
       );
       break;
     }