Merged in ksd-243 (pull request #49)

KSD-243 remove arguments shifting from queue

lib/run.js

@@ -93,7 +93,7 @@ function collectResultsFromFrames(context, options, callback) {
 		var node = frame.node,
 			win = node.contentWindow;
 
-		q.defer(function (_, done) {
+		q.defer(function (done) {
 
 			var timeout = setTimeout(function () {
 				done({});
@@ -124,7 +124,7 @@ function collectResultsFromFrames(context, options, callback) {
 					}
 				});
 			});
-		}, node);
+		});
 	}
 
 	for (var i = 0, l = frames.length; i < l; i++) {
@@ -150,7 +150,9 @@ dqre.run = function run(context, options, callback) {
 		var q = utils.queue();
 
 		if (context.frames.length) {
-			q.defer(collectResultsFromFrames, context, options);
+			q.defer(function (done) {
+				collectResultsFromFrames(context, options, done);
+			});
 		}
 		q.defer(function (cb) {
 			dqre.audit.run(context, options, cb);

lib/utils/queue.js

@@ -1,7 +1,5 @@
 (function () {
 	'use strict';
-
-	var slice = [].slice;
 	function noop() {}
 
 	/**
@@ -45,8 +43,8 @@
 			 * First parameter should be the function to execute with subsequent
 			 * parameters being passed as arguments to that function
 			 */
-			defer: function () {
-				tasks.push(slice.call(arguments));
+			defer: function (fn) {
+				tasks.push([fn]);
 				++remaining;
 				pop();
 			},

test/unit/utils/queue.js

@@ -53,21 +53,6 @@ describe('utils.queue', function () {
 
 		});
 
-		it('should allow parameters to be passed', function (done) {
-			var q = utils.queue(),
-				expected = { monkeys: 'bananas' };
-
-			q.defer(function (data, callback) {
-				assert.deepEqual(data, expected);
-				callback(data);
-			}, expected);
-
-			q.then(function (data) {
-				assert.deepEqual(data, [expected]);
-				done();
-			});
-		});
-
 	});
 
 	describe('then', function () {