Skip to content

Commit 54dc128

Browse files
committed
test: add comprehensive tests for delete_on_destroy attribute
- Add TestAccSubnet_deleteOnDestroy_default to verify default true behavior - Add TestAccSubnet_deleteOnDestroy_stateOnly to test state-only destroy
1 parent 40ae4f0 commit 54dc128

File tree

1 file changed

+61
-0
lines changed

1 file changed

+61
-0
lines changed

internal/services/network/subnet_resource_test.go

Lines changed: 61 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,43 @@ func TestAccSubnet_deleteOnDestroy(t *testing.T) {
173173
})
174174
}
175175

176+
func TestAccSubnet_deleteOnDestroy_default(t *testing.T) {
177+
data := acceptance.BuildTestData(t, "azurerm_subnet", "internal")
178+
r := SubnetResource{}
179+
180+
data.ResourceTest(t, r, []acceptance.TestStep{
181+
{
182+
Config: r.deleteOnDestroyDefault(data),
183+
Check: acceptance.ComposeTestCheckFunc(
184+
check.That(data.ResourceName).ExistsInAzure(r),
185+
check.That(data.ResourceName).Key("delete_on_destroy").HasValue("true"),
186+
),
187+
},
188+
data.ImportStep("delete_on_destroy"),
189+
})
190+
}
191+
192+
func TestAccSubnet_deleteOnDestroy_stateOnly(t *testing.T) {
193+
data := acceptance.BuildTestData(t, "azurerm_subnet", "internal")
194+
r := SubnetResource{}
195+
196+
data.ResourceTest(t, r, []acceptance.TestStep{
197+
{
198+
Config: r.deleteOnDestroy(data, false),
199+
Check: acceptance.ComposeTestCheckFunc(
200+
check.That(data.ResourceName).ExistsInAzure(r),
201+
check.That(data.ResourceName).Key("delete_on_destroy").HasValue("false"),
202+
),
203+
},
204+
{
205+
Config: r.deleteOnDestroyStateOnly(data),
206+
Check: acceptance.ComposeTestCheckFunc(
207+
check.That("data.azurerm_subnet.internal").Key("name").HasValue("internal"),
208+
),
209+
},
210+
})
211+
}
212+
176213
func TestAccSubnet_delegation(t *testing.T) {
177214
data := acceptance.BuildTestData(t, "azurerm_subnet", "test")
178215
r := SubnetResource{}
@@ -742,6 +779,30 @@ resource "azurerm_subnet" "internal" {
742779
`, r.template(data), deleteOnDestroy)
743780
}
744781

782+
func (r SubnetResource) deleteOnDestroyDefault(data acceptance.TestData) string {
783+
return fmt.Sprintf(`
784+
%s
785+
resource "azurerm_subnet" "internal" {
786+
name = "internal"
787+
resource_group_name = azurerm_resource_group.test.name
788+
virtual_network_name = azurerm_virtual_network.test.name
789+
address_prefixes = ["10.0.2.0/24"]
790+
}
791+
`, r.template(data))
792+
}
793+
794+
func (r SubnetResource) deleteOnDestroyStateOnly(data acceptance.TestData) string {
795+
return fmt.Sprintf(`
796+
%s
797+
798+
data "azurerm_subnet" "internal" {
799+
name = "internal"
800+
resource_group_name = azurerm_resource_group.test.name
801+
virtual_network_name = azurerm_virtual_network.test.name
802+
}
803+
`, r.template(data))
804+
}
805+
745806
func (r SubnetResource) delegationUpdated(data acceptance.TestData) string {
746807
return fmt.Sprintf(`
747808
%s

0 commit comments

Comments
 (0)