Additions to azurerm_key_vault

[tombuildsstuff]

Adding the missing fields:

• application_id
• certificate_permissions

Also added application_id to the azurerm_client_config data source as it was needed

Fixes #48

[tombuildsstuff]

Tests pass.

Data Source:

 $ acctests azurerm TestAccDataSourceAzureRMClientConfig_
=== RUN   TestAccDataSourceAzureRMClientConfig_basic
--- PASS: TestAccDataSourceAzureRMClientConfig_basic (10.81s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	10.831s

Key Vault:

$ acctests azurerm TestAccAzureRMKeyVault_
=== RUN   TestAccAzureRMKeyVault_importBasic
--- PASS: TestAccAzureRMKeyVault_importBasic (75.21s)
=== RUN   TestAccAzureRMKeyVault_name
--- PASS: TestAccAzureRMKeyVault_name (0.00s)
=== RUN   TestAccAzureRMKeyVault_basic
--- PASS: TestAccAzureRMKeyVault_basic (66.83s)
=== RUN   TestAccAzureRMKeyVault_complete
--- PASS: TestAccAzureRMKeyVault_complete (76.32s)
=== RUN   TestAccAzureRMKeyVault_update
--- PASS: TestAccAzureRMKeyVault_update (101.52s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	319.903s

[radeksimko]

LGTM, assuming all relevant acceptance tests are passing.

[radeksimko]

Wondering if there's an array/slice of all permissions, so that we don't need to update this list after Azure introduces new ones. 🤔

[tombuildsstuff]

it's exposed as a collection of Constants, so it might be hard in the short term - however we could ask the SDK team to expose that as a function?

[radeksimko]

Up to you, it's no big deal - I just thought it may save us some pain/work in the future.
Feel free to 🚢 it as is.

[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!