[to be removed] debug logs on mpu

Author: benzekrimaha

tests/multipleBackend/multipartUpload.js

@@ -37,11 +37,9 @@ const fileLocation = 'scality-internal-file';
 const awsLocation = 'awsbackend';
 const awsLocationMismatch = 'awsbackendmismatch';
 const awsConfig = getRealAwsConfig(awsLocation);
-console.log('[INIT] AWS Config for awsbackend:', JSON.stringify(awsConfig, null, 2));
-console.log('[INIT] Location constraints awsbackend:', JSON.stringify(config.locationConstraints[awsLocation], null, 2));
-console.log('[INIT] Location constraints awsbackendmismatch:', JSON.stringify(config.locationConstraints[awsLocationMismatch], null, 2));
 const s3 = new S3Client(awsConfig);
-const log = new DummyRequestLogger();
+// Use real logger instead of DummyRequestLogger to see Arsenal's error logs
+const log = logger.newRequestLogger();
 
 const fakeUploadId = 'fakeuploadid';
 const canonicalID = 'accessKey1';
@@ -265,26 +263,20 @@ function assertObjOnBackend(expectedBackend, objectKey, cb) {
 }
 
 function putParts(uploadId, key, cb) {
-    console.log(`[DEBUG] putParts called - uploadId: ${uploadId}, key: ${key}`);
     const putPartParam1 = getPartParams(key, uploadId, 1);
     const partRequest = new DummyRequest(putPartParam1, bigBody);
-    console.log(`[DEBUG] Putting part 1`);
     objectPutPart(authInfo, partRequest, undefined, log, err => {
-        console.log(`[DEBUG] Part 1 put callback - err:`, err);
         assert.equal(err, null, `Error putting part: ${err}`);
         const putPartParam2 = getPartParams(key, uploadId, 2);
         const partRequest2 = new DummyRequest(putPartParam2, smallBody);
-        console.log(`[DEBUG] Putting part 2`);
         objectPutPart(authInfo, partRequest2, undefined, log, err => {
-            console.log(`[DEBUG] Part 2 put callback - err:`, err);
             assert.equal(err, null, `Error putting part: ${err}`);
             cb();
         });
     });
 }
 
 function mpuSetup(location, key, cb) {
-    console.log(`[DEBUG] mpuSetup called with location: ${location}, key: ${key}`);
     const initiateRequest = {
         bucketName,
         namespace,
@@ -295,15 +287,11 @@ function mpuSetup(location, key, cb) {
         parsedHost: 'localhost',
         actionImplicitDenies: false,
     };
-    console.log(`[DEBUG] Calling initiateMultipartUpload for ${location}`);
     initiateMultipartUpload(authInfo, initiateRequest, log,
     (err, result) => {
-        console.log(`[DEBUG] initiateMultipartUpload callback - err:`, err, 'result:', result ? 'present' : 'null');
         assert.strictEqual(err, null, 'Error initiating MPU');
         assertMpuInitResults(result, key, uploadId => {
-            console.log(`[DEBUG] assertMpuInitResults callback - uploadId: ${uploadId}`);
             putParts(uploadId, key, () => {
-                console.log(`[DEBUG] putParts completed`);
                 cb(uploadId);
             });
         });
@@ -391,11 +379,7 @@ describe('Multipart Upload API with AWS Backend', function mpuTestSuite() {
 
     it('should initiate a multipart upload on AWS location with ' +
     'bucketMatch equals false', done => {
-        console.log('[TEST START] Initiate MPU on AWS with bucketMatch=false');
         const objectKey = `key-${Date.now()}`;
-        console.log('[TEST] objectKey:', objectKey);
-        console.log('[TEST] awsLocationMismatch:', awsLocationMismatch);
-        console.log('[TEST] awsMismatchBucket:', awsMismatchBucket);
         const initiateRequest = {
             bucketName,
             namespace,
@@ -408,16 +392,10 @@ describe('Multipart Upload API with AWS Backend', function mpuTestSuite() {
             actionImplicitDenies: false,
         };
 
-        console.log('[TEST] Calling initiateMultipartUpload');
         initiateMultipartUpload(authInfo, initiateRequest, log,
         (err, result) => {
-            console.log('[TEST] initiateMultipartUpload returned - err:', err, 'result:', result ? 'present' : 'null');
-            if (err) {
-                console.error('[TEST] Error details:', JSON.stringify(err, null, 2));
-            }
             assert.strictEqual(err, null, 'Error initiating MPU');
             assertMpuInitResults(result, objectKey, uploadId => {
-                console.log('[TEST] About to call abortMPU with uploadId:', uploadId);
                 abortMPU(uploadId, getAwsParamsBucketNotMatch(objectKey), done);
             });
         });
@@ -450,9 +428,17 @@ describe('Multipart Upload API with AWS Backend', function mpuTestSuite() {
 
     it('should list the parts of a multipart upload on real AWS', done => {
         const objectKey = `key-${Date.now()}`;
+        console.log('[LIST TEST AWS] Starting listParts test');
         mpuSetup(awsLocation, objectKey, uploadId => {
+            console.log('[LIST TEST AWS] MPU setup completed, uploadId:', uploadId);
             const listParams = getListParams(objectKey, uploadId);
+            console.log('[LIST TEST AWS] listParams:', listParams);
+            console.log('[LIST TEST AWS] Calling listParts...');
             listParts(authInfo, listParams, log, (err, result) => {
+                console.log('[LIST TEST AWS] listParts returned - err:', err, 'result:', result ? 'present' : 'null');
+                if (err) {
+                    console.error('[LIST TEST AWS] Error details:', err);
+                }
                 assert.equal(err, null, `Error listing parts on AWS: ${err}`);
                 assertListResults(result, null, uploadId, objectKey);
                 abortMPU(uploadId, getAwsParams(objectKey), done);
@@ -463,9 +449,17 @@ describe('Multipart Upload API with AWS Backend', function mpuTestSuite() {
     it('should list the parts of a multipart upload on real AWS location ' +
     'with bucketMatch set to false', done => {
         const objectKey = `key-${Date.now()}`;
+        console.log('[LIST TEST] Starting listParts test with bucketMatch=false');
         mpuSetup(awsLocationMismatch, objectKey, uploadId => {
+            console.log('[LIST TEST] MPU setup completed, uploadId:', uploadId);
             const listParams = getListParams(objectKey, uploadId);
+            console.log('[LIST TEST] listParams:', listParams);
+            console.log('[LIST TEST] Calling listParts...');
             listParts(authInfo, listParams, log, (err, result) => {
+                console.log('[LIST TEST] listParts returned - err:', err, 'result:', result ? 'present' : 'null');
+                if (err) {
+                    console.error('[LIST TEST] Error details:', err);
+                }
                 assert.equal(err, null, `Error listing parts on AWS: ${err}`);
                 assertListResults(result, null, uploadId, objectKey);
                 abortMPU(uploadId, getAwsParamsBucketNotMatch(objectKey), done);
@@ -495,6 +489,7 @@ describe('Multipart Upload API with AWS Backend', function mpuTestSuite() {
             const listParams = getListParams(objectKey, uploadId);
             listParams.query['part-number-marker'] = '1';
             listParts(authInfo, listParams, log, (err, result) => {
+                console.log('[PART NUMBER MARKER TEST] listParts returned, err:', err);
                 assert.equal(err, null);
                 assertListResults(result, 'partNumMarker', uploadId, objectKey);
                 abortMPU(uploadId, getAwsParams(objectKey), done);
@@ -508,6 +503,7 @@ describe('Multipart Upload API with AWS Backend', function mpuTestSuite() {
             abortMPU(uploadId, getAwsParams(objectKey), () => {
                 const listParams = getListParams(objectKey, uploadId);
                 listParts(authInfo, listParams, log, err => {
+                    console.log('[LIST DELETED MPU TEST] listParts returned, err:', err);
                     let wantedDesc = 'Error returned from AWS: ' +
                         'The specified upload does not exist. The upload ID ' +
                         'may be invalid, or the upload may have been aborted' +