nestjs / nest / b3db2e98-b73d-4d44-995e-94028a9d0900

89%

master: 90%

Build Type

Pull #16005

circleci

Committed by evgeniy.chernomortsev

Commit Message

fix(core): resolve all providers when using resolve() with each option

When resolving multiple default-scoped providers registered under the same
token using `moduleRef.resolve(token, undefined, { each: true })`, the method
was returning the same instance repeated N times instead of all distinct
provider instances.

The issue was in `resolvePerContext` where for static dependency trees it
called `this.get()` which always returned the last registered provider,
ignoring the specific `instanceLink` being processed. Now it returns
`wrapperRef.instance` directly from each instance link.

Closes #15979

Pull Request Pull Request #16005: fix(core): resolve all providers when using resolve() with each option

Run Details

2774 of 3498 branches covered (79.3%)

1 of 1 new or added line in 1 file covered. (100.0%)

7327 of 8236 relevant lines covered (88.96%)

17.99 hits per line