feat(check): printing error if trying to upgrade non-existent module, closes #78

Author: bahmutov

src/dependencies.js

@@ -118,8 +118,10 @@ function getDependenciesToCheck(options, moduleNames) {
             });
         });
         if (!options.tldr) {
-            console.log('only checking');
-            console.json(toCheck);
+            console.log('only checking modules');
+            console.log(toCheck.map(function (m) {
+                return m.name + '@' + m.version;
+            }));
         }
     }
     return toCheck;

src/next-update.js

@@ -76,6 +76,16 @@ function checkCurrentInstall(options) {
 
 var isOnline = Q.denodeify(require('is-online'));
 
+function makeSureValidModule(moduleNames, checkingModules) {
+    la(check.array(moduleNames), 'expected list of modules', moduleNames);
+    la(check.array(checkingModules), 'expected list of modules to check', checkingModules);
+    if (moduleNames.length === 1 && check.empty(checkingModules)) {
+        console.error('Could not find module "%s" in the list of dependencies', moduleNames[0]);
+        console.error('Please check the name');
+        process.exit(-1);
+    }
+}
+
 // returns promise
 function checkAllUpdates(options) {
     options = options || {};
@@ -101,6 +111,8 @@ function checkAllUpdates(options) {
     var toCheck = getDependenciesToCheck(options, moduleName);
     check.verify.array(toCheck, 'dependencies to check should be an array');
 
+    makeSureValidModule(moduleName, toCheck);
+
     var testVersionsBound = testVersions.bind(null, {
         modules: toCheck,
         command: checkCommand,

test/e2e.coffee

@@ -10,6 +10,7 @@ gt.module 'end 2 end tests',
   teardown: ->
     process.chdir __dirname
 
+# TODO set the final version limit to check
 gt.async '--module', ->
     gt.exec 'node', [index, '--module', moduleName], 0
 , ONE_MINUTE