Fix #69. Allow compose to pass full argument list through to first fu…

…nction

lib/functional.js

@@ -15,7 +15,9 @@
 define(['when'], function (when) {
 "use strict";
 
-	var slice = [].slice;
+	var slice, undef;
+
+	slice = [].slice;
 
 	/**
 	 * Create a partial function
@@ -84,17 +86,17 @@ define(['when'], function (when) {
 	 * @param funcs {Array} array of functions to compose
 	 * @return {Function} composed function
 	 */
-	function compose(funcs /*, ...*/) {
+	function compose(funcs) {
 
-		if(arguments.length > 1) {
-			funcs = slice.call(arguments);
-		}
+		var first;
+		first = funcs[0];
+		funcs = funcs.slice(1);
 
-		return function composed(x) {
+		return function composed() {
 			var context = this;
 			return funcs.reduce(function(result, f) {
 				return f.call(context, result);
-			}, x);
+			}, first.apply(this, arguments));
 		};
 	}
 

test/node/base/functional.js

@@ -8,7 +8,11 @@ refute = buster.refute;
 fail = buster.assertions.fail;
 
 function plusOne(x) {
-	return x+1;
+	return plus(x, 1);
+}
+
+function plus(x, y) {
+	return x + y;
 }
 
 function Thing(x) {
@@ -55,6 +59,20 @@ buster.testCase('base:functional', {
 			).then(done, done);
 		},
 
+		'should allow multiple args when composing a string specification': function(done) {
+			wire({
+				f1: plus,
+				f2: plusOne,
+				composed: {
+					compose: 'f1 | f2'
+				}
+			}).then(
+				function(c) {
+					assert.equals(c.composed(1, 2), 4);
+				}
+			).then(done, done);
+		},
+
 		'should compose a string specification with single function': function(done) {
 			wire({
 				f1: plusOne,

test/node/base/module.js

@@ -11,9 +11,9 @@ buster.testCase('base:module', {
 	'should use module exports value as component': function(done) {
 		wire({
 			test: {
-				module: 'test/node/fixtures/module'
+				module: '../fixtures/module'
 			}
-		}).then(
+		}, { require: require }).then(
 			function(context) {
 				assert(context.test.success);
 			},