Adds test case for singleton constructor that don't implement dispose method

Author: cesarenaldi

README.md

@@ -20,7 +20,7 @@
 	- [x] remove done call
 	- [x] add experimental contribution phase into a revealing construction pattern
 	- [x] refactor APIs to be more explicit
-- [ ] test case when singletons do NOT implement a dispose method (see test coverage)
+- [x] test case when singletons do NOT implement a dispose method (see test coverage)
 - [x] cleanup
 	- [x] remove memoize if not used
 	- [x] remove merge if not used

test/spec/dispose.spec.js

@@ -56,7 +56,7 @@ describe('Given a container', function () {
 			BAR_2.dispose.reset()
 		})
 
-		it('should not release the instance if still in use', function () {
+		it('should not dispose the instance if still in use', function () {
 			var firstRequest = container.get('bar')
 			var secondRequest = container.get('bar')
 
@@ -69,6 +69,27 @@ describe('Given a container', function () {
 			expect(container.get('bar')).to.equal(firstRequest)
 		})
 
+		describe('and the instance does not have a dispose method', function () {
+
+			class Foo {}
+
+			beforeEach(function () {
+				container = iniettore.create(function (context) {
+					context
+						.map('foo').to(Foo)
+						.as(TRANSIENT, SINGLETON, CONSTRUCTOR)
+				})
+				container.get('foo')
+			})
+
+			it('should not throw an Error', function () {
+				function testCase() {
+					container.release('foo')
+				}
+				expect(testCase).to.not.throw(TypeError)
+			})
+		})
+
 		describe('the same amount of times it has been acquired', function () {
 
 			it('should release the instance, call dispose() method', function () {
@@ -117,7 +138,7 @@ describe('Given a container', function () {
 		})
 	})
 
-	describe('when releasing a persistent singleton', function () {
+	describe('when releasing a persistent singleton adn there is no more reference to the singleton', function () {
 
 		before(function () {
 			container = iniettore.create(function (context) {
@@ -127,7 +148,7 @@ describe('Given a container', function () {
 			})
 		})
 
-		it('should have test coverage', function () {
+		it('should not dispose the instance', function () {
 			container.get('baz')
 			container.release('baz')
 			expect(BazDisposeSpy).to.not.be.called