Merge pull request #25 from Spy-Seth/feature/singleton-by-default

Service are now singleton by default

src/ContainerFactory.js

@@ -43,7 +43,7 @@ var buildServiceDefinitionCollection = function buildServiceDefinitionCollection
             value.name,
             value.service,
             new FunctionArgumentCollection(functionArgumentList),
-            value.singleton || undefined,
+            value.singleton,
             new CallCollection(callList)
         ));
     });

src/ServiceDefinition.js

@@ -14,7 +14,7 @@ var ServiceDefinition = function ServiceDefinition(name, serviceConstructor, fun
     this.name = name;
     this.serviceConstructor = serviceConstructor;
     this.functionArgumentCollection = functionArgumentCollection;
-    this.isSingletonService = !!isSingletonService;
+    this.isSingletonService = (isSingletonService !== undefined ? !!isSingletonService : true);
     this.callCollection = callCollection;
 };
 

tests/fixture/valid/ServiceE.js

@@ -0,0 +1,11 @@
+'use strict';
+
+/**
+ * A service with no dependency.
+ *
+ * @constructor
+ */
+var ServiceE = function ServiceE() {
+};
+
+module.exports = ServiceE;

tests/fixture/valid/services.js

@@ -42,6 +42,10 @@ module.exports = {
                     51
                 ]
             }
+        },
+        {
+            'name': 'foo.serviceE',
+            'service': require('./ServiceE')
         }
     ]
 };

tests/functionals/ContainerFactorySpec.js

@@ -42,6 +42,15 @@ describe('ContainerFactory', function () {
         expect(firstCall).to.not.be.equal(secondCall);
     });
 
+    it('should concider a service as singleton if it have no singleton configuration', function () {
+        var container = ContainerFactory.create(servicesConfigurationValid.services, servicesConfigurationValid.parameters);
+
+        var firstCall = container.getService('foo.serviceE');
+        var secondCall = container.getService('foo.serviceE');
+
+        expect(firstCall).to.equals(secondCall);
+    });
+
     it('should throw an exception on undefined parameter', function () {
         var container = ContainerFactory.create(servicesConfigurationValid.services, servicesConfigurationValid.parameters);