azurerm_cosmosdb_account: add new capabilities EnableAggregationPipeline, MongoDBv3.4, mongoEnableDocLevelTTL

[katbyte]

fixes #1848 #1927

uses workaround ad described in Azure/azure-sdk-for-go#2864

[katbyte]

[12:57:22] kt@snowbook:~/hashi/..3../terraform-providers/terraform-provider-azurerm▸f-cosmos-mongo34$ testazure TestAccAzureRMCosmosDBAccount_capability
==> Fixing source code with gofmt...
# This logic should match the search logic in scripts/gofmtcheck.sh
gofmt -s -w `find . -name '*.go' | grep -v vendor`
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./azurerm -v -test.run=TestAccAzureRMCosmosDBAccount_capability -timeout 180m -ldflags="-X=github.com/terraform-providers/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccAzureRMCosmosDBAccount_capabilityGremlin
=== PAUSE TestAccAzureRMCosmosDBAccount_capabilityGremlin
=== RUN   TestAccAzureRMCosmosDBAccount_capabilityTable
=== PAUSE TestAccAzureRMCosmosDBAccount_capabilityTable
=== RUN   TestAccAzureRMCosmosDBAccount_capabilityCassandra
=== PAUSE TestAccAzureRMCosmosDBAccount_capabilityCassandra
=== RUN   TestAccAzureRMCosmosDBAccount_capabilityAggregationPipeline
=== PAUSE TestAccAzureRMCosmosDBAccount_capabilityAggregationPipeline
=== RUN   TestAccAzureRMCosmosDBAccount_capabilityMongo35
=== PAUSE TestAccAzureRMCosmosDBAccount_capabilityMongo35
=== RUN   TestAccAzureRMCosmosDBAccount_capabilityDocLevelTTL
=== PAUSE TestAccAzureRMCosmosDBAccount_capabilityDocLevelTTL
=== CONT  TestAccAzureRMCosmosDBAccount_capabilityGremlin
=== CONT  TestAccAzureRMCosmosDBAccount_capabilityMongo35
=== CONT  TestAccAzureRMCosmosDBAccount_capabilityCassandra
=== CONT  TestAccAzureRMCosmosDBAccount_capabilityDocLevelTTL
=== CONT  TestAccAzureRMCosmosDBAccount_capabilityAggregationPipeline
=== CONT  TestAccAzureRMCosmosDBAccount_capabilityTable
--- PASS: TestAccAzureRMCosmosDBAccount_capabilityMongo35 (1321.83s)
--- PASS: TestAccAzureRMCosmosDBAccount_capabilityGremlin (1321.91s)
--- PASS: TestAccAzureRMCosmosDBAccount_capabilityCassandra (1323.35s)
--- PASS: TestAccAzureRMCosmosDBAccount_capabilityDocLevelTTL (1338.59s)
--- PASS: TestAccAzureRMCosmosDBAccount_capabilityAggregationPipeline (1339.40s)
--- PASS: TestAccAzureRMCosmosDBAccount_capabilityTable (1421.25s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	1422.289s
[13:42:28] kt@snowbook:~/hashi/..3../terraform-providers/terraform-provider-azurerm▸f-cosmos-mongo34$ gitacp "patch capabilities"

[tombuildsstuff]

Aside from some missing docs (and the comments) this otherwise LGTM

[tombuildsstuff]

It’d be good to add a test handling an update here

[dcfsc]

Also a test with more than one - we have "MongoDB3.4" and "EnableAggregationPipeline" and a) I have no idea how to add to the .tf file and b) maybe the test should confirm that >1 capabilities can be added.

[tombuildsstuff]

LGTM 👍

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