Fix angular-busy promise-compare issues and added UT

angular-busy.js

@@ -185,6 +185,16 @@ angular.module('cgBusy').directive('cgBusy',['$compile','$templateCache','cgBusy
 
                     templateScope.$message = options.message;
 
+                    if (options.promise) {
+						// Assign each promise a "id" field, so the equals below will work correctly
+						// Specifically - if we counter a new promise object, reset the tracker.
+                        angular.forEach(options.promise, function(promise) {
+                            if (promise) {
+                                promise.id = promise.id || Math.random();
+                            }
+                        });
+                    }
+
                     if (!angular.equals(tracker.promises,options.promise)) {
                         tracker.reset({
                             promises:options.promise,

dist/angular-busy.js

@@ -185,6 +185,16 @@ angular.module('cgBusy').directive('cgBusy',['$compile','$templateCache','cgBusy
 
                     templateScope.$message = options.message;
 
+                    if (options.promise) {
+						// Assign each promise a "id" field, so the equals below will work correctly
+						// Specifically - if we counter a new promise object, reset the tracker.
+                        angular.forEach(options.promise, function(promise) {
+                            if (promise) {
+                                promise.id = promise.id || Math.random();
+                            }
+                        });
+                    }
+
                     if (!angular.equals(tracker.promises,options.promise)) {
                         tracker.reset({
                             promises:options.promise,

test/spec.js

@@ -67,6 +67,44 @@ describe('cgBusy', function() {
 		expect(this.element.children().css('display')).toBe('none'); //ensure its now invisible as the promise is resolved
 	});
 
+
+	it('should show the overlay when changing an unresolved promise with other unresolved promise', function() {
+
+		this.element = compile('<div cg-busy="my_promise"></div>')(scope);
+		angular.element('body').append(this.element);
+
+		this.testPromise = q.defer();
+		scope.my_promise = this.testPromise.promise;
+
+		//httpBackend.flush();
+
+		scope.$apply();
+
+		expect(this.element.children().length).toBe(2); //ensure the elements are added
+
+		expect(this.element.children().css('display')).toBe('block');//ensure its visible (promise is ongoing)
+
+		// Change promise
+		this.testPromise2 = q.defer();
+		scope.my_promise = this.testPromise2.promise;
+		scope.$apply();
+
+		// Now resolve the first one promise, the overlay should still be visible
+		this.testPromise.resolve();
+		scope.$apply();
+
+		expect(this.element.children().length).toBe(2); //ensure the elements are added
+
+		expect(this.element.children().css('display')).toBe('block');//ensure its visible (promise is ongoing)
+
+		// Now resolve the second promise, and overlay should be removed
+		this.testPromise2.resolve();
+		scope.$apply();
+
+		expect(this.element.children().css('display')).toBe('none'); //ensure its now invisible as the promise is resolved
+
+	});
+
 	it('should load custom templates', function(){
 
 		this.element = compile('<div cg-busy="{promise:my_promise,templateUrl:\'test-custom-template.html\'}"></div>')(scope);
@@ -132,6 +170,6 @@ describe('cgBusy', function() {
 		timeout.flush(101); //1001ms total
 		expect(this.element.children().css('display')).toBe('none');
 
-	});	
+	});
 
 });