-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws-sdk-php: fix AWS S3 compatible fixes #275
Conversation
ea64db4
to
43488ef
Compare
run/core/aws-sdk-php/quick-tests.php
Outdated
@@ -646,7 +646,7 @@ function testGetBucketLocation($s3Client, $params) { | |||
// Run failure tests. | |||
$params = [ | |||
// InvalidBucketName test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: could we change the comment to be "Bucket not found test" instead of "invalidbucketname tests"?
run/core/aws-sdk-php/quick-tests.php
Outdated
@@ -646,7 +646,7 @@ function testGetBucketLocation($s3Client, $params) { | |||
// Run failure tests. | |||
$params = [ | |||
// InvalidBucketName test | |||
'InvalidBucketName' => ['Bucket' => $bucket . '--'], | |||
'NoSuchBucket' => ['Bucket' => $bucket . '--'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we have a valid bucket name which is known up front to not exist during the test run instead of using an invalid bucket name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the reason is to verify S3 doesn't return invalid bucket name - since we know an invalid bucket name cannot exist - this is a good test still.
43488ef
to
1901fe3
Compare
No description provided.