[bug] 6.0.0 Dependency Inject dont work in Testingmodule override class

[12548]

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.

Minimal reproduction of the problem with instructions

demo code:

/* tslint:disable:max-classes-per-file */
import {Injectable} from '@nestjs/common'
import {Test} from '@nestjs/testing'

@Injectable()
class WorldService {

	get() {
		return 'world'
	}

}

@Injectable()
class HelloService {

	constructor(private readonly world: WorldService) {
	}

	hello() {
		console.log('hello ' + this.world.get())
	}

}

@Injectable()
class GoodbyeService {
	constructor(private readonly world: WorldService) {
	}

	hello() {
		console.log('goodbye ' + this.world.get())
	}

}

(async () => {
	const mod = await Test.createTestingModule({
		providers: [
			HelloService,
			WorldService,
		],
	})
		.overrideProvider(HelloService)
		.useClass(GoodbyeService)
		.compile()

	// should print 'goodbye world' but fails
	mod.get(HelloService).hello()

})()

Current behavior

the code fails with error:

(node:9176) UnhandledPromiseRejectionWarning: Error: Nest can't resolve dependencies of the HelloService (?). Please make sure that the argument at index [0] is available in the InternalCoreModule context.
    at Injector.lookupComponentInExports (C:\xxx\node_modules\@nestjs\core\injector\injector.js:180:19)
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at Function.Module.runMain (internal/modules/cjs/loader.js:745:11)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:743:3)
(node:9176) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:9176) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior

It works well in 5.7.4 and prints goodbye world.

Environment

Nest version: 6.0.0

 
For Tooling issues:
- Node version: 10.15.1
- Platform:  Windows

[kamilmysliwiec]

Have you updated both @nestjs/core and @nestjs/testing packages? If so, we need a small repository which reproduces your issue.

[12548]

@kamilmysliwiec
Yes of course I upgraded them both.
I uploaded the small repo: https://github.com/12548/BugDemo1

[kamilmysliwiec]

Thanks, I'll take a look :)

[kamilmysliwiec]

It should be fixed in 6.0.1 :) Please, let me know if you encounter any issues.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.