fix($$testability): Call notifiyWhenNoOutstandingRequests async

Services such as $location and $http will initiate async calls. Thus,
we need to wrap the call to $browser.notifyWhenNoOutstandingRequest
in $timeout() in order to give any potentially outstanding async calls a
chance to run.

Fixes angular#13782

Author: heathkit

src/ng/testability.js

@@ -2,8 +2,8 @@
 
 
 function $$TestabilityProvider() {
-  this.$get = ['$rootScope', '$browser', '$location',
-       function($rootScope,   $browser,   $location) {
+  this.$get = ['$rootScope', '$browser', '$location', '$window',
+       function($rootScope,   $browser,   $location, $window) {
 
     /**
      * @name $testability
@@ -109,7 +109,11 @@ function $$TestabilityProvider() {
      * @param {function} callback
      */
     testability.whenStable = function(callback) {
-      $browser.notifyWhenNoOutstandingRequests(callback);
+      // Register the callback asynchronously, so that services that trigger
+      // async callbacks have a chance to run.
+      $rootScope.$evalAsync(function() {
+        $browser.notifyWhenNoOutstandingRequests(callback);
+      });
     };
 
     return testability;

test/ng/testabilitySpec.js

@@ -189,10 +189,35 @@ describe('$$testability', function() {
 
   describe('waiting for stability', function() {
     it('should process callbacks immediately with no outstanding requests',
-      inject(function($$testability) {
+      inject(function($$testability, $timeout) {
         var callback = jasmine.createSpy('callback');
-        $$testability.whenStable(callback);
-        expect(callback).toHaveBeenCalled();
+        runs(function() {
+          $$testability.whenStable(callback);
+          $timeout.flush();
+        });
+
+        waitsFor(function() {
+          return callback.calls.length > 0;
+        }, "The callback should be called.", 500);
+      }));
+
+    it('should wait for new $http calls asynchronously',
+      inject(function($$testability, $httpBackend, $http, $timeout) {
+        var callback = jasmine.createSpy('callback');
+        runs(function() {
+          $httpBackend.when('GET').respond(200);
+
+          $http.get('');
+          $$testability.whenStable(callback);
+          expect(callback).not.toHaveBeenCalled();
+
+          $httpBackend.flush();
+          $timeout.flush();
+        });
+
+        waitsFor(function() {
+          return callback.calls.length > 0;
+        }, "The callback should be called.", 500);
       }));
   });
 });