Merge pull request #51 from creynders/fix_51

BUG: Singleton (backbone) instances resolved twice

Author: geekdave

backbone.geppetto.js

@@ -124,10 +124,8 @@
 
         wireSingleton: function(key, clazz, wiring) {
 
-            var constructor = (clazz.prototype.initialize ? this._wrapConstructor(clazz, wiring) : clazz);
-
             this._mappings[key] = {
-                clazz: constructor,
+                clazz: clazz,
                 object: null,
                 type: TYPES.SINGLETON,
                 wiring: wiring

dist/backbone.geppetto.min.js

@@ -363,6 +363,24 @@ define([
                 throw (/no mapping found/);
             });
         });
+        describe('when used with Backbone objects', function(){
+            var clazzInstantiated;
+            var clazz = function(){
+                clazzInstantiated++;
+            };
+            var singleton = Backbone.Model.extend({
+                wiring : ['clazz']
+            });
+            beforeEach(function(){
+                clazzInstantiated=0;
+                resolver.wireClass('clazz', clazz);
+                resolver.wireSingleton('singleton', singleton);
+            });
+            it("should not resolve singleton dependencies twice, see #51", function(){
+                var actual = resolver.getObject('singleton');
+                expect(clazzInstantiated ).to.equal(1);
+            });
+        });
     });
 
 });