Add tests

Issue: CLDSRV-613

Author: Seragonia

tests/unit/api/apiUtils/coldStorage.js

@@ -243,5 +243,16 @@ describe('cold storage', () => {
                 done();
             });
         });
+
+        it('should fail if _updateRestoreInfo fails', done => {
+            const objectMd = new ObjectMD().setDataStoreName(
+                'location-dmf-v1'
+            ).setArchive(false).getValue();
+
+            startRestore(objectMd, { days: 7 }, log, err => {
+                assert.deepStrictEqual(err, errors.InternalError);
+                done();
+            });
+        });
     });
 });

tests/unit/api/objectDelete.js

@@ -8,7 +8,7 @@ const bucketPutACL = require('../../../lib/api/bucketPutACL');
 const constants = require('../../../constants');
 const { cleanup, DummyRequestLogger, makeAuthInfo } = require('../helpers');
 const objectPut = require('../../../lib/api/objectPut');
-const { objectDelete } = require('../../../lib/api/objectDelete');
+const { objectDelete, objectDeleteInternal } = require('../../../lib/api/objectDelete');
 const objectGet = require('../../../lib/api/objectGet');
 const DummyRequest = require('../DummyRequest');
 const mpuUtils = require('../utils/mpuUtils');
@@ -210,6 +210,38 @@ describe('objectDelete API', () => {
         });
     });
 
+    it('should log when expiration is trying to delete a master version', done => {
+        const warnStub = sinon.stub(log, 'warn');
+        const testBucketPutVersionRequest = new DummyRequest({
+            bucketName,
+            namespace,
+            headers: { 'x-amz-bucket-object-lock-enabled': 'true' },
+            url: `/${bucketName}`,
+        });
+
+        bucketPut(authInfo, testBucketPutVersionRequest, log, () => {
+            objectPut(authInfo, testPutObjectRequest,
+                undefined, log, (err, data) => {
+                    const deleteObjectVersionRequest = new DummyRequest({
+                        bucketName,
+                        namespace,
+                        objectKey,
+                        headers: {},
+                        url: `/${bucketName}/${objectKey}?versionId=${data['x-amz-version-id']}`,
+                        query: {
+                            versionId: data['x-amz-version-id'],
+                        },
+                    });
+                    objectDeleteInternal(authInfo, deleteObjectVersionRequest, log, true, err => {
+                        assert.strictEqual(err, null);
+                        sinon.assert.calledWith(warnStub, 'expiration is trying to delete a master version ' +
+                            'of an object with versioning enabled');
+                        done();
+                    });
+                });
+        });
+    });
+
     describe('with \'modified\' headers', () => {
         beforeEach(done => {
             bucketPut(authInfo, testBucketPutRequest, log, () => {