[Groonga-commit] groonga/express-kotoumi [master] Mocked callback should accept one or two arguments

Back to archive index

YUKI Hiroshi null+****@clear*****
Wed Jan 23 17:33:29 JST 2013


YUKI Hiroshi	2013-01-23 17:33:29 +0900 (Wed, 23 Jan 2013)

  New Revision: 6da2b1625fead5a89bb9d4d9005f13854fc9e020
  https://github.com/groonga/express-kotoumi/commit/6da2b1625fead5a89bb9d4d9005f13854fc9e020

  Log:
    Mocked callback should accept one or two arguments

  Modified files:
    test/backend-adaptor.test.js
    test/test-utils.js

  Modified: test/backend-adaptor.test.js (+2 -2)
===================================================================
--- test/backend-adaptor.test.js    2013-01-23 17:07:11 +0900 (9d975db)
+++ test/backend-adaptor.test.js    2013-01-23 17:33:29 +0900 (cacedd2)
@@ -52,12 +52,12 @@ suite('Connection', function() {
       type:       'testResponse',
       body:       'first call'
     };
-    callback.takes(null, message);
+    callback.takes(message);
     receiver.emitMessage(message);
     callback.assert();
 
     message.body = 'second call';
-    callback.takes(null, message);
+    callback.takes(message);
     receiver.emitMessage(message);
     callback.assert();
 

  Modified: test/test-utils.js (+5 -6)
===================================================================
--- test/test-utils.js    2013-01-23 17:07:11 +0900 (28fafe6)
+++ test/test-utils.js    2013-01-23 17:33:29 +0900 (3bfb768)
@@ -54,16 +54,15 @@ exports.createMockedReceiver = createMockedReceiver;
 
 function createMockedMessageCallback() {
   var mockedCallback = nodemock;
-  var callback = function(error, message) {
-    mockedCallback.receive(error, message);
+  var callback = function() {
+    mockedCallback.receive.apply(mockedCallback, arguments);
   };
-  callback.takes = function(error, message) {
+  callback.takes = function() {
     callback.assert = function() {
       mockedCallback.assertThrows();
     };
-    mockedCallback = mockedCallback
-                       .mock('receive')
-                       .takes(error, message);
+    mockedCallback = mockedCallback.mock('receive');
+    mockedCallback = mockedCallback.takes.apply(mockedCallback, arguments);
   };
   callback.mock = mockedCallback;
   return callback;
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index