Merge branch 'module-token-factory-replacer' of https://github.com/zenozen/nest into zenozen-module-token-factory-replacer

Author: kamilmysliwiec

packages/core/injector/module-token-factory.ts

@@ -25,7 +25,7 @@ export class ModuleTokenFactory {
   ): string {
     // Uses safeStringify instead of JSON.stringify
     // to support circular dynamic modules
-    return dynamicModuleMetadata ? stringify(dynamicModuleMetadata) : '';
+    return dynamicModuleMetadata ? stringify(dynamicModuleMetadata, this.replacer) : '';
   }
 
   public getModuleName(metatype: Type<any>): string {
@@ -50,4 +50,12 @@ export class ModuleTokenFactory {
     const scope = Reflect.getMetadata(SHARED_MODULE_METADATA, metatype);
     return scope ? scope : 'global';
   }
+
+  private replacer(key: string, value: any) {
+    if (typeof value === 'function') {
+      return value.name;
+    } else {
+      return value;
+    }
+  }
 }

packages/core/test/injector/module-token-factory.spec.ts

@@ -65,6 +65,13 @@ describe('ModuleTokenFactory', () => {
           JSON.stringify(metadata),
         );
       });
+      it('should return hash with class', () => {
+        class Provider {}
+        const metadata = { providers: [Provider], exports: [Provider] };
+        expect(factory.getDynamicMetadataToken(metadata)).to.be.eql(
+          '{"providers":["Provider"],"exports":["Provider"]}'
+        );
+      });
     });
     describe('when metadata does not exist', () => {
       it('should return empty string', () => {