Skip to content

Commit 25d42a2

Browse files
committed
Fix metadata.getBucket callback to handle raftSessionId parameter
Update test calls to metadata.getBucket to properly handle the third raftSessionId parameter that is returned but not needed in these tests. This prevents the parameter from being incorrectly passed to the async series callback chain.
1 parent 43420a9 commit 25d42a2

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

tests/unit/api/bucketDeleteLifecycle.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ describe('deleteBucketLifecycle API', () => {
4747
async.series([
4848
next => bucketPutLifecycle(authInfo, _makeRequest(true), log, next),
4949
next => bucketDeleteLifecycle(authInfo, _makeRequest(), log, next),
50-
next => metadata.getBucket(bucketName, log, next),
50+
next => metadata.getBucket(bucketName, log, (err, bucket, raftSessionId) => next(err, bucket)),
5151
], (err, results) => {
5252
assert.equal(err, null, `Expected success, got error: ${err}`);
5353
const bucket = results[2];

tests/unit/api/bucketDeletePolicy.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ describe('deleteBucketPolicy API', () => {
5353
async.series([
5454
next => bucketPutPolicy(authInfo, _makeRequest(true), log, next),
5555
next => bucketDeletePolicy(authInfo, _makeRequest(), log, next),
56-
next => metadata.getBucket(bucketName, log, next),
56+
next => metadata.getBucket(bucketName, log, (err, bucket, raftSessionId) => next(err, bucket)),
5757
], (err, results) => {
5858
assert.equal(err, null, `Expected success, got error: ${err}`);
5959
const bucket = results[2];

0 commit comments

Comments
 (0)