fix mongodb testcases (#1771)

hellertang · web-flow · commit b7cb474136d0 · 2023-05-15T11:48:40.000+08:00
diff --git a/tencentcloud/data_source_tc_mongodb_instance_backups_test.go b/tencentcloud/data_source_tc_mongodb_instance_backups_test.go
@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceBackupsDataSource_basic(t *testing.T) {
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstanceBackupsDataSource,
-				Check:  resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_backups.instance_backups")),
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstanceBackupsDataSource,
+				Check:     resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_backups.instance_backups")),
 			},
 		},
 	})
diff --git a/tencentcloud/data_source_tc_mongodb_instance_connections_test.go b/tencentcloud/data_source_tc_mongodb_instance_connections_test.go
@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceConnectionsDataSource_basic(t *testing.T)
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstanceConnectionsDataSource,
-				Check:  resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_connections.instance_connections")),
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstanceConnectionsDataSource,
+				Check:     resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_connections.instance_connections")),
 			},
 		},
 	})
diff --git a/tencentcloud/data_source_tc_mongodb_instance_current_op_test.go b/tencentcloud/data_source_tc_mongodb_instance_current_op_test.go
@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceCurrentOpDataSource_basic(t *testing.T) {
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstanceCurrentOpDataSource,
-				Check:  resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_current_op.instance_current_op")),
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstanceCurrentOpDataSource,
+				Check:     resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_current_op.instance_current_op")),
 			},
 		},
 	})
diff --git a/tencentcloud/data_source_tc_mongodb_instance_params_test.go b/tencentcloud/data_source_tc_mongodb_instance_params_test.go
@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceParamsDataSource_basic(t *testing.T) {
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstanceParamsDataSource,
-				Check:  resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_params.instance_params")),
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstanceParamsDataSource,
+				Check:     resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_params.instance_params")),
 			},
 		},
 	})
diff --git a/tencentcloud/data_source_tc_mongodb_instance_slow_log_test.go b/tencentcloud/data_source_tc_mongodb_instance_slow_log_test.go
@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceSlowLogDataSource_basic(t *testing.T) {
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstanceSlowLogDataSource,
-				Check:  resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_slow_log.instance_slow_log")),
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstanceSlowLogDataSource,
+				Check:     resource.ComposeTestCheckFunc(testAccCheckTencentCloudDataSourceID("data.tencentcloud_mongodb_instance_slow_log.instance_slow_log")),
 			},
 		},
 	})
diff --git a/tencentcloud/data_source_tc_mongodb_instances_test.go b/tencentcloud/data_source_tc_mongodb_instances_test.go
@@ -14,7 +14,8 @@ func TestAccTencentCloudMongodbInstancesDataSource(t *testing.T) {
 		CheckDestroy: testAccCheckMongodbInstanceDestroy,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstancesDataSource,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstancesDataSource,
 				Check: resource.ComposeAggregateTestCheckFunc(
 					resource.TestCheckResourceAttrSet("data.tencentcloud_mongodb_instances.mongodb_instances", "instance_list.#"),
 				),
diff --git a/tencentcloud/data_source_tc_mongodb_zone_config_test.go b/tencentcloud/data_source_tc_mongodb_zone_config_test.go
@@ -13,7 +13,8 @@ func TestAccTencentCloudMongodbZoneConfigDataSource(t *testing.T) {
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbZoneConfigDataSource,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbZoneConfigDataSource,
 				Check: resource.ComposeAggregateTestCheckFunc(
 					resource.TestCheckResourceAttrSet("data.tencentcloud_mongodb_zone_config.zone_config", "list.#"),
 					resource.TestCheckResourceAttr("data.tencentcloud_mongodb_zone_config.zone_config", "list.0.available_zone", defaultAZone),
diff --git a/tencentcloud/resource_tc_mongodb_instance_backup_download_task_test.go b/tencentcloud/resource_tc_mongodb_instance_backup_download_task_test.go
@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceBackupDownloadTaskResource_basic(t *testi
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstanceBackupDownloadTask,
-				Check:  resource.ComposeTestCheckFunc(resource.TestCheckResourceAttrSet("tencentcloud_mongodb_instance_backup_download_task.instance_backup_download_task", "id")),
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstanceBackupDownloadTask,
+				Check:     resource.ComposeTestCheckFunc(resource.TestCheckResourceAttrSet("tencentcloud_mongodb_instance_backup_download_task.instance_backup_download_task", "id")),
 			},
 			{
 				ResourceName:      "tencentcloud_mongodb_instance_backup_download_task.instance_backup_download_task",
diff --git a/tencentcloud/resource_tc_mongodb_instance_test.go b/tencentcloud/resource_tc_mongodb_instance_test.go
@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"log"
+	"strings"
 	"testing"
 	"time"
 
@@ -33,6 +34,11 @@ func init() {
 				ins := instances[i]
 				id := *ins.InstanceId
 				name := *ins.InstanceName
+
+				if strings.HasPrefix(name, keepResource) || strings.HasPrefix(name, defaultResource) {
+					continue
+				}
+
 				created, err := time.Parse("2006-01-02 15:04:05", *ins.CreateTime)
 				if err != nil {
 					created = time.Time{}
@@ -69,7 +75,8 @@ func TestAccTencentCloudMongodbInstanceResourcePostPaid(t *testing.T) {
 		CheckDestroy: testAccCheckMongodbInstanceDestroy,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstance,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstance,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_instance.mongodb"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_instance.mongodb", "instance_name", "tf-mongodb-test"),
@@ -99,7 +106,8 @@ func TestAccTencentCloudMongodbInstanceResourcePostPaid(t *testing.T) {
 					log.Printf("[WARN] MongoDB Update Need DealID query available, skip checking.")
 					return true, nil
 				},
-				Config: testAccMongodbInstance_update,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbInstance_update,
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_instance.mongodb", "instance_name", "tf-mongodb-update"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_instance.mongodb", "memory", "8"),
@@ -119,7 +127,8 @@ func TestAccTencentCloudNeedFixMongodbInstanceResourcePrepaid(t *testing.T) {
 		Providers: testAccProviders,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbInstancePrepaid,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_PREPAY) },
+				Config:    testAccMongodbInstancePrepaid,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_instance.mongodb_prepaid"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_instance.mongodb_prepaid", "instance_name", "tf-mongodb-test-prepaid"),
@@ -140,7 +149,8 @@ func TestAccTencentCloudNeedFixMongodbInstanceResourcePrepaid(t *testing.T) {
 				),
 			},
 			{
-				Config: testAccMongodbInstancePrepaid_update,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_PREPAY) },
+				Config:    testAccMongodbInstancePrepaid_update,
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_instance.mongodb_prepaid", "instance_name", "tf-mongodb-test-prepaid-update"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_instance.mongodb_prepaid", "memory", "4"),
diff --git a/tencentcloud/resource_tc_mongodb_sharding_instance_test.go b/tencentcloud/resource_tc_mongodb_sharding_instance_test.go
@@ -17,7 +17,8 @@ func TestAccTencentCloudMongodbShardingInstanceResource_postpaid(t *testing.T) {
 		CheckDestroy: testAccCheckMongodbShardingInstanceDestroy,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbShardingInstance,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbShardingInstance,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_sharding_instance.mongodb"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_sharding_instance.mongodb", "instance_name", "tf-mongodb-sharding"),
@@ -42,7 +43,8 @@ func TestAccTencentCloudMongodbShardingInstanceResource_postpaid(t *testing.T) {
 				),
 			},
 			{
-				Config: testAccMongodbShardingInstanceUpdate,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbShardingInstanceUpdate,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_sharding_instance.mongodb"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_sharding_instance.mongodb", "instance_name", "tf-mongodb-sharding-update"),
@@ -68,7 +70,8 @@ func TestAccTencentCloudMongodbShardingInstanceResource_prepaid(t *testing.T) {
 		CheckDestroy: testAccCheckMongodbShardingInstanceDestroy,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbShardingInstancePrepaid,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_PREPAY) },
+				Config:    testAccMongodbShardingInstancePrepaid,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_sharding_instance.mongodb_prepaid"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_sharding_instance.mongodb_prepaid", "instance_name", "tf-mongodb-sharding-prepaid"),
@@ -89,7 +92,8 @@ func TestAccTencentCloudMongodbShardingInstanceResource_prepaid(t *testing.T) {
 				),
 			},
 			{
-				Config: testAccMongodbShardingInstancePrepaid_update,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_PREPAY) },
+				Config:    testAccMongodbShardingInstancePrepaid_update,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_sharding_instance.mongodb_prepaid"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_sharding_instance.mongodb_prepaid", "instance_name", "tf-mongodb-sharding-prepaid-update"),
diff --git a/tencentcloud/resource_tc_mongodb_standby_instance_test.go b/tencentcloud/resource_tc_mongodb_standby_instance_test.go
@@ -17,7 +17,8 @@ func TestAccTencentCloudNeedFixMongodbStandbyInstanceResource(t *testing.T) {
 		CheckDestroy: testAccCheckMongodbStandbyInstanceDestroy,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccMongodbStandbyInstance,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbStandbyInstance,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_standby_instance.mongodb"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_standby_instance.mongodb", "instance_name", "tf-mongodb-standby-test"),
@@ -39,7 +40,8 @@ func TestAccTencentCloudNeedFixMongodbStandbyInstanceResource(t *testing.T) {
 				),
 			},
 			{
-				Config: testAccMongodbStandbyInstance_update,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },
+				Config:    testAccMongodbStandbyInstance_update,
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_standby_instance.mongodb", "instance_name", "tf-mongodb-standby-test-update"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_standby_instance.mongodb", "memory", "8"),
@@ -55,7 +57,8 @@ func TestAccTencentCloudNeedFixMongodbStandbyInstanceResource(t *testing.T) {
 				ImportStateVerifyIgnore: []string{"security_groups", "auto_renew_flag", "password"},
 			},
 			{
-				Config: testAccMongodbStandbyInstancePrepaid,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_PREPAY) },
+				Config:    testAccMongodbStandbyInstancePrepaid,
 				Check: resource.ComposeTestCheckFunc(
 					testAccCheckMongodbInstanceExists("tencentcloud_mongodb_standby_instance.mongodb_prepaid"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_standby_instance.mongodb_prepaid", "instance_name", "tf-mongodb-standby-test-prepaid"),
@@ -78,7 +81,8 @@ func TestAccTencentCloudNeedFixMongodbStandbyInstanceResource(t *testing.T) {
 				),
 			},
 			{
-				Config: testAccMongodbStandbyInstancePrepaid_update,
+				PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_PREPAY) },
+				Config:    testAccMongodbStandbyInstancePrepaid_update,
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_standby_instance.mongodb_prepaid", "instance_name", "tf-mongodb-standby-prepaid-update"),
 					resource.TestCheckResourceAttr("tencentcloud_mongodb_standby_instance.mongodb_prepaid", "memory", "4"),

Original file line number	Diff line number	Diff line change
`@@ -15,8 +15,9 @@ func TestAccTencentCloudMongodbInstanceBackupDownloadTaskResource_basic(t *testi`
`15`	`15`	`Providers: testAccProviders,`
`16`	`16`	`Steps: []resource.TestStep{`
`17`	`17`	`{`
`18`		`- Config: testAccMongodbInstanceBackupDownloadTask,`
`19`		`- Check: resource.ComposeTestCheckFunc(resource.TestCheckResourceAttrSet("tencentcloud_mongodb_instance_backup_download_task.instance_backup_download_task", "id")),`
	`18`	`+ PreConfig: func() { testAccStepPreConfigSetTempAKSK(t, ACCOUNT_TYPE_COMMON) },`
	`19`	`+ Config: testAccMongodbInstanceBackupDownloadTask,`
	`20`	`+ Check: resource.ComposeTestCheckFunc(resource.TestCheckResourceAttrSet("tencentcloud_mongodb_instance_backup_download_task.instance_backup_download_task", "id")),`
`20`	`21`	`},`
`21`	`22`	`{`
`22`	`23`	`ResourceName: "tencentcloud_mongodb_instance_backup_download_task.instance_backup_download_task",`