Bug Fix - azuredevops_serviceendpoint_azurecr - Expect serviceprincipalkey only if ServicePrincipal authentication is used

[04n0]

All Submissions:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My code follows the code style of this project.
• I ran lint checks locally prior to submission.
• Have you checked to ensure there aren't other open PRs for the same update/change?

What about the current behavior has changed?

When mapping Azure API response for azuredevops_serviceendpoint_azurecr resource to Terraform data structure, the credential key value authorization.parameters.serviceprincipalkey should only be processed when ServicePrincipal authentication scheme is used.
WorkloadIdentityFederation scheme does not have such a key in the API response.

Issue Number: #1124

Does this introduce a change to go.mod, go.sum or vendor/?

• Yes
• No

Does this introduce a breaking change?

• Yes
• No

Any relevant logs, error output, etc?

No

Other information

[xuzhang3]

=== RUN   TestAccServiceEndpointAzureCR_DataSource
=== PAUSE TestAccServiceEndpointAzureCR_DataSource
=== RUN   TestAccServiceEndpointAzureCR_Spn_Basic
=== PAUSE TestAccServiceEndpointAzureCR_Spn_Basic
=== RUN   TestAccServiceEndpointAzureCR_Spn_Update
=== PAUSE TestAccServiceEndpointAzureCR_Spn_Update
=== CONT  TestAccServiceEndpointAzureCR_DataSource
=== CONT  TestAccServiceEndpointAzureCR_Spn_Update
=== CONT  TestAccServiceEndpointAzureCR_Spn_Basic
--- PASS: TestAccServiceEndpointAzureCR_DataSource (90.04s)
--- PASS: TestAccServiceEndpointAzureCR_Spn_Basic (92.58s)
--- PASS: TestAccServiceEndpointAzureCR_Spn_Update (142.12s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        142.643s