fix(core): Calculate module distance after bindGlobalScope

[wenlong-chen]

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?

The global module’s onModuleInit is not called before its dependent’s onModuleInit. This happens because, during the distance calculation, the global module is not yet included in the dependent’s imports. It only appears in the imports after bindGlobalScope, so the distance should be calculated at that point.

Issue Number: #14117

What is the new behavior?

The global module’s onModuleInit is called before its dependent’s onModuleInit.

The test mentioned above passes.

Does this PR introduce a breaking change?

• Yes
• No

Other information

[coveralls]

Pull Request Test Coverage Report for Build 16217631-6001-4694-9201-514ddbcaa94a

Details

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 92.243%

---

Totals
Change from base Build 72f148bc-4145-44b3-a3c4-06a616307114:	0.0%
Covered Lines:	6754
Relevant Lines:	7322

---

💛 - Coveralls

[kamilmysliwiec]

lgtm