From fbcb7fdd141c277d326dc3ed34545210c4d5628f Mon Sep 17 00:00:00 2001
From: Misko Hevery <misko@hevery.com>
Date: Wed, 22 Feb 2012 11:31:49 -0800
Subject: [PATCH] fix($injector): circular dependency instatiation

---
 src/Injector.js      |  7 ++++++-
 test/InjectorSpec.js | 21 +++++++++++++++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/src/Injector.js b/src/Injector.js
index a9ec6e5db7ea..c24cbb511f18 100644
--- a/src/Injector.js
+++ b/src/Injector.js
@@ -273,7 +273,8 @@ function inferInjectionArgs(fn) {
 
 
 function createInjector(modulesToLoad) {
-  var providerSuffix = 'Provider',
+  var INSTANTIATING = {},
+      providerSuffix = 'Provider',
       path = [],
       loadedModules = new HashMap(),
       providerCache = {
@@ -394,10 +395,14 @@ function createInjector(modulesToLoad) {
         throw Error('Service name expected');
       }
       if (cache.hasOwnProperty(serviceName)) {
+        if (cache[serviceName] === INSTANTIATING) {
+          throw Error('Circular dependency: ' + path.join(' <- '));
+        }
         return cache[serviceName];
       } else {
         try {
           path.unshift(serviceName);
+          cache[serviceName] = INSTANTIATING;
           return cache[serviceName] = factory(serviceName);
         } finally {
           path.shift();
diff --git a/test/InjectorSpec.js b/test/InjectorSpec.js
index df3265801cbb..e0cabbacf276 100644
--- a/test/InjectorSpec.js
+++ b/test/InjectorSpec.js
@@ -483,6 +483,27 @@ describe('injector', function() {
           createInjector([['$injector', myModule]]);
         }).toThrow('Unknown provider: $injector from ' + myModule);
       });
+
+
+      it('should throw error when trying to inject oneself', function() {
+        expect(function() {
+          createInjector([function($provide){
+            $provide.factory('service', function(service){});
+            return function(service) {}
+          }])
+        }).toThrow('Circular dependency: service');
+      });
+
+
+      it('should throw error when trying to inject circular dependency', function() {
+        expect(function() {
+          createInjector([function($provide){
+            $provide.factory('a', function(b){});
+            $provide.factory('b', function(a){});
+            return function(a) {}
+          }])
+        }).toThrow('Circular dependency: b <- a');
+      });
     });
   });