Skip to content

Commit 86b15fa

Browse files
authored
Merge pull request #10317 from Jukie/redshift-refactor-randomString
Refactor: Use acctest.RandStringFromCharSet() instead of randomString() in redshift tests
2 parents 232c4df + 7341956 commit 86b15fa

3 files changed

+3
-3
lines changed

aws/resource_aws_redshift_parameter_group_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ func TestResourceAWSRedshiftParameterGroupNameValidation(t *testing.T) {
185185
ErrCount: 1,
186186
},
187187
{
188-
Value: randomString(256),
188+
Value: acctest.RandStringFromCharSet(256, acctest.CharSetAlpha),
189189
ErrCount: 1,
190190
},
191191
}

aws/resource_aws_redshift_security_group_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ func TestResourceAWSRedshiftSecurityGroupNameValidation(t *testing.T) {
295295
ErrCount: 1,
296296
},
297297
{
298-
Value: randomString(256),
298+
Value: acctest.RandStringFromCharSet(256, acctest.CharSetAlpha),
299299
ErrCount: 1,
300300
},
301301
}

aws/resource_aws_redshift_subnet_group_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ func TestResourceAWSRedshiftSubnetGroupNameValidation(t *testing.T) {
184184
ErrCount: 1,
185185
},
186186
{
187-
Value: randomString(256),
187+
Value: acctest.RandStringFromCharSet(256, acctest.CharSetAlpha),
188188
ErrCount: 1,
189189
},
190190
}

0 commit comments

Comments
 (0)