New resource: azurerm_api_management_identity_provider_aadb2c

[manicminer]

Add new resource azurerm_api_management_identity_provider_aadb2c.

This is tricky to test because creation requires a preexisting B2C tenant with matching policies already configured, plus an application registration in that tenant with sufficient privileges to create another application configured for the APIM instance.

I've set one up by hand (it's the only way at present) and confirmed it works locally. Any B2C tenant can be used, using the basic configuration described at https://docs.microsoft.com/en-us/azure/api-management/api-management-howto-aad-b2c

Accordingly, the acctests rely on additional environment variables to be set (and be valid). The tests are skipped if these are not set.

Very much open to suggestions if there's a better way to inject/detect/consume these!

Variable	Description
ARM_TEST_B2C_TENANT_ID	the UUID of the B2C tenant
ARM_TEST_B2C_TENANT_SLUG	the first part of the *.onmicrosoft.com domain of the B2C tenant
ARM_TEST_B2C_CLIENT_ID	client ID of an application in the B2C domain with privileges to create applications
ARM_TEST_B2C_CLIENT_SECRET	client secret for that application

Test results

GOROOT=/Users/tom/.goenv/versions/1.14.5 #gosetup
GOPATH=/Users/tom/go #gosetup
/Users/tom/.goenv/versions/1.14.5/bin/go test -c -o /private/var/folders/56/y8d3vjgj1qg6mqv6c_m0cy_w0000gp/T/___TestAccAzureRMApiManagementIdentityProviderAADB2C_basic_in_github_com_terraform_providers_terraform_provider_azurerm_azurerm_internal_services_apimanagement github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/apimanagement #gosetup
/Users/tom/.goenv/versions/1.14.5/bin/go tool test2json -t /private/var/folders/56/y8d3vjgj1qg6mqv6c_m0cy_w0000gp/T/___TestAccAzureRMApiManagementIdentityProviderAADB2C_basic_in_github_com_terraform_providers_terraform_provider_azurerm_azurerm_internal_services_apimanagement -test.v -test.run ^\QTestAccAzureRMApiManagementIdentityProviderAADB2C_basic\E$
=== RUN   TestAccAzureRMApiManagementIdentityProviderAADB2C_basic
=== PAUSE TestAccAzureRMApiManagementIdentityProviderAADB2C_basic
=== CONT  TestAccAzureRMApiManagementIdentityProviderAADB2C_basic
--- PASS: TestAccAzureRMApiManagementIdentityProviderAADB2C_basic (2189.53s)
PASS

Process finished with exit code 0

Replaces: #7785

[favoretti]

Since I just finished creating generators for all APIM resources, would you also mind adding one here? :)

[manicminer]

Test results for the resources with updated ID parsers

[tombuildsstuff]

@manicminer would you mind rebasing this one?

[tombuildsstuff]

hey @manicminer

Thanks for this PR - I've taken a look through and left some comments inline but if we can fix those up (and rebase this) then this otherwise LGTM 👍

Thanks!

[manicminer]

Thanks for the review! I've fixed it up per the suggestions. Test results pending.

[manicminer]

Test Results for other azurerm_api_management_identity_service_* resources:

Test results for this resource:

[katbyte]

LGTM 👍

stale

[ghost]

This has been released in version 2.46.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.46.0"
}
# ... other configuration ...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!