fixup

Author: benzekrimaha

tests/functional/aws-node-sdk/test/service/get.js

@@ -22,7 +22,9 @@ async function cleanBucket(bucketUtils, s3, Bucket) {
         await bucketUtils.empty(Bucket, true);
         await bucketUtils.deleteOne(Bucket);
     } catch (error) {
-        console.error(`Error cleaning bucket ${Bucket}:`, error);
+        process.stdout
+            .write(`Error emptying and deleting bucket: ${error}\n`);
+        // ignore the error and continue
     }
 }
 
@@ -39,7 +41,6 @@ async function cleanAllBuckets(bucketUtils, s3) {
 
         // clean sequentially to avoid overloading
         for (const bucket of list.Buckets) {
-            console.log(`Cleaning bucket: ${bucket.Name}`);
             await cleanBucket(bucketUtils, s3, bucket.Name);
         }
     }
@@ -127,7 +128,6 @@ describeFn('GET Service - AWS.S3.listBuckets', function getService() {
                 .map(i => `getservicebuckets-${i}`);
 
             before(done => {
-                console.log('sigCfg', sigCfg);
                 bucketUtil = new BucketUtility('default', sigCfg);
                 s3 = bucketUtil.s3;
                 async.series([
@@ -143,13 +143,13 @@ describeFn('GET Service - AWS.S3.listBuckets', function getService() {
                                     moveOn();
                                 })
                                 .catch(err => {
-                                    console.log('Error creating bucket:', err);
                                     moveOn(err);
                                 });
                         },
                         err => {
                             if (err) {
-                                process.stdout.write(`err creating buckets: ${err}`);
+                                process.stdout.write(`err creating buckets: ${err}\n`);
+                                return next(err);
                             }
                             next(err);
                         })
@@ -167,7 +167,9 @@ describeFn('GET Service - AWS.S3.listBuckets', function getService() {
                             }
                             moveOn();
                         })
-                        .catch(moveOn);
+                        .catch(() => {
+                            moveOn();
+                        });
                 },
                 err => {
                     if (err) {
@@ -177,11 +179,15 @@ describeFn('GET Service - AWS.S3.listBuckets', function getService() {
                 });
             });
 
-            it.only('should list buckets concurrently', done => {
+            it('should list buckets concurrently', done => {
                 async.times(20, (n, next) => {
                     s3.send(new ListBucketsCommand({}))
                         .then(result => {
-                            assert.equal(result.Buckets.length,
+                            // Filter for our test buckets only
+                            const ourBuckets = result.Buckets.filter(bucket => 
+                                bucket.Name.startsWith('getservicebuckets-')
+                            );
+                            assert.equal(ourBuckets.length,
                                 createdBuckets.length,
                                 'Created buckets are missing in response');
                             next();