Last necessary test

lib/executor.js

@@ -28,7 +28,7 @@ function Executor () {
         }
         lastArg.apply(null, arguments);
       };
-    } else if (!lastArg && task.arguments.length) {
+    } else if (!lastArg && task.arguments.length !== 0) {
       // false/undefined/null supplied as callbback
       newArguments[newArguments.length - 1] = function () { cb(); };
     } else {

test/executor.test.js

@@ -99,8 +99,6 @@ function testRightOrder (d, done) {
   });
 }
 
-
-
 // Note:  The following test does not have any assertion because it
 // is meant to address the deprecation warning:
 // (node) warning: Recursive process.nextTick detected. This will break in the next version of node. Please use setImmediate for recursive deferral.
@@ -116,7 +114,18 @@ var testEventLoopStarvation = function(d, done){
    done();
 };
 
-describe('Executor', function () {
+// Test that operations are executed in the right order even with no callback
+function testExecutorWorksWithoutCallback (d, done) {
+  d.insert({ a: 1 });
+  d.insert({ a: 2 }, false);
+  d.find({}, function (err, docs) {
+    docs.length.should.equal(2);
+    done();
+  });
+}
+
+
+describe.only('Executor', function () {
 
   describe('With persistent database', function () {
     var d;
@@ -162,6 +171,10 @@ describe('Executor', function () {
       testEventLoopStarvation(d, done);
     });
 
+    it('Works in the right order even with no supplied callback', function(done){
+      testExecutorWorksWithoutCallback(d, done);
+    });
+
   });   // ==== End of 'With persistent database' ====
 
 
@@ -191,6 +204,10 @@ describe('Executor', function () {
       testRightOrder(d, done);
     });
 
+    it('Works in the right order even with no supplied callback', function(done){
+      testExecutorWorksWithoutCallback(d, done);
+    });
+
   });   // ==== End of 'With non persistent database' ====
 
 });