Rename MockMessaging.nextResult() to respondNext()

Author: raman-nbg

src/messaging.js

@@ -60,17 +60,20 @@ MockMessaging.prototype.autoFlush = function (delay) {
 };
 
 MockMessaging.prototype.failNext = function(methodName, err) {
+  assert(_.isString(methodName), 'methodName must be a string');
   assert(err instanceof Error, 'err must be an "Error" object');
   this.results[methodName] = err;
 };
 
-MockMessaging.prototype.nextResult = function(methodName, result) {
+MockMessaging.prototype.respondNext = function(methodName, result) {
+  assert(_.isString(methodName), 'methodName must be a string');
   assert(!_.isUndefined(result), 'result must not be undefined');
   assert(!(result instanceof Error), 'result must not be an "Error" object');
   this.results[methodName] = result;
 };
 
 MockMessaging.prototype.on = function(methodName, callback) {
+  assert(_.isString(methodName), 'methodName must be a string');
   assert(_.isFunction(callback), 'callback must be a function');
   this.callbacks[methodName] = callback;
 };
@@ -101,9 +104,9 @@ MockMessaging.prototype._invokeOn = function(methodName, args) {
 };
 
 MockMessaging.prototype._nextResult = function(type) {
-  var err = this.results[type];
+  var result = this.results[type];
   delete this.results[type];
-  return err || null;
+  return result || null;
 };
 
 MockMessaging.prototype._defer = function(sourceMethod, sourceArgs, callback) {

test/unit/messaging.js

@@ -88,7 +88,7 @@ describe('MockMessaging', function() {
     });
 
     it('can return user defined results', function(done) {
-      messaging.nextResult('send', 'the result');
+      messaging.respondNext('send', 'the result');
       var thenable = messaging.send({message: 'foo'});
       messaging.flush();
       thenable.then(function(result) {
@@ -152,7 +152,7 @@ describe('MockMessaging', function() {
     });
 
     it('can return user defined results', function(done) {
-      messaging.nextResult('sendAll', 'the result');
+      messaging.respondNext('sendAll', 'the result');
       var thenable = messaging.sendAll([{message: 'foobar'}]);
       messaging.flush();
       thenable.then(function(result) {
@@ -223,7 +223,7 @@ describe('MockMessaging', function() {
     });
 
     it('can return user defined results', function(done) {
-      messaging.nextResult('sendMulticast', 'the result');
+      messaging.respondNext('sendMulticast', 'the result');
       var thenable = messaging.sendMulticast({message: 'foobar', tokens: ['t1']});
       messaging.flush();
       thenable.then(function(result) {