edit tests

Author: Unitech

lib/API/Extra.js

@@ -326,6 +326,12 @@ module.exports = function(CLI) {
           Common.printError(err);
           return cb ? cb(Common.retErr(err)) : that.exitCli(cst.ERROR_EXIT);
         }
+
+        if (data.process_count == 0) {
+          Common.printError('Not any process has received a command (offline or unexistent)');
+          return cb ? cb(Common.retErr('Unknown process')) : that.exitCli(cst.ERROR_EXIT);
+        }
+
         process_wait_count = data.process_count;
         Common.printOut(chalk.bold('%s processes have received command %s'),
                         data.process_count, action_name);

lib/API/Keymetrics/cli-api.js

@@ -54,11 +54,13 @@ module.exports = function(CLI) {
   CLI.prototype.monitorState = function(state, target, cb) {
     var that = this;
 
-    try {
-      fs.statSync(this._conf.INTERACTION_CONF);
-    } catch(e) {
-      printMotd();
-      return registerPrompt();
+    if (process.env.NODE_ENV !== 'test') {
+      try {
+        fs.statSync(this._conf.INTERACTION_CONF);
+      } catch(e) {
+        printMotd();
+        return registerPrompt();
+      }
     }
 
     if (!target) {

lib/God/ActionMethods.js

@@ -733,9 +733,14 @@ module.exports = function(God) {
       var sent = 0;
 
       (function ex(arr) {
-        if (arr[0] == null) return cb(null, { process_count : sent, success : true });
+        if (arr[0] == null || !arr) {
+          return cb(null, {
+            process_count : sent,
+            success : true
+          });
+        }
 
-        var id      = arr[0];
+        var id = arr[0];
 
         if (!God.clusters_db[id] || !God.clusters_db[id].pm2_env) {
           arr.shift();
@@ -744,19 +749,30 @@ module.exports = function(God) {
 
         var proc_env = God.clusters_db[id].pm2_env;
 
-        if (p.basename(proc_env.pm_exec_path) == name || proc_env.name == name) {
-          if (proc_env.status == cst.ONLINE_STATUS || proc.pm2_env.status == cst.LAUNCHING_STATUS) {
+        if ((p.basename(proc_env.pm_exec_path) == name ||
+             proc_env.name == name) &&
+            (proc_env.status == cst.ONLINE_STATUS ||
+             proc_env.status == cst.LAUNCHING_STATUS)) {
 
-            if (cmd.opts == null)
-              God.clusters_db[id].send(cmd.msg);
-            else
-              God.clusters_db[id].send(cmd);
+          proc_env.axm_actions.forEach(function(action) {
+            if (action.action_name == cmd.msg) {
+              action_exist = true;
+            }
+          });
 
-            sent++;
+          if (action_exist == false || proc_env.axm_actions.length == 0) {
             arr.shift();
             return ex(arr);
-
           }
+
+          if (cmd.opts == null)
+            God.clusters_db[id].send(cmd.msg);
+          else
+            God.clusters_db[id].send(cmd);
+
+          sent++;
+          arr.shift();
+          return ex(arr);
         }
         else {
           arr.shift();

test/interface/monitor.mocha.js

@@ -1,5 +1,7 @@
 /* eslint-env mocha */
 
+process.env.NODE_ENV='test';
+
 'use strict';
 
 var pm2 = require('../../index.js');

test/programmatic/custom_action.mocha.js

@@ -41,6 +41,13 @@ describe('Custom actions via CLI/API', function() {
     });
   });
 
+  it('should handle unknown application', function(done) {
+    pm2.trigger('indexxo', 'ping', function(err, ret) {
+      should(err).not.be.null();
+      done();
+    });
+  });
+
   it('should cannot trigger message if unknow id', function(done) {
     pm2.trigger(10, 'ping', function(err, ret) {
       should(err).not.be.null();

test/programmatic/lazy_api.mocha.js

@@ -8,10 +8,6 @@ var PM2 = require('../..');
 var should = require('should');
 
 describe('Lazy API usage', function() {
-  before(function(done) {
-    PM2.kill(function() { done() });
-  });
-
   it('should start a script without passing any args', function(done) {
     PM2.start('./../fixtures/child.js', done);
   });