Fix: jest-worker: should not expose .default babel interop (#10623)

Co-authored-by: Tayeeb Hasan <tayeeb.hasan@gmail.com>

CHANGELOG.md

@@ -37,6 +37,7 @@
 - `[jest-transform]` Show enhanced `SyntaxError` message for all `SyntaxError`s ([#10749](https://github.com/facebook/jest/pull/10749))
 - `[jest-transform]` [**BREAKING**] Refactor API to pass an options bag around rather than multiple boolean options ([#10753](https://github.com/facebook/jest/pull/10753))
 - `[jest-transform]` [**BREAKING**] Refactor API of transformers to pass an options bag rather than separate `config` and other options ([#10834](https://github.com/facebook/jest/pull/10834))
+- `[jest-worker]` [**BREAKING**] Use named exports ([#10623] (https://github.com/facebook/jest/pull/10623))
 - `[pretty-format]` [**BREAKING**] Convert to ES Modules ([#10515](https://github.com/facebook/jest/pull/10515))
 
 ### Chore & Maintenance

packages/jest-haste-map/src/__tests__/index.test.js

@@ -19,8 +19,8 @@ jest.mock('child_process', () => ({
   execSync() {},
 }));
 
-jest.mock('jest-worker', () =>
-  jest.fn(worker => {
+jest.mock('jest-worker', () => ({
+  Worker: jest.fn(worker => {
     mockWorker = jest.fn((...args) => require(worker).worker(...args));
     mockEnd = jest.fn();
 
@@ -29,7 +29,7 @@ jest.mock('jest-worker', () =>
       worker: mockWorker,
     };
   }),
-);
+}));
 
 jest.mock('../crawlers/node');
 jest.mock('../crawlers/watchman', () =>
@@ -1197,7 +1197,7 @@ describe('HasteMap', () => {
   });
 
   it('distributes work across workers', () => {
-    const jestWorker = require('jest-worker');
+    const jestWorker = require('jest-worker').Worker;
     const path = require('path');
     const dependencyExtractor = path.join(__dirname, 'dependencyExtractor.js');
     return new HasteMap({

packages/jest-haste-map/src/index.ts

@@ -16,7 +16,7 @@ import type {Stats} from 'graceful-fs';
 import type {Config} from '@jest/types';
 import {escapePathForRegex} from 'jest-regex-util';
 import serializer from 'jest-serializer';
-import Worker from 'jest-worker';
+import {Worker} from 'jest-worker';
 import HasteFS from './HasteFS';
 import HasteModuleMap from './ModuleMap';
 import H from './constants';

packages/jest-reporters/src/CoverageReporter.ts

@@ -24,7 +24,7 @@ import type {
 } from '@jest/test-result';
 import type {Config} from '@jest/types';
 import {clearLine, isInteractive} from 'jest-util';
-import Worker from 'jest-worker';
+import {Worker} from 'jest-worker';
 import BaseReporter from './BaseReporter';
 import getWatermarks from './getWatermarks';
 import type {

packages/jest-runner/src/__tests__/testRunner.test.ts

@@ -11,8 +11,8 @@ import TestRunner from '../index';
 
 let mockWorkerFarm;
 
-jest.mock('jest-worker', () =>
-  jest.fn(
+jest.mock('jest-worker', () => ({
+  Worker: jest.fn(
     worker =>
       (mockWorkerFarm = {
         end: jest.fn().mockResolvedValue({forceExited: false}),
@@ -21,7 +21,7 @@ jest.mock('jest-worker', () =>
         worker: jest.fn((data, callback) => require(worker)(data, callback)),
       }),
   ),
-);
+}));
 
 jest.mock('../testWorker', () => {});
 

packages/jest-runner/src/index.ts

@@ -12,7 +12,7 @@ import throat from 'throat';
 import type {SerializableError, TestResult} from '@jest/test-result';
 import type {Config} from '@jest/types';
 import {deepCyclicCopy} from 'jest-util';
-import Worker, {PromiseWithCustomMessage} from 'jest-worker';
+import {PromiseWithCustomMessage, Worker} from 'jest-worker';
 import runTest from './runTest';
 import type {SerializableResolver, worker} from './testWorker';
 import type {

packages/jest-worker/README.md

@@ -181,7 +181,7 @@ This example covers the usage with a `computeWorkerKey` method:
 ### File `parent.js`
 
 ```javascript
-import JestWorker from 'jest-worker';
+import {Worker as JestWorker} from 'jest-worker';
 
 async function main() {
   const myWorker = new JestWorker(require.resolve('./Worker'), {

packages/jest-worker/src/__performance_tests__/test.js

@@ -10,7 +10,7 @@
 const assert = require('assert');
 // eslint-disable-next-line import/no-extraneous-dependencies
 const workerFarm = require('worker-farm');
-const JestWorker = require('../../build').default;
+const JestWorker = require('../../build').Worker;
 
 // Typical tests: node --expose-gc test.js empty 100000
 //                node --expose-gc test.js loadTest 10000

packages/jest-worker/src/__tests__/index.test.js

@@ -52,7 +52,7 @@ beforeEach(() => {
     virtual: true,
   });
 
-  Farm = require('..').default;
+  Farm = require('..').Worker;
   Queue = require('../Farm').default;
   WorkerPool = require('../WorkerPool').default;
 });

packages/jest-worker/src/__tests__/process-integration.test.js

@@ -55,7 +55,7 @@ describe('Jest Worker Integration', () => {
       },
     }));
 
-    Farm = require('../index').default;
+    Farm = require('../index').Worker;
   });
 
   afterEach(() => {

packages/jest-worker/src/__tests__/thread-integration.test.js

@@ -56,7 +56,7 @@ describe('Jest Worker Process Integration', () => {
       };
     });
 
-    Farm = require('../index').default;
+    Farm = require('../index').Worker;
   });
 
   afterEach(() => {

packages/jest-worker/src/index.ts

@@ -67,7 +67,7 @@ function getExposedMethods(
  *     processed by the same worker. This is specially useful if your workers
  *     are caching results.
  */
-export default class JestWorker {
+export class Worker {
   private _ending: boolean;
   private _farm: Farm;
   private _options: FarmOptions;