Fix double response on conditional batchDelete

Issue: CLDSRV-394

Author: francoisferrand

lib/routes/routeBackbeat.js

@@ -311,7 +311,7 @@ function handleTaggingOperation(request, response, type, dataStoreVersionId,
         const dataRetrievalInfo = {
             versionId: dataStoreVersionId,
         };
-        return _respond(response, dataRetrievalInfo, log, callback);
+        return response ? _respond(response, dataRetrievalInfo, log, callback) : callback();
     });
 }
 
@@ -1135,17 +1135,16 @@ function _azureConditionalDelete(request, response, log, cb) {
 }
 
 function _putTagging(request, response, log, cb) {
-    return handleTaggingOperation(
-        request, response, 'Put', undefined, log, err => {
-            if (err) {
-                log.error('put tagging failed', {
-                    method: '_putTagging',
-                    error: err,
-                });
-                return cb(err);
-            }
-            return _respond(response, null, log, cb);
-        });
+    return handleTaggingOperation(request, null, 'Put', undefined, log, err => {
+        if (err) {
+            log.error('put tagging failed', {
+                method: '_putTagging',
+                error: err,
+            });
+            return cb(err);
+        }
+        return _respond(response, null, log, cb);
+    });
 }
 
 function _conditionalTagging(request, response, locations, log, cb) {

tests/multipleBackend/routes/routeBackbeat.js

@@ -1650,8 +1650,6 @@ describe('backbeat routes', () => {
         it('should PUT tags for a non-versioned bucket', function test(done) {
             this.timeout(10000);
             const bucket = NONVERSIONED_BUCKET;
-            const awsBucket =
-                  config.locationConstraints[awsLocation].details.bucketName;
             const awsKey = uuidv4();
             async.waterfall([
                 next =>
@@ -2337,9 +2335,7 @@ describe('backbeat routes', () => {
 
         // The ceph image does not support putting tags during initiate MPU.
         itSkipCeph('should put tags if the source is AWS and tags are ' +
-        'provided when initiating the multipart upload', done => {
-            const awsBucket =
-                config.locationConstraints[awsLocation].details.bucketName;
+            'provided when initiating the multipart upload', done => {
             const awsKey = uuidv4();
             const multipleBackendPath =
                 `/_/backbeat/multiplebackenddata/${awsBucket}/${awsKey}`;