Skip to content

Commit da7043b

Browse files
committed
post reviews fixups
1 parent c244695 commit da7043b

File tree

3 files changed

+1
-11
lines changed

3 files changed

+1
-11
lines changed

tests/functional/aws-node-sdk/test/multipleBackend/acl/aclAwsVersioning.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,7 @@ function putObjectAcl(s3, key, versionId, acp, cb) {
7474

7575
const command = new PutObjectAclCommand(params);
7676
s3.send(command)
77-
.then(() => {
78-
cb();
79-
})
77+
.then(() => cb())
8078
.catch(err => {
8179
assert.strictEqual(err, null, 'Expected success ' +
8280
`putting object acl, got error ${err}`);

tests/functional/aws-node-sdk/test/multipleBackend/delete/deleteAwsVersioning.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,6 @@ function _getAssertDeleted(s3, params, cb) {
150150
});
151151
}
152152

153-
// Update AWS S3 direct calls
154153
function _awsGetAssertDeleted(params, cb) {
155154
const { key, versionId, errorCode } = params;
156155
return getAwsRetry({ key, versionId }, 0, err => {

tests/functional/aws-node-sdk/test/multipleBackend/objectCopy/objectCopy.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -148,15 +148,8 @@ function testSuite() {
148148
await bucketUtil.empty(bucket);
149149
await bucketUtil.empty(bucketAws);
150150
await bucketUtil.empty(awsServerSideEncryptionbucket);
151-
152-
process.stdout.write(`Deleting bucket ${bucket}\n`);
153151
await bucketUtil.deleteOne(bucket);
154-
155-
process.stdout.write('Deleting bucket ' +
156-
`${awsServerSideEncryptionbucket}\n`);
157152
await bucketUtil.deleteOne(awsServerSideEncryptionbucket);
158-
159-
process.stdout.write(`Deleting bucket ${bucketAws}\n`);
160153
await bucketUtil.deleteOne(bucketAws);
161154
});
162155

0 commit comments

Comments
 (0)