refactor(): remove duplicate function getClassScope

[zanminkian]

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: N/A

What is the new behavior?

Does this PR introduce a breaking change?

• Yes
• No

Other information

[coveralls]

Pull Request Test Coverage Report for Build 9229029f-6efc-4ed1-9dc4-37cf04265af3

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.002%) to 94.128%

---

Totals
Change from base Build 6c6442b4-a131-4d7f-a0f8-d2eb1814def7:	-0.002%
Covered Lines:	5691
Relevant Lines:	6046

---

💛 - Coveralls

[zanminkian]

I want to move get-class-scope.ts from @nestjs/core to @nestjs/common and add a function:

export function setClassScope(provider: Type<unknown>, scope: Scope): void {
  Reflect.defineMetadata(SCOPE_OPTIONS_METADATA, scope, provider);
}

I think get and set should be a pair. And alse easy to write unit tests.
Is it will be better? -: )

[micalevisk]

nice catch! DRY them all

[kamilmysliwiec]

Sounds good @zanminkian!

[zanminkian]

I want to move get-class-scope.ts from @nestjs/core to @nestjs/common and add a function:

export function setClassScope(provider: Type<unknown>, scope: Scope): void {
  Reflect.defineMetadata(SCOPE_OPTIONS_METADATA, scope, provider);
}

I think get and set should be a pair. And alse easy to write unit tests. Is it will be better? -: )

@kamilmysliwiec Could you please review this PR? I will create a next PR to optimize class scope management.

[kamilmysliwiec]

lgtm