Remove es5 builds (#9945)

.circleci/config.yml

@@ -104,17 +104,6 @@ jobs:
       - store_test_results:
           path: reports/junit
 
-  test-browser:
-    working_directory: ~/jest
-    docker:
-      - image: circleci/node:12-browsers
-    steps:
-      - checkout
-      - restore-cache: *restore-cache
-      - run: *install
-      - save-cache: *save-cache
-      - run: yarn test-ci-es5-build-in-browser
-
   test-or-deploy-website:
     working_directory: ~/jest
     docker:
@@ -140,6 +129,5 @@ workflows:
       - test-node-13
       - test-node-14 # current
       - test-jest-circus
-      - test-browser
       - test-or-deploy-website:
           filters: *filter-ignore-gh-pages

.eslintignore

@@ -3,7 +3,6 @@
 bin/
 flow-typed/**
 packages/*/build/**
-packages/*/build-es5/**
 packages/jest-diff/src/cleanupSemantic.ts
 website/blog
 website/build

.eslintrc.js

@@ -95,7 +95,6 @@ module.exports = {
       files: [
         'packages/jest-jasmine2/src/jasmine/**/*',
         'packages/expect/src/jasmineUtils.ts',
-        'e2e/browser-support/browserTest.js',
         '**/vendor/**/*',
       ],
       rules: {

.github/workflows/nodejs.yml

@@ -45,8 +45,6 @@ jobs:
         run: yarn verify-old-ts
       - name: run eslint
         run: yarn lint
-      - name: run eslint on browser builds
-        run: yarn lint-es5-build
       - name: run prettier
         run: yarn lint:prettier:ci
       - name: check copyright headers

.gitignore

@@ -17,7 +17,6 @@
 /node_modules
 
 /packages/*/build/
-/packages/*/build-es5/
 /packages/*/coverage/
 /packages/*/node_modules/
 /packages/*/package-lock.json

CHANGELOG.md

@@ -6,6 +6,8 @@
 
 ### Chore & Maintenance
 
+- `[expect, jest-mock, pretty-format]` [**BREAKING**] Remove `build-es5` from package ([#9945](https://github.com/facebook/jest/pull/9945))
+
 ### Performance
 
 ## 25.5.4

jest.config.js

@@ -23,7 +23,6 @@ module.exports = {
   modulePathIgnorePatterns: [
     'examples/.*',
     'packages/.*/build',
-    'packages/.*/build-es5',
     'packages/jest-runtime/src/__tests__/test_root.*',
     'website/.*',
     'e2e/runtime-internal-module-registry/__mocks__',
@@ -44,7 +43,6 @@ module.exports = {
     '/e2e/global-teardown',
     '\\.snap$',
     '/packages/.*/build',
-    '/packages/.*/build-es5',
     '/packages/.*/src/__tests__/setPrettyPrint.ts',
     '/packages/jest-core/src/__tests__/test_root',
     '/packages/jest-core/src/__tests__/__fixtures__/',

package.json

@@ -59,10 +59,6 @@
     "jest-snapshot-serializer-raw": "^1.1.0",
     "jest-watch-typeahead": "^0.5.0",
     "jquery": "^3.2.1",
-    "karma": "^4.0.1",
-    "karma-chrome-launcher": "^3.0.0",
-    "karma-mocha": "^1.3.0",
-    "karma-webpack": "^4.0.2",
     "lerna": "^3.20.2",
     "make-dir": "^3.0.0",
     "micromatch": "^4.0.2",
@@ -82,7 +78,6 @@
     "tempy": "~0.3.0",
     "throat": "^5.0.0",
     "typescript": "^3.8.2",
-    "webpack": "^4.28.4",
     "which": "^2.0.1"
   },
   "scripts": {
@@ -96,14 +91,12 @@
     "jest": "node ./packages/jest-cli/bin/jest.js",
     "jest-coverage": "yarn jest --coverage",
     "lint": "eslint . --cache --ext js,jsx,ts,tsx,md",
-    "lint-es5-build": "eslint --no-eslintrc --no-ignore --env=browser packages/*/build-es5",
     "lint:prettier": "prettier '**/*.{md,yml,yaml}' 'website/static/**/*.{css,js}' --write --ignore-path .gitignore",
     "lint:prettier:ci": "prettier '**/*.{md,yml,yaml}' 'website/static/**/*.{css,js}' --check --ignore-path .gitignore",
     "postinstall": "opencollective postinstall && yarn build",
     "install-no-ts-build": "node ./scripts/remove-postinstall && yarn --no-progress --frozen-lockfile && node ./scripts/build",
     "remove-prettier-dep": "node ./scripts/remove-prettier-dep",
     "publish": "yarn build-clean && yarn build && lerna publish --silent",
-    "test-ci-es5-build-in-browser": "karma start --single-run",
     "test-ci": "yarn jest-coverage --color -i --config jest.config.ci.js && yarn test-leak && node ./scripts/mapCoverage.js && codecov",
     "test-ci-partial": "yarn jest --color -i --config jest.config.ci.js",
     "test-pretty-format-perf": "node packages/pretty-format/perf/test.js",

packages/expect/package.json

@@ -16,7 +16,6 @@
       ]
     }
   },
-  "browser": "build-es5/index.js",
   "dependencies": {
     "@jest/types": "^25.5.0",
     "ansi-styles": "^4.0.0",

packages/jest-mock/package.json

@@ -25,7 +25,6 @@
       ]
     }
   },
-  "browser": "build-es5/index.js",
   "publishConfig": {
     "access": "public"
   },

packages/pretty-format/package.json

@@ -17,7 +17,6 @@
       ]
     }
   },
-  "browser": "build-es5/index.js",
   "author": "James Kyle <me@thejameskyle.com>",
   "dependencies": {
     "@jest/types": "^25.5.0",

scripts/build.js

@@ -30,11 +30,9 @@ const chalk = require('chalk');
 const micromatch = require('micromatch');
 const prettier = require('prettier');
 const {getPackages, adjustToTerminalWidth, OK} = require('./buildUtils');
-const browserBuild = require('./browserBuild');
 
 const SRC_DIR = 'src';
 const BUILD_DIR = 'build';
-const BUILD_ES5_DIR = 'build-es5';
 const JS_FILES_PATTERN = '**/*.js';
 const TS_FILES_PATTERN = '**/*.ts';
 const IGNORE_PATTERN = '**/__{tests,mocks}__/**';
@@ -72,39 +70,6 @@ function buildNodePackage(p) {
   process.stdout.write(`${OK}\n`);
 }
 
-function buildBrowserPackage(p) {
-  const srcDir = path.resolve(p, SRC_DIR);
-  const pkgJsonPath = path.resolve(p, 'package.json');
-
-  if (!fs.existsSync(pkgJsonPath)) {
-    return;
-  }
-
-  const browser = require(pkgJsonPath).browser;
-  if (browser) {
-    if (browser.indexOf(BUILD_ES5_DIR) !== 0) {
-      throw new Error(
-        `browser field for ${pkgJsonPath} should start with "${BUILD_ES5_DIR}"`,
-      );
-    }
-    let indexFile = path.resolve(srcDir, 'index.js');
-
-    if (!fs.existsSync(indexFile)) {
-      indexFile = indexFile.replace(/\.js$/, '.ts');
-    }
-
-    browserBuild(p.split('/').pop(), indexFile, path.resolve(p, browser))
-      .then(() => {
-        process.stdout.write(adjustToTerminalWidth(`${path.basename(p)}\n`));
-        process.stdout.write(`${OK}\n`);
-      })
-      .catch(e => {
-        console.error(e);
-        process.exit(1);
-      });
-  }
-}
-
 function buildFile(file, silent) {
   const destPath = getBuildPath(file, BUILD_DIR);
 
@@ -187,8 +152,4 @@ if (files.length) {
   const packages = getPackages();
   process.stdout.write(chalk.inverse(' Building packages \n'));
   packages.forEach(buildNodePackage);
-  process.stdout.write('\n');
-
-  process.stdout.write(chalk.inverse(' Building browser packages \n'));
-  packages.forEach(buildBrowserPackage);
 }

tsconfig.json

@@ -29,7 +29,6 @@
     ".yarn/releases/*",
     "**/__tests__/**/*",
     "**/__mocks__/**/*",
-    "**/build/**/*",
-    "**/build-es5/**/*"
+    "**/build/**/*"
   ]
 }