Merge pull request #63 from inversify/fix/update-lazy-service-identif…

…ier-is-to-accept-nullish-values

Update LazyServiceIdentifier.is to support nullish values

.changeset/eight-waves-roll.md

@@ -0,0 +1,5 @@
+---
+"@inversifyjs/common": patch
+---
+
+Updated LazyServiceIdentifier.is to support nullish values

packages/container/libraries/common/src/services/models/LazyServiceIdentifier.spec.ts

@@ -15,6 +15,46 @@ describe(LazyServiceIdentifier.name, () => {
   });
 
   describe('.is', () => {
+    describe('having a non object', () => {
+      let valueFixture: unknown;
+
+      beforeAll(() => {
+        valueFixture = Symbol();
+      });
+
+      describe('when called', () => {
+        let result: unknown;
+
+        beforeAll(() => {
+          result = LazyServiceIdentifier.is(valueFixture);
+        });
+
+        it('should return false', () => {
+          expect(result).toBe(false);
+        });
+      });
+    });
+
+    describe('having a null object', () => {
+      let valueFixture: null;
+
+      beforeAll(() => {
+        valueFixture = null;
+      });
+
+      describe('when called', () => {
+        let result: unknown;
+
+        beforeAll(() => {
+          result = LazyServiceIdentifier.is(valueFixture);
+        });
+
+        it('should return false', () => {
+          expect(result).toBe(false);
+        });
+      });
+    });
+
     describe('having an object with islazyServiceIdentifierSymbol property', () => {
       let valueFixture: LazyServiceIdentifier;
 

packages/container/libraries/common/src/services/models/LazyServiceIdentifier.ts

@@ -18,6 +18,8 @@ export class LazyServiceIdentifier<TInstance = unknown> {
     value: unknown,
   ): value is LazyServiceIdentifier<TInstance> {
     return (
+      typeof value === 'object' &&
+      value !== null &&
       (value as Partial<LazyServiceIdentifier>)[
         islazyServiceIdentifierSymbol
       ] === true