[nestjs#1384] Allow async NestModule.configure method

Author: zailic

package-lock.json

@@ -1,5 +1,5 @@
 import { MiddlewareConsumer } from '../middleware/middleware-consumer.interface';
 
 export interface NestModule {
-  configure(consumer: MiddlewareConsumer): Promise<MiddlewareConsumer | void>;
+  configure(consumer: MiddlewareConsumer);
 }

packages/common/interfaces/modules/nest-module.interface.ts

@@ -5,7 +5,6 @@ import {
   ClassProvider,
   Controller,
   DynamicModule,
-  // @ts-ignore
   ExistingProvider,
   FactoryProvider,
   Injectable,
@@ -168,15 +167,13 @@ export class Module {
       return this.addCustomProvider(injectable, this._injectables);
     }
     const instanceWrapper = new InstanceWrapper({
-      // @ts-ignore
       name: injectable.name,
       metatype: injectable,
       instance: null,
       isResolved: false,
       scope: this.getClassScope(injectable),
       host: this,
     });
-    // @ts-ignore
     this._injectables.set(injectable.name, instanceWrapper);
 
     if (host) {

packages/core/injector/module.ts

@@ -49,13 +49,13 @@ describe('MiddlewareModule', () => {
   });
 
   describe('loadConfiguration', () => {
-    it('should call "configure" method if method is implemented', () => {
+    it('should call "configure" method if method is implemented', async () => {
       const configureSpy = sinon.spy();
       const mockModule = {
         configure: configureSpy,
       };
 
-      middlewareModule.loadConfiguration(
+      await middlewareModule.loadConfiguration(
         new MiddlewareContainer(),
         mockModule as any,
         'Test' as any,