Added spec to cover subscribing arrays

Author: Web Advanced

dev/signals.js

@@ -54,22 +54,24 @@ var signals = (function (global, undefined) {
 	* @private
 	* @name subscribeToSignal
 	* @param {string} [signalName] Name of the signal. (default = 'any').
-	* @param {Function} [fn] Callback function bound to the signal.
+	* @param {Object} [callbacks] Callback function or Array of functions to be bound to the signal.
 	* @param {Object} [context] Context on which listener will be executed (object that should represent the `this`. (default = window)
 	*/
-    subscribeToSignal = function (signalName, fn, context) {
+    subscribeToSignal = function (signalName, callbacks, context) {
 		var signalType = getSignalType(signalName), 
 			i = 0, 
-			l;
-		if (fn instanceof Array) {
-			l = fn.length;
-			for (i; i < l; i++){
-				checkArgument(fn, FUNCTION, 'Callback must be a function');	
-				evts[signalType].push({callback: fn[i], obj: context || global});
+			l,
+			callback;
+		if (callbacks instanceof Array) {
+			l = callbacks.length;
+			for (i; i < l; i++) {
+				callback = callbacks[i];
+				checkArgument(callback, FUNCTION, 'Callback must be a function');	
+				evts[signalType].push({callback: callback, obj: context || global});
 			}
 		} else {
-			checkArgument(fn, FUNCTION, 'Callback must be a function');
-			evts[signalType].push({callback: fn, obj: context || global});
+			checkArgument(callbacks, FUNCTION, 'Callback must be a function');
+			evts[signalType].push({callback: callbacks, obj: context || global});
 		}
     };
 	/**

dev/specs/signals.specs.js

@@ -10,6 +10,9 @@ describe("Signals JS", function () {
 		flag_2 = arg.flag_2;
 		mockString = arg.mockString;
 	}
+    function func_1() {};
+    function func_2() {};
+    function func_3() {};
     beforeEach(function () {
         falg_1 = false;
         falg_2 = false;
@@ -29,6 +32,11 @@ describe("Signals JS", function () {
         expect(signals.isObservable('mock:event')).toEqual(true);
     });
 
+    it("should be able to subscribe a collection of funcs to mock:collection", function() {
+        signals.subscribe('mock:collection', [func_1, func_2, func_3]);
+        expect(signals.listenerCount('mock:collection')).toEqual(3);
+    });
+
     it("should throw when adding anything other then a function as the callback", function() {
         var failingFunc = function () {
             signals.subscribe('mock:event', 'fakeFunc');