added test for replication that should not happen

Author: SylvainSenechal

tests/ctst/features/crrReplicationS3utils.feature

@@ -2,15 +2,24 @@ Feature: Replication
     This feature tests replication using the script crrExistingObjects.js from S3utils,
     that replicates objects created before the replication was enabled on source the bucket.
 
+    @2.12.0
+    @PreMerge
+    @ReplicationTest
+    Scenario Outline: Objects created before setting up replication should not be replicated automatically
+        Given an existing bucket "source-bucket-0" "with" versioning, "without" ObjectLock "without" retention mode
+        And an object "source-object-1" that "exists"
+        And a replication configuration to "awsbackendmismatch" location
+        Then the object replication should eventually "never happen"
+
     @2.12.0
     @PreMerge
     @ReplicationTest
     Scenario Outline: Replicate objects created before creating the replication rule
-        Given an existing bucket "source-bucket" "with" versioning, "without" ObjectLock "without" retention mode
+        Given an existing bucket "source-bucket-1" "with" versioning, "without" ObjectLock "without" retention mode
         And an object "source-object-1" that "exists"
         And a replication configuration to "awsbackendmismatch" location
         When we run the job to replicate existing objects with status "NEW"
-        Then the object should eventually "be" replicated
+        Then the object replication should eventually "succeed"
         And the replicated object should be the same as the source object
 
     @2.12.0
@@ -21,8 +30,8 @@ Feature: Replication
         And a replication configuration to "awsbackendreplicationctstfail" location
         And a deleted destination bucket on that location
         And an object "source-object-2" that "exists"
-        Then the object should eventually "fail to be" replicated
+        Then the object replication should eventually "fail"
         When the destination bucket on the location is created again
         And we run the job to replicate existing objects with status "FAILED"
-        Then the object should eventually "be" replicated
+        Then the object replication should eventually "succeed"
         And the replicated object should be the same as the source object

tests/ctst/steps/replication.ts

@@ -88,8 +88,11 @@ When('we run the job to replicate existing objects with status {string}',
         await createAndRunPod(this, podManifest);
     });
 
-Then('the object should eventually {string} replicated', { timeout: 360_000 },
-    async function (this: Zenko, replicate: 'be' | 'fail to be') {
+Then('the object replication should eventually {string}', { timeout: 360_000 },
+    async function (
+        this: Zenko,
+        expectedOutcome: 'succeed' | 'fail' | 'never happen'
+    ) {
         const objectName = this.getSaved<string>('objectName');
         const bucketSource = this.getSaved<string>('bucketName');
         const startTime = Date.now();
@@ -111,12 +114,19 @@ Then('the object should eventually {string} replicated', { timeout: 360_000 },
             assert(parsed.ok);
             const replicationStatus = parsed.result?.ReplicationStatus;
 
-            if (replicate === 'be') {
+            if ((expectedOutcome === 'succeed' || expectedOutcome === 'fail') && 
+                (replicationStatus === 'PENDING' || replicationStatus === 'PROCESSING')) {
+                continue;
+            }
+            
+            switch (expectedOutcome) {
+            case 'succeed':
                 assert.notStrictEqual(replicationStatus, 'FAILED', `replication failed for object ${objectName}`);
                 if (replicationStatus === 'COMPLETED') {
                     return;
                 }
-            } else if (replicate === 'fail to be') {
+                break;
+            case 'fail':
                 assert.notStrictEqual(
                     replicationStatus,
                     'COMPLETED',
@@ -125,12 +135,20 @@ Then('the object should eventually {string} replicated', { timeout: 360_000 },
                 if (replicationStatus === 'FAILED') {
                     return;
                 }
+                break;
+            case 'never happen':
+                // When the replication is not expected to happen, replicationStatus is never set
+                assert.strictEqual(
+                    replicationStatus, undefined,
+                    'expected object to have no ReplicationStatus (should remain undefined)'
+                );
+                break;
             }
-            if (replicationStatus === 'PENDING' || replicationStatus === 'PROCESSING') {
-                continue;
-            }
         }
-        assert.fail(`Timeout: Object '${objectName}' is still pending/processing after timeout`);
+        if (expectedOutcome === 'never happen') {
+            return;
+        }
+        assert.fail(`Timeout: Object '${objectName}' is still in pending/processing state after timeout`);
     });
 
 Then(