Skip to content

Commit cd307ec

Browse files
committed
fixup
1 parent d556342 commit cd307ec

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

tests/functional/sse-kms-migration/helpers.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ const s3config = {
6868

6969
const s3Client = new S3Client(s3config);
7070

71+
// Remove logger middleware to avoid noisy logs during testing
7172
if (s3Client.middlewareStack.identify().includes('loggerMiddleware')) {
7273
s3Client.middlewareStack.remove('loggerMiddleware');
7374
}

tests/functional/sse-kms-migration/scenarios.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -280,12 +280,11 @@ async function mpuUploadPartCopy(
280280
// before has no headers to assert
281281
async function mpuComplete({ UploadId, Bucket, Key }, { existingParts, newParts }, mpuOverviewMDSSE, algo, testCase) {
282282
const extractETag = part => {
283-
const eTag = part.CopyPartResult?.ETag || part.ETag;
284-
assert(eTag, `Could not find ETag in part: ${JSON.stringify(part)}`);
283+
const eTag = part.CopyPartResult?.ETag || part.ETag || undefined;
284+
assert(eTag !== undefined, `Could not find ETag in part: ${JSON.stringify(part)}`);
285285
return eTag;
286286
};
287287

288-
// Build the parts array with proper ETag extraction
289288
const allParts = [
290289
...existingParts.map(part => ({
291290
PartNumber: part.PartNumber,

0 commit comments

Comments
 (0)