Merge pull request #34 from xxxtonixxx/fix/edge-cases

fix(Container): Added comprobation to edge cases

Author: pleerock

src/Container.ts

@@ -70,7 +70,7 @@ export class Container {
         let service = this.findService(identifier);
 
         // find if instance of this object already initialized in the container and return it if it is
-        if (service && service.instance)
+        if (service && service.instance !== null && service.instance !== undefined)
             return service.instance as T;
 
         // if named service was requested and its instance was not found plus there is not type to know what to initialize,

test/Container.spec.ts

@@ -9,6 +9,34 @@ describe("Container", function() {
 
     beforeEach(() => Container.reset());
 
+    describe("get", () => {
+
+        it("should be able to get a boolean", () => {
+            const booleanTrue = "boolean.true";
+            const booleanFalse = "boolean.false";
+            Container.set(booleanTrue, true);
+            Container.set(booleanFalse, false);
+
+            Container.get(booleanTrue).should.be.true;
+            Container.get(booleanFalse).should.be.false;
+        });
+
+        it("should be able to get an empty string", () => {
+            const emptyString = "emptyString";
+            Container.set(emptyString, "");
+
+            Container.get(emptyString).should.be.eq("");
+        });
+
+        it("should be able to get the 0 number", () => {
+            const zero = "zero";
+            Container.set(zero, 0);
+
+            Container.get(zero).should.be.eq(0);
+        });
+
+    });
+
     describe("set", function() {
 
         it("should be able to set a class into the container", function() {