Skip to content

Commit

Permalink
provider tests: removes the requires import feature flag [A-K] (#6558)
Browse files Browse the repository at this point in the history
  • Loading branch information
@tracypholmes
tracypholmes committed Apr 21, 2020
1 parent 83974f7 commit c1707b0
Show file tree
Hide file tree
Showing 130 changed files with 14 additions and 740 deletions.
5 changes: 0 additions & 5 deletions ...nal/services/apimanagement/tests/resource_arm_api_management_api_operation_policy_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -37,10 +36,6 @@ func TestAccAzureRMApiManagementAPIOperationPolicy_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementAPIOperationPolicy_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_api_operation_policy", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...m/internal/services/apimanagement/tests/resource_arm_api_management_api_operation_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMApiManagementApiOperation_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementApiOperation_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_api_operation", "test")

resource.Test(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_api_policy_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -37,10 +36,6 @@ func TestAccAzureRMApiManagementAPIPolicy_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementAPIPolicy_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_api_policy", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_api_schema_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMApiManagementApiSchema_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementApiSchema_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_api_schema", "test")

resource.Test(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_api_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -121,10 +120,6 @@ func TestAccAzureRMApiManagementApi_version(t *testing.T) {
}

func TestAccAzureRMApiManagementApi_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_api", "test")

resource.Test(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...internal/services/apimanagement/tests/resource_arm_api_management_api_version_set_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMApiManagementApiVersionSet_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementApiVersionSet_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_api_version_set", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...nal/services/apimanagement/tests/resource_arm_api_management_authorization_server_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMAPIManagementAuthorizationServer_basic(t *testing.T) {
}

func TestAccAzureRMAPIManagementAuthorizationServer_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_authorization_server", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_backend_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -187,10 +186,6 @@ func TestAccAzureRMApiManagementBackend_disappears(t *testing.T) {
}

func TestAccAzureRMApiManagementBackend_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_backend", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...erm/internal/services/apimanagement/tests/resource_arm_api_management_certificate_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -44,10 +43,6 @@ func TestAccAzureRMAPIManagementCertificate_basic(t *testing.T) {
}

func TestAccAzureRMAPIManagementCertificate_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_certificate", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_diagnostic_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMApiManagementDiagnostic_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementDiagnostic_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_diagnostic", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_group_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -34,10 +33,6 @@ func TestAccAzureRMAPIManagementGroup_basic(t *testing.T) {
}

func TestAccAzureRMAPIManagementGroup_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_group", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_group_user_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMAPIManagementGroupUser_basic(t *testing.T) {
}

func TestAccAzureRMAPIManagementGroupUser_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_group_user", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...al/services/apimanagement/tests/resource_arm_api_management_identity_provider_aad_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -67,10 +66,6 @@ func TestAccAzureRMApiManagementIdentityProviderAAD_update(t *testing.T) {
}

func TestAccAzureRMApiManagementIdentityProviderAAD_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_identity_provider_aad", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...rvices/apimanagement/tests/resource_arm_api_management_identity_provider_facebook_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -65,10 +64,6 @@ func TestAccAzureRMApiManagementIdentityProviderFacebook_update(t *testing.T) {
}

func TestAccAzureRMApiManagementIdentityProviderFacebook_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_identity_provider_facebook", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...services/apimanagement/tests/resource_arm_api_management_identity_provider_google_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -62,10 +61,6 @@ func TestAccAzureRMApiManagementIdentityProviderGoogle_update(t *testing.T) {
}

func TestAccAzureRMApiManagementIdentityProviderGoogle_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_identity_provider_google", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...vices/apimanagement/tests/resource_arm_api_management_identity_provider_microsoft_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -65,10 +64,6 @@ func TestAccAzureRMApiManagementIdentityProviderMicrosoft_update(t *testing.T) {
}

func TestAccAzureRMApiManagementIdentityProviderMicrosoft_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_identity_provider_microsoft", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...ervices/apimanagement/tests/resource_arm_api_management_identity_provider_twitter_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -65,10 +64,6 @@ func TestAccAzureRMApiManagementIdentityProviderTwitter_update(t *testing.T) {
}

func TestAccAzureRMApiManagementIdentityProviderTwitter_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_identity_provider_twitter", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions azurerm/internal/services/apimanagement/tests/resource_arm_api_management_logger_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand Down Expand Up @@ -41,10 +40,6 @@ func TestAccAzureRMApiManagementLogger_basicEventHub(t *testing.T) {
}

func TestAccAzureRMApiManagementLogger_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_logger", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions .../services/apimanagement/tests/resource_arm_api_management_openid_connect_provider_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMApiManagementOpenIDConnectProvider_basic(t *testing.T) {
}

func TestAccAzureRMApiManagementOpenIDConnectProvider_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_openid_connect_provider", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...erm/internal/services/apimanagement/tests/resource_arm_api_management_product_api_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMAPIManagementProductApi_basic(t *testing.T) {
}

func TestAccAzureRMAPIManagementProductApi_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_product_api", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down
5 changes: 0 additions & 5 deletions ...m/internal/services/apimanagement/tests/resource_arm_api_management_product_group_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import (
"github.com/hashicorp/terraform-plugin-sdk/terraform"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/acceptance"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/clients"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/features"
"github.com/terraform-providers/terraform-provider-azurerm/azurerm/utils"
)

Expand All @@ -32,10 +31,6 @@ func TestAccAzureRMAPIManagementProductGroup_basic(t *testing.T) {
}

func TestAccAzureRMAPIManagementProductGroup_requiresImport(t *testing.T) {
if !features.ShouldResourcesBeImported() {
t.Skip("Skipping since resources aren't required to be imported")
return
}
data := acceptance.BuildTestData(t, "azurerm_api_management_product_group", "test")

resource.ParallelTest(t, resource.TestCase{
Expand Down

0 comments on commit c1707b0

Please sign in to comment.