Update AzResourceTypeFactory cache key to take account of required flags

[jeskew]

Resolves #13534

The cache within AzResourceTypeFactory was using the serialized type as a key, but since the AZ resource types reuse types pretty aggressively, this was leading to values being cached with the wrong flag assignments (which are dependent on where in a resource a particular type is used). This PR updates the cache to use a compound key instead to keep these usages separate.

Microsoft Reviewers: Open in CodeFlow

[github-actions]

Test this change out locally with the following install scripts (Action run 8159701342)

VSCode

• Mac/Linux

  bash <(curl -Ls https://aka.ms/bicep/nightly-vsix.sh) --run-id 8159701342

• Windows

  iex "& { $(irm https://aka.ms/bicep/nightly-vsix.ps1) } -RunId 8159701342"

Azure CLI

• Mac/Linux

  bash <(curl -Ls https://aka.ms/bicep/nightly-cli.sh) --run-id 8159701342

• Windows

  iex "& { $(irm https://aka.ms/bicep/nightly-cli.ps1) } -RunId 8159701342"

[github-actions]

Test Results

    65 files   -     34      65 suites   - 34   20m 21s ⏱️ - 20m 51s
10 672 tests  -     24  10 671 ✅  -     23  1 💤 ±0  0 ❌  - 1 
22 423 runs   - 15 436  22 421 ✅  - 15 434  2 💤  - 1  0 ❌  - 1 

Results for commit e02cbc9. ± Comparison against base commit 2dc234c.

♻️ This comment has been updated with latest results.