Merge pull request #19 from nestjs-community/fix/for-async-promise

Added promise type to async forRootAsync

Author: Shady Khalifa

src/access-control.module.async.spec.ts

@@ -2,9 +2,10 @@ import {AccessControlModule} from './access-control.module';
 import {Test, TestingModule} from '@nestjs/testing';
 import { RolesBuilder } from './roles-builder.class';
 import {ROLES_BUILDER_TOKEN} from './constants';
+import { delay } from 'rxjs/operators';
 
 describe('forRootAsync', () => {
-  it('Can instance with asnyc provider method', async () => {
+  it('Can instance with provider method', async () => {
 
     const module: TestingModule = await Test.createTestingModule({
       imports: [
@@ -18,4 +19,22 @@ describe('forRootAsync', () => {
 
     expect(roles).toBeInstanceOf(RolesBuilder);
   });
+
+  it('Can instance with asnyc provider method', async () => {
+
+    const module: TestingModule = await Test.createTestingModule({
+      imports: [
+        AccessControlModule.forRootAsync({
+          useFactory: async (): Promise<RolesBuilder> => {
+            await delay(100);
+            return new RolesBuilder();
+          },
+        }),
+      ],
+    }).compile();
+
+    const roles = module.get(ROLES_BUILDER_TOKEN);
+
+    expect(roles).toBeInstanceOf(RolesBuilder);
+  });
 });

src/access-control.module.ts

@@ -30,7 +30,7 @@ export class AccessControlModule {
 
   public static forRootAsync(options: {
     inject?: Provider[],
-    useFactory: (...args: any) => RolesBuilder,
+    useFactory: (...args: any) => RolesBuilder | Promise<RolesBuilder>,
   }): DynamicModule {
 
     const provider = {