From fdd12e15e9aedce56e6ddfd66aaa1d387d4be5fa Mon Sep 17 00:00:00 2001 From: Ben Newman Date: Wed, 21 Jun 2017 20:35:47 -0400 Subject: [PATCH] Stop clearing the npm cache unnecessarily before tests. Based on this warning: npm ERR! As of npm@5, the npm cache self-heals from corruption issues and npm ERR! data extracted from the cache is guaranteed to be valid. If you npm ERR! want to make sure everything is consistent, use 'npm cache npm ERR! verify' instead. npm ERR! npm ERR! If you're sure you want to delete the entire cache, rerun this npm ERR! command with --force. --- packages/test-in-console/run.sh | 3 --- scripts/ci.sh | 3 --- 2 files changed, 6 deletions(-) diff --git a/packages/test-in-console/run.sh b/packages/test-in-console/run.sh index 0509632277..f2ef762a1b 100755 --- a/packages/test-in-console/run.sh +++ b/packages/test-in-console/run.sh @@ -4,9 +4,6 @@ cd "`dirname "$0"`" cd ../.. export METEOR_HOME=`pwd` -# Clear dev_bundle/.npm to ensure consistent test runs. -./meteor npm cache clear - # Just in case these packages haven't been installed elsewhere. ./meteor npm install -g phantomjs-prebuilt browserstack-webdriver diff --git a/scripts/ci.sh b/scripts/ci.sh index 5958c87d4a..d9a079eb9c 100755 --- a/scripts/ci.sh +++ b/scripts/ci.sh @@ -35,9 +35,6 @@ if [ -z "$CIRCLE_NODE_TOTAL" ] || [ -z "$CIRCLE_NODE_INDEX" ]; then echo "Running all tests!" fi -# Clear dev_bundle/.npm to ensure consistent test runs. -./meteor npm cache clear - # Since PhantomJS has been removed from dev_bundle/lib/node_modules # (#6905), but self-test still needs it, install it now. ./meteor npm install -g phantomjs-prebuilt browserstack-webdriver