Merge pull request #824 from jmcdo29/fix/property-injection-optional

fix: add optional decorator for property based injection

Author: kamilmysliwiec

jest.config.js

@@ -1,10 +1,11 @@
 module.exports = {
   moduleFileExtensions: ['js', 'json', 'ts'],
   rootDir: '.',
-  testMatch: ['<rootDir>/test/*.e2e-spec.ts'],
+  testMatch: ['<rootDir>/test/**/*.e2e-spec.ts'],
   transform: {
     '^.+\\.ts$': 'ts-jest'
   },
   testEnvironment: 'node',
-  collectCoverage: true
+  collectCoverage: true,
+  collectCoverageFrom: ['lib/**/*.ts']
 };

lib/auth.guard.ts

@@ -31,6 +31,7 @@ const NO_STRATEGY_ERROR = `In order to use "defaultStrategy", please, ensure to
 
 function createAuthGuard(type?: string | string[]): Type<CanActivate> {
   class MixinAuthGuard<TUser = any> implements CanActivate {
+    @Optional()
     @Inject(AuthModuleOptions)
     protected options: AuthModuleOptions;
     constructor(@Optional() options?: AuthModuleOptions) {

test/app.controller.ts renamed to test/common/app.controller.ts

@@ -1,5 +1,5 @@
 import { Body, Controller, Get, Post, Req, UseGuards } from '@nestjs/common';
-import { AuthGuard } from '../lib';
+import { AuthGuard } from '../../lib';
 
 import { AppService } from './app.service';
 

test/app.e2e-spec.ts renamed to test/common/app.e2e-spec.ts

@@ -1,9 +1,14 @@
 import { INestApplication } from '@nestjs/common';
 import { Test } from '@nestjs/testing';
 import { spec, request } from 'pactum';
-import { AppModule } from './app.module';
+import { AppModule as WithRegisterModule } from '../with-register/app.module';
+import { AppModule as WithoutRegisterModule } from '../without-register/app.module';
 
-describe('Passport Module', () => {
+describe.each`
+  AppModule                | RegisterUse
+  ${WithRegisterModule}    | ${'with'}
+  ${WithoutRegisterModule} | ${'without'}
+`('Passport Module $RegisterUse register()', ({ AppModule }) => {
   let app: INestApplication;
 
   beforeAll(async () => {

test/app.service.ts renamed to test/common/app.service.ts

@@ -1,6 +1,6 @@
 import { Injectable } from '@nestjs/common';
 import { ExtractJwt, Strategy } from 'passport-jwt';
-import { PassportStrategy } from '../lib';
+import { PassportStrategy } from '../../lib';
 
 @Injectable()
 export class JwtStrategy extends PassportStrategy(Strategy) {

test/jwt.strategy.ts renamed to test/common/jwt.strategy.ts

@@ -1,6 +1,6 @@
 import { Injectable } from '@nestjs/common';
 import { Strategy } from 'passport-local';
-import { PassportStrategy } from '../lib';
+import { PassportStrategy } from '../../lib';
 import { AppService } from './app.service';
 
 @Injectable()

test/local.strategy.ts renamed to test/common/local.strategy.ts

@@ -1,10 +1,10 @@
 import { Module } from '@nestjs/common';
 import { JwtModule } from '@nestjs/jwt';
-import { PassportModule } from '../lib';
-import { AppController } from './app.controller';
-import { AppService } from './app.service';
-import { JwtStrategy } from './jwt.strategy';
-import { LocalStrategy } from './local.strategy';
+import { PassportModule } from '../../lib';
+import { AppController } from '../common/app.controller';
+import { AppService } from '../common/app.service';
+import { JwtStrategy } from '../common/jwt.strategy';
+import { LocalStrategy } from '../common/local.strategy';
 
 @Module({
   controllers: [AppController],

test/app.module.ts renamed to test/with-register/app.module.ts

@@ -0,0 +1,19 @@
+import { Module } from '@nestjs/common';
+import { JwtModule } from '@nestjs/jwt';
+import { PassportModule } from '../../lib';
+import { AppController } from '../common/app.controller';
+import { AppService } from '../common/app.service';
+import { JwtStrategy } from '../common/jwt.strategy';
+import { LocalStrategy } from '../common/local.strategy';
+
+@Module({
+  controllers: [AppController],
+  imports: [
+    JwtModule.register({
+      secret: 's3cr3t'
+    }),
+    PassportModule
+  ],
+  providers: [AppService, LocalStrategy, JwtStrategy]
+})
+export class AppModule {}