Skip to content

Commit

Permalink
Merge pull request #2366 from isaiahvita/remove-redundant-outpost-tests
Browse files Browse the repository at this point in the history
Remove S3 outposts dual stack unit tests
  • Loading branch information
wty-Bryant committed Nov 13, 2023
2 parents 27137a8 + 8f609ef commit 5a1ec8a
Showing 1 changed file with 0 additions and 17 deletions.
17 changes: 0 additions & 17 deletions service/s3control/internal/customizations/update_endpoint_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -298,15 +298,6 @@ func TestEndpointWithARN(t *testing.T) {
expectedHeaderForOutpostID: "op-01234567890123456",
expectedHeaderForAccountID: true,
},
"Outpost AccessPoint with DualStack": {
accessPoint: "arn:aws:s3-outposts:us-west-2:123456789012:outpost:op-01234567890123456:accesspoint:myaccesspoint",
options: s3control.Options{
Region: "us-west-2",
UseARNRegion: true,
UseDualstack: true,
},
expectedErr: "Invalid configuration: Outpost Access Points do not support dual-stack",
},
"Invalid outpost resource format": {
bucket: "arn:aws:s3-outposts:us-west-2:123456789012:outpost",
options: s3control.Options{
Expand Down Expand Up @@ -439,14 +430,6 @@ func TestEndpointWithARN(t *testing.T) {
expectedHeaderForOutpostID: "op-01234567890123456",
expectedHeaderForAccountID: true,
},
"Outpost Bucket with DualStack": {
bucket: "arn:aws:s3-outposts:us-west-2:123456789012:outpost:op-01234567890123456:bucket:mybucket",
options: s3control.Options{
Region: "us-west-2",
UseDualstack: true,
},
expectedErr: "Invalid configuration: Outpost buckets do not support dual-stack",
},
"Missing bucket id": {
bucket: "arn:aws:s3-outposts:us-west-2:123456789012:outpost:op-01234567890123456:bucket",
options: s3control.Options{
Expand Down

0 comments on commit 5a1ec8a

Please sign in to comment.