fix(core): throw on circular factories and custom providers

[kamilmysliwiec]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other... Please describe:

What is the current behavior?

Issue Number: #11630

What is the new behavior?

Does this PR introduce a breaking change?

• Yes
• No

Other information

[coveralls]

Pull Request Test Coverage Report for Build 8ff3efcf-664e-4098-9c48-22cfdcb38d74

• 24 of 27 (88.89%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.01%) to 92.853%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/core/injector/settlement-signal.ts	8	9	88.89%
packages/core/injector/injector.ts	14	16	87.5%

Totals
Change from base Build 3dfb6545-1975-4a99-b2ed-aaa49f412a91:	-0.01%
Covered Lines:	6379
Relevant Lines:	6870

---

💛 - Coveralls

[jmcdo29]

LGTM, glad to see a solution come in here. Tracing it back to that done promise was a bit of a doozy