Skip to content

Commit 51d6ad9

Browse files
committed
test: removing account id assertion for non BYOC accounts and removing debug code
1 parent e1c000e commit 51d6ad9

File tree

2 files changed

+0
-10
lines changed

2 files changed

+0
-10
lines changed

provider/pro/flatten.go

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -73,12 +73,6 @@ func FlattenCloudDetails(cloudDetails []*subscriptions.CloudDetail, isResource b
7373
var cdl []map[string]interface{}
7474

7575
for _, currentCloudDetail := range cloudDetails {
76-
// Debug logging for aws_account_id
77-
fmt.Printf("[DEBUG] FlattenCloudDetails: Provider=%s, CloudAccountID=%d, AWSAccountID=%v (nil=%v)\n",
78-
redis.StringValue(currentCloudDetail.Provider),
79-
redis.IntValue(currentCloudDetail.CloudAccountID),
80-
currentCloudDetail.AWSAccountID,
81-
currentCloudDetail.AWSAccountID == nil)
8276

8377
var regions []interface{}
8478
for _, currentRegion := range currentCloudDetail.Regions {
@@ -108,10 +102,7 @@ func FlattenCloudDetails(cloudDetails []*subscriptions.CloudDetail, isResource b
108102
}
109103

110104
if currentCloudDetail.AWSAccountID != nil {
111-
fmt.Printf("[DEBUG] FlattenCloudDetails: Setting aws_account_id=%s\n", redis.StringValue(currentCloudDetail.AWSAccountID))
112105
cdlMapString["aws_account_id"] = redis.StringValue(currentCloudDetail.AWSAccountID)
113-
} else {
114-
fmt.Printf("[DEBUG] FlattenCloudDetails: AWSAccountID is nil, NOT setting aws_account_id in state\n")
115106
}
116107

117108
cdl = append(cdl, cdlMapString)

provider/resource_rediscloud_pro_subscription_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ func TestAccResourceRedisCloudProSubscription_CRUDI(t *testing.T) {
5050
resource.TestCheckResourceAttr(resourceName, "payment_method", "credit-card"),
5151
resource.TestCheckResourceAttr(resourceName, "public_endpoint_access", "true"),
5252
resource.TestCheckResourceAttr(resourceName, "cloud_provider.0.provider", "AWS"),
53-
resource.TestCheckResourceAttrSet(resourceName, "cloud_provider.0.aws_account_id"),
5453
resource.TestCheckResourceAttr(resourceName, "cloud_provider.0.region.0.preferred_availability_zones.#", "1"),
5554
resource.TestCheckResourceAttrSet(resourceName, "cloud_provider.0.region.0.networks.0.networking_subnet_id"),
5655
resource.TestCheckResourceAttr(resourceName, "creation_plan.#", "1"),

0 commit comments

Comments
 (0)