Skip to content

Commit cdc2126

Browse files
authored
test: Fixes streams connection kafka update tests (#3479)
1 parent d80ef21 commit cdc2126

File tree

1 file changed

+15
-8
lines changed

1 file changed

+15
-8
lines changed

internal/service/streamconnection/resource_stream_connection_test.go

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import (
55
_ "embed"
66
"fmt"
77
"os"
8+
"regexp"
89
"testing"
910

1011
"github.com/hashicorp/terraform-plugin-testing/helper/resource"
@@ -88,12 +89,6 @@ func TestAccStreamRSStreamConnection_kafkaNetworkingVPC(t *testing.T) {
8889
ProtoV6ProviderFactories: acc.TestAccProviderV6Factories,
8990
CheckDestroy: CheckDestroyStreamConnection,
9091
Steps: []resource.TestStep{
91-
{
92-
Config: networkPeeringConfig + configureKafka(projectID, instanceName, "user", "rawpassword", "localhost:9092", "earliest", kafkaNetworkingPublic, true),
93-
Check: resource.ComposeAggregateTestCheckFunc(
94-
checkKafkaAttributes(resourceName, instanceName, "user", "rawpassword", "localhost:9092", "earliest", networkingTypePublic, true, true),
95-
),
96-
},
9792
{
9893
Config: networkPeeringConfig + configureKafka(projectID, instanceName, "user", "rawpassword", "localhost:9092", "earliest", kafkaNetworkingVPC, true),
9994
Check: checkKafkaAttributes(resourceName, instanceName, "user", "rawpassword", "localhost:9092", "earliest", networkingTypeVPC, true, true),
@@ -111,8 +106,15 @@ func TestAccStreamRSStreamConnection_kafkaNetworkingVPC(t *testing.T) {
111106

112107
func TestAccStreamRSStreamConnection_kafkaSSL(t *testing.T) {
113108
var (
114-
projectID = acc.ProjectIDExecution(t)
115-
instanceName = acc.RandomName()
109+
projectID = acc.ProjectIDExecution(t)
110+
instanceName = acc.RandomName()
111+
vpcID = os.Getenv("AWS_VPC_ID")
112+
vpcCIDRBlock = os.Getenv("AWS_VPC_CIDR_BLOCK")
113+
awsAccountID = os.Getenv("AWS_ACCOUNT_ID")
114+
containerRegion = os.Getenv("AWS_REGION")
115+
peerRegion = conversion.MongoDBRegionToAWSRegion(containerRegion)
116+
providerName = "AWS"
117+
networkPeeringConfig = configNetworkPeeringAWS(projectID, providerName, vpcID, awsAccountID, vpcCIDRBlock, containerRegion, peerRegion)
116118
)
117119
resource.ParallelTest(t, resource.TestCase{
118120
PreCheck: func() { acc.PreCheckBasic(t) },
@@ -123,6 +125,11 @@ func TestAccStreamRSStreamConnection_kafkaSSL(t *testing.T) {
123125
Config: configureKafka(projectID, instanceName, "user", "rawpassword", "localhost:9092", "earliest", kafkaNetworkingPublic, true),
124126
Check: checkKafkaAttributes(resourceName, instanceName, "user", "rawpassword", "localhost:9092", "earliest", networkingTypePublic, true, true),
125127
},
128+
// cannot change networking access type once set
129+
{
130+
Config: networkPeeringConfig + configureKafka(projectID, instanceName, "user", "rawpassword", "localhost:9092", "earliest", kafkaNetworkingVPC, true),
131+
ExpectError: regexp.MustCompile("STREAM_NETWORKING_ACCESS_TYPE_CANNOT_BE_MODIFIED"),
132+
},
126133
{
127134
ResourceName: resourceName,
128135
ImportStateIdFunc: checkStreamConnectionImportStateIDFunc(resourceName),

0 commit comments

Comments
 (0)