Nest: More informative error message when dependencies are improperly setup

[bytasv]

Hello,

I'm just starting to work with nest.js framework and I noticed a couple of times, that I get a similar message:

Nest can't resolve dependencies of the SomeService. Please verify whether all of them are available in the current context

Wouldn't it be possible to tell precisely which dependency is not being picked up? Something like:

Nest was unable to resolve someDependency of SomeService in some.service.ts:15:25.

Some more informative error description would really help here

[kamilmysliwiec]

Hi @bytasv,
In progress 🖌

[wbhob]

While you're at it, maybe PR that feature against Angular! That would be a really good fx to have to help resolve deps.

[kamilmysliwiec]

Hi @bytasv,
It's done. More informative error messages are available with the latest update (4.3.0) 🙂

[ExceptionHandler] Nest can't resolve dependencies of the UsersService (+, +, ?, +, +, +). Please verify whether [2] argument is available in the current context.

[JamesCoonce]

The error message doesn't make much sense. + and ? don't really say much. Is there documentation that can explain?

[wbhob]

@JamesCoonce +'s mean the dependency was found, ? means it is missing. it tells you which dependency is missing in the order of injection

[XGHeaven]

Why not use class name instand of +. And keep ? For unknown class.

[JamesCoonce]

That makes a lot more sense. + and ? are extremely vague.

[BrunnerLivio]

Submitted a PR #814 for this @XGHeaven @JamesCoonce.

The new error message looks like this:

[lock[bot]]

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.