Skip to content

Commit

Permalink
r/api_management: setting identity_ids as a []interface{} so this…
Browse files Browse the repository at this point in the history
…'ll be inferred

Fixes #10136
  • Loading branch information
tombuildsstuff committed Feb 2, 2021
1 parent 22719ad commit 11837c5
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,26 @@ func TestAccDataSourceApiManagement_identitySystemAssigned(t *testing.T) {
})
}

func TestAccDataSourceApiManagement_identityUserAssigned(t *testing.T) {
data := acceptance.BuildTestData(t, "data.azurerm_api_management", "test")
r := ApiManagementDataSource{}

data.DataSourceTest(t, []resource.TestStep{
{
Config: r.identityUserAssigned(data),
Check: resource.ComposeTestCheckFunc(
check.That(data.ResourceName).Key("publisher_email").HasValue("[email protected]"),
check.That(data.ResourceName).Key("publisher_name").HasValue("pub1"),
check.That(data.ResourceName).Key("sku_name").HasValue("Developer_1"),
check.That(data.ResourceName).Key("tags.%").HasValue("0"),
check.That(data.ResourceName).Key("public_ip_addresses.#").Exists(),
check.That(data.ResourceName).Key("identity.#").HasValue("1"),
check.That(data.ResourceName).Key("identity.0.type").HasValue("UserAssigned"),
),
},
})
}

func TestAccDataSourceApiManagement_virtualNetwork(t *testing.T) {
data := acceptance.BuildTestData(t, "data.azurerm_api_management", "test")
r := ApiManagementDataSource{}
Expand Down Expand Up @@ -129,6 +149,18 @@ data "azurerm_api_management" "test" {
`, data.RandomInteger, data.Locations.Primary, data.RandomInteger)
}

func (ApiManagementDataSource) identityUserAssigned(data acceptance.TestData) string {
template := ApiManagementResource{}.identityUserAssigned(data)
return fmt.Sprintf(`
%s
data "azurerm_api_management" "test" {
name = azurerm_api_management.test.name
resource_group_name = azurerm_api_management.test.resource_group_name
}
`, template)
}

func (ApiManagementDataSource) virtualNetwork(data acceptance.TestData) string {
return fmt.Sprintf(`
provider "azurerm" {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1203,7 +1203,8 @@ func flattenAzureRmApiManagementMachineIdentity(identity *apimanagement.ServiceI
}
identityIds = append(identityIds, parsedId.ID())
}
result["identity_ids"] = schema.NewSet(schema.HashString, identityIds)
// not casting to a Set since this is a List in the DS and a Set in the Resource
result["identity_ids"] = identityIds
}

return []interface{}{result}, nil
Expand Down

0 comments on commit 11837c5

Please sign in to comment.