From 970cde5db406ed8be2976f3376dfd62da3467b09 Mon Sep 17 00:00:00 2001
From: Austen Collins <austencollins@gmail.com>
Date: Mon, 7 Sep 2015 13:03:10 -0700
Subject: [PATCH] generate: remind users to do npm install, if necessary

---
 lib/commands/generate.js | 14 ++++++++++++--
 tests/all.js             | 10 +++++-----
 2 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/lib/commands/generate.js b/lib/commands/generate.js
index 3767c3958..349cf557c 100644
--- a/lib/commands/generate.js
+++ b/lib/commands/generate.js
@@ -26,6 +26,7 @@ var skeletonData = {
   isLambda: false,
   isApi: false,
   handlerPath: false,
+  isNew: false
 };
 
 /**
@@ -190,11 +191,15 @@ function _createSkeleton() {
             utils.writeFile(path.join(fullLambdaDir, 'event.json'), '{}')
         );
 
-        if (!fs.existsSync(targetPackageJsonPath)) {  //Don't make package json if one already exists
-          utils.logIfVerbose('creating package.json as it does not exist');
+        // Make package.json, if it doesn't exist already
+        if (!fs.existsSync(targetPackageJsonPath)) {
+          utils.logIfVerbose('creating package.json since it does not exist');
           writeFilesDeferred.push(
               utils.writeFile(targetPackageJsonPath, JSON.stringify(packageJson, null, 2))
           );
+
+          // Remind user to npm install
+          skeletonData.isNew = true;
         }
 
         break;
@@ -218,5 +223,10 @@ module.exports.generate = function(isLambda, isApi, functionName, resourceName,
       .then(function() {
         // Success message
         console.log('Successfully created new folders and files in your "back/lambdas" folder.');
+
+        // npm install reminder
+        if (skeletonData.isNew) {
+          console.log('Remember to run \'npm install\' in your "back" directory.');
+        }
       });
 };
diff --git a/tests/all.js b/tests/all.js
index 28207a73f..2ec03cd6c 100644
--- a/tests/all.js
+++ b/tests/all.js
@@ -14,15 +14,15 @@ describe('AllTests', function() {
   });
 
   //require tests vs inline so we can run sequentially
-  //require('./cli/tag');
-  //require('./cli/install');
-  //require('./cli/env');
-  //require('./cli/generate');
+  require('./cli/tag');
+  require('./cli/install');
+  require('./cli/env');
+  require('./cli/generate');
 
   /**
    * Tests below create AWS Resources
    */
-  require('./cli/deploy_lambda');
+  //require('./cli/deploy_lambda');
   //require('./cli/deploy_api');
   //require('./cli/new');           //Must be run last
 });