nest: [bug] 6.0.1 Dependency Injection in Testing Module

I’m submitting a…

[ ] Regression 
[x ] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

It is not possible to inject dependencies into modules in test scenarios using the overrideProvider mechanism

Expected behavior

It should be possible to inject dependencies into modules in test scenarios using the overrideProvider mechanism

Minimal reproduction of the problem with instructions

https://github.com/WonderPanda/nest-testing-bug Just run yarn test:e2e and you’ll see the following output:

Nest can't resolve dependencies of the BugModule (?). Please make sure that the argument at index [0] is available in the BugModule context.

      at Injector.lookupComponentInExports (../node_modules/@nestjs/core/injector/injector.js:180:19)

Prior to version 6, it was simple to provide mocks for dependencies (both for other providers/controllers as well as modules) using the overrideProvider fluent methods. The change introduced here https://github.com/nestjs/nest/pull/1722/files with an additional check for this.hasProvider(toReplace) seems to have broken those scenarios. It’s a very common scenario in both the libraries I’ve built and the tests that I’ve written to want to be able to supply a provider without it needing to have been previously available in the module context.

The attached repo shows a simple example of this where there is a module that requires a dependency in it’s constructor:

@Module({})
export class BugModule {
  constructor(@Inject('BugProvider') private readonly bugProvider: string) {}
}

but there doesn’t appear to be any mechanism now to be able to achieve this in tests.

I’m a little concerned with the initial launch of version 6 as this and the bug that was fixed as part of 6.0.1 are both core testing scenarios that have come up very often in mine and my team’s use of NestJS. It seems like there’s a gap right now in tests that actually validate that the TestingModule works properly and I’m hesitant to suggest people try version 6 until this is addressed.