From da423b8b4d08b9afac5ea40c940a1602f03880b5 Mon Sep 17 00:00:00 2001
From: Ward Bell <wardbell@hotmail.com>
Date: Fri, 14 Nov 2014 23:34:26 -0800
Subject: [PATCH] Tweak $location decorator and visit(); all tests pass. -
 Guard against infinite $location.path loop as explained in comment - Give
 visit() one more tick before it invokes the callback; determined empirically
 that this is necessary on occasion else the viewElement() may return empty
 object

---
 src/ngMidwayTester.js | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/src/ngMidwayTester.js b/src/ngMidwayTester.js
index 08216b6..8b36978 100644
--- a/src/ngMidwayTester.js
+++ b/src/ngMidwayTester.js
@@ -37,8 +37,12 @@
         var _path = $delegate.path();
         $delegate.path = function(path) {
           if(path) {
-            _path = path;
-            $rootScope.$broadcast('$locationChangeSuccess', path);
+            // sometimes the broadcast triggers a new request for same path
+            // added this conditional to mitigate risk of this infinite loop
+            if (_path !== path) {
+              _path = path;
+              $rootScope.$broadcast('$locationChangeSuccess', path);                           
+            } 
             return this;
           }
           else {
@@ -220,7 +224,10 @@
       this.rootScope().__view_status = ++$viewCounter;
       this.until(function() {
         return parseInt($terminalElement.attr('status')) >= $viewCounter;
-      }, callback || noop);
+      }, function(){ 
+        // give it another tick to settle
+        setTimeout(callback || noop, 0)
+      });
 
       var $location = this.inject('$location');
       this.apply(function() {