Add tests around how plugin framework provider configuration code handles scopes values

[SarahFrench]

This comment shares context about why we are adding these tests: #8818 (comment)

I recommend hiding whitespace changes when reviewing this PR!

In this PR, I:

• Add test cases to match existing scopes SDK tests
• Ensure tests present about handling empty values (e.g. empty strings, empty arrays)
• Ensure tests present about handling Unknown values
• Did some small refactoring:
  • (FrameworkProviderConfig).Scopes is now using types.List instead of []string
  • I removed a redundant var called defaultClientScopes

---

Release Note Template for Downstream PRs (will be copied)

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 6 files changed, 511 insertions(+), 33 deletions(-))
Terraform Beta: Diff ( 6 files changed, 511 insertions(+), 33 deletions(-))

[modular-magician]

Tests analytics

Total tests: 3011
Passed tests 2715
Skipped tests: 296
Affected tests: 0

$\textcolor{green}{\textsf{All tests passed in REPLAYING mode.}}$
View the build log

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 3 files changed, 104 insertions(+), 22 deletions(-))
Terraform Beta: Diff ( 3 files changed, 104 insertions(+), 22 deletions(-))

[modular-magician]

Tests analytics

Total tests: 3014
Passed tests 2718
Skipped tests: 296
Affected tests: 0

$\textcolor{green}{\textsf{All tests passed in REPLAYING mode.}}$
View the build log

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Breaking Change Detection Failed

The breaking change detector crashed during execution. This is usually due to the downstream provider(s) failing to compile. Please investigate or follow up with your reviewer.

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 3 files changed, 105 insertions(+), 23 deletions(-))
Terraform Beta: Diff ( 3 files changed, 105 insertions(+), 23 deletions(-))

[SarahFrench]

☝️ Problem due to a change lost when addressing merge conflicts

[SarahFrench]

This defaultClientScopes was removed because it's a duplicate of transport_tpg.DefaultClientScopes

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 3 files changed, 104 insertions(+), 22 deletions(-))
Terraform Beta: Diff ( 3 files changed, 104 insertions(+), 22 deletions(-))

[modular-magician]

Tests analytics

Total tests: 3020
Passed tests 2723
Skipped tests: 297
Affected tests: 0

$\textcolor{green}{\textsf{All tests passed in REPLAYING mode.}}$
View the build log

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 3 files changed, 104 insertions(+), 22 deletions(-))
Terraform Beta: Diff ( 3 files changed, 104 insertions(+), 22 deletions(-))

[modular-magician]

Tests analytics

Total tests: 3027
Passed tests 2730
Skipped tests: 297
Affected tests: 0

$\textcolor{green}{\textsf{All tests passed in REPLAYING mode.}}$
View the build log