CLDSRV-717: Test lifecycle Transition condition

For S3C Integration that don't support Transition

Author: BourgoisMickael

tests/functional/aws-node-sdk/test/bucket/putBucketLifecycle.js

@@ -484,6 +484,9 @@ describe('aws-sdk test put bucket lifecycle', () => {
         });
 
         describe('with Transitions', () => {
+            // Integration for S3C that doesn't support Transitions expects NotImplemented
+            const isSupported = !!process.env.S3_END_TO_END;
+
             // Get lifecycle request params with Transitions.
             function getParams(transitions) {
                 const rule = {
@@ -505,7 +508,11 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.ifError(err);
+                    if (isSupported) {
+                        assert.ifError(err);
+                    } else {
+                        assert.strictEqual(err && err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -517,7 +524,11 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.strictEqual(err.code, 'MalformedXML');
+                    if (isSupported) {
+                        assert.strictEqual(err.code, 'MalformedXML');
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -529,9 +540,13 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.strictEqual(err.code, 'InvalidArgument');
-                    assert.strictEqual(err.message,
-                        "'Days' in Transition action must be nonnegative");
+                    if (isSupported) {
+                        assert.strictEqual(err.code, 'InvalidArgument');
+                        assert.strictEqual(err.message,
+                            "'Days' in Transition action must be nonnegative");
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -546,10 +561,14 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.strictEqual(err.code, 'InvalidRequest');
-                    assert.strictEqual(err.message,
-                        "'StorageClass' must be different for 'Transition' " +
-                        "actions in same 'Rule' with prefix ''");
+                    if (isSupported) {
+                        assert.strictEqual(err.code, 'InvalidRequest');
+                        assert.strictEqual(err.message,
+                            "'StorageClass' must be different for 'Transition' " +
+                            "actions in same 'Rule' with prefix ''");
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -562,7 +581,11 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.ifError(err);
+                    if (isSupported) {
+                        assert.ifError(err);
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -577,7 +600,11 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.strictEqual(err.code, 'MalformedXML');
+                    if (isSupported) {
+                        assert.strictEqual(err.code, 'MalformedXML');
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -594,10 +621,14 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 }];
                 const params = getParams(transitions);
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.strictEqual(err.code, 'InvalidRequest');
-                    assert.strictEqual(err.message,
-                        "Found mixed 'Date' and 'Days' based Transition " +
-                        "actions in lifecycle rule for prefix ''");
+                    if (isSupported) {
+                        assert.strictEqual(err.code, 'InvalidRequest');
+                        assert.strictEqual(err.message,
+                            "Found mixed 'Date' and 'Days' based Transition " +
+                            "actions in lifecycle rule for prefix ''");
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });
@@ -612,10 +643,14 @@ describe('aws-sdk test put bucket lifecycle', () => {
                 const params = getParams(transitions);
                 params.LifecycleConfiguration.Rules[0].Expiration = { Date: 0 };
                 s3.putBucketLifecycleConfiguration(params, err => {
-                    assert.strictEqual(err.code, 'InvalidRequest');
-                    assert.strictEqual(err.message,
-                        "Found mixed 'Date' and 'Days' based Expiration and " +
-                        "Transition actions in lifecycle rule for prefix ''");
+                    if (isSupported) {
+                        assert.strictEqual(err.code, 'InvalidRequest');
+                        assert.strictEqual(err.message,
+                            "Found mixed 'Date' and 'Days' based Expiration and " +
+                            "Transition actions in lifecycle rule for prefix ''");
+                    } else {
+                        assert.strictEqual(err.code, 'NotImplemented');
+                    }
                     done();
                 });
             });