refactor: improve testability of context construction

Author: teoincontatto

stackgres-k8s/src/operator/src/main/java/io/stackgres/operator/conciliation/factory/backup/BackupRole.java

@@ -18,6 +18,7 @@
 import io.fabric8.kubernetes.api.model.rbac.SubjectBuilder;
 import io.stackgres.common.StackGresUtil;
 import io.stackgres.common.crd.CommonDefinition;
+import io.stackgres.common.crd.sgbackup.BackupStatus;
 import io.stackgres.common.crd.sgbackup.StackGresBackup;
 import io.stackgres.common.labels.LabelFactoryForBackup;
 import io.stackgres.operator.conciliation.OperatorVersionBinder;
@@ -50,7 +51,8 @@ public static String roleName(String backupName) {
 
   @Override
   public Stream<HasMetadata> generateResource(StackGresBackupContext context) {
-    if (isNotBackupCopy(context)) {
+    if (isNotBackupCopy(context)
+        || BackupStatus.isFinished(context.getSource())) {
       return Stream.of();
     }
     return Stream.of(

stackgres-k8s/src/operator/src/main/java/io/stackgres/operator/conciliation/factory/dbops/DbOpsRole.java

@@ -21,6 +21,7 @@
 import io.fabric8.kubernetes.api.model.rbac.RoleBuilder;
 import io.fabric8.kubernetes.api.model.rbac.RoleRefBuilder;
 import io.fabric8.kubernetes.api.model.rbac.SubjectBuilder;
+import io.stackgres.common.DbOpsUtil;
 import io.stackgres.common.crd.CommonDefinition;
 import io.stackgres.common.crd.sgcluster.StackGresCluster;
 import io.stackgres.common.crd.sgdbops.StackGresDbOps;
@@ -54,6 +55,10 @@ public static String roleName(String dbOpsName) {
 
   @Override
   public Stream<HasMetadata> generateResource(StackGresDbOpsContext context) {
+    if (DbOpsUtil.isAlreadyCompleted(context.getSource())) {
+      return Stream.of();
+    }
+
     return Stream.<HasMetadata>of(
         createServiceAccount(context),
         createRole(context),