From ebddb83dd6e07a25474ab8dd24dd3a1cccd22611 Mon Sep 17 00:00:00 2001
From: Joe Haddad <timer150@gmail.com>
Date: Thu, 11 Jan 2018 01:40:03 -0500
Subject: [PATCH] Remove redundant steps in e2e tests (#3747)

* This doesn't look needed anymore

* Remove unnecessary rebuilds
---
 tasks/e2e-installs.sh    | 12 +-----------
 tasks/e2e-kitchensink.sh | 12 +-----------
 tasks/e2e-simple.sh      |  9 +--------
 3 files changed, 3 insertions(+), 30 deletions(-)

diff --git a/tasks/e2e-installs.sh b/tasks/e2e-installs.sh
index 3a40a78cb..73ca1c04b 100755
--- a/tasks/e2e-installs.sh
+++ b/tasks/e2e-installs.sh
@@ -73,18 +73,8 @@ then
   npm cache clean || npm cache verify
 fi
 
-# Prevent bootstrap, we only want top-level dependencies
-cp package.json package.json.bak
-grep -v "postinstall" package.json > temp && mv temp package.json
+# Bootstrap monorepo
 yarn
-mv package.json.bak package.json
-
-# We removed the postinstall, so do it manually
-node bootstrap.js
-
-cd packages/react-error-overlay/
-yarn run build:prod
-cd ../..
 
 # ******************************************************************************
 # First, publish the monorepo.
diff --git a/tasks/e2e-kitchensink.sh b/tasks/e2e-kitchensink.sh
index 709d82753..20c6dbb05 100755
--- a/tasks/e2e-kitchensink.sh
+++ b/tasks/e2e-kitchensink.sh
@@ -65,18 +65,8 @@ then
   npm cache clean || npm cache verify
 fi
 
-# Prevent bootstrap, we only want top-level dependencies
-cp package.json package.json.bak
-grep -v "postinstall" package.json > temp && mv temp package.json
+# Bootstrap monorepo
 yarn
-mv package.json.bak package.json
-
-# We removed the postinstall, so do it manually
-node bootstrap.js
-
-cd packages/react-error-overlay/
-yarn build:prod
-cd ../..
 
 # ******************************************************************************
 # First, publish the monorepo.
diff --git a/tasks/e2e-simple.sh b/tasks/e2e-simple.sh
index b162adc92..44ee14fb1 100755
--- a/tasks/e2e-simple.sh
+++ b/tasks/e2e-simple.sh
@@ -77,11 +77,8 @@ then
   npm cache clean || npm cache verify
 fi
 
-# Prevent bootstrap, we only want top-level dependencies
-cp package.json package.json.bak
-grep -v "postinstall" package.json > temp && mv temp package.json
+# Bootstrap monorepo
 yarn
-mv package.json.bak package.json
 
 # Start local registry
 tmp_registry_log=`mktemp`
@@ -96,9 +93,6 @@ yarn config set registry http://localhost:4873
 # Login so we can publish packages
 npx npm-cli-login@0.0.10 -u user -p password -e user@example.com -r http://localhost:4873 --quotes
 
-# We removed the postinstall, so do it manually here
-node bootstrap.js
-
 # Lint own code
 ./node_modules/.bin/eslint --max-warnings 0 packages/babel-preset-react-app/
 ./node_modules/.bin/eslint --max-warnings 0 packages/create-react-app/
@@ -108,7 +102,6 @@ node bootstrap.js
 cd packages/react-error-overlay/
 ./node_modules/.bin/eslint --max-warnings 0 src/
 yarn test
-yarn build:prod
 cd ../..
 cd packages/react-dev-utils/
 yarn test
-- 
GitLab