test and patch for cache overriding checked-in node_modules

lib/build.sh

@@ -164,7 +164,6 @@ install_npm() {
 }
 
 function build_dependencies() {
-  restore_cache
 
   if [ "$modules_source" == "" ]; then
     info "Skipping dependencies (no source for node_modules)"
@@ -176,6 +175,7 @@ function build_dependencies() {
     npm install --unsafe-perm --quiet --userconfig $build_dir/.npmrc 2>&1 | indent
 
   else
+    restore_cache
     info "Installing node modules"
     npm install --unsafe-perm --quiet --userconfig $build_dir/.npmrc 2>&1 | indent
   fi

test/run

@@ -201,7 +201,11 @@ testBuildWithUserCacheDirectoriesCamel() {
 }
 
 testModulesCheckedIn() {
-  compile "modules-checked-in"
+  cache=$(mktmpdir)
+  compile "modules-checked-in" $cache
+  assertCapturedSuccess
+
+  compile "modules-checked-in" $cache
   assertCaptured "node_modules source: prebuilt"
   assertCaptured "Rebuilding any native modules for this architecture"
   assertCaptured "(preinstall script)"