fix: properly init nodes from dependent resource spec

Author: metacosm

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/workflow/DefaultManagedWorkflow.java

@@ -74,8 +74,11 @@ public Workflow<P> resolve(KubernetesClient client,
       ControllerConfiguration<P> configuration) {
     final var alreadyResolved = new HashMap<String, DependentResourceNode>(specs.size());
     for (DependentResourceSpec spec : specs) {
-      final var node =
-          new DependentResourceNode(spec.getName(), resolve(spec, client, configuration));
+      final var node = new DependentResourceNode(spec.getName(),
+          spec.getReconcileCondition(),
+          spec.getDeletePostCondition(),
+          spec.getReadyCondition(),
+          resolve(spec, client, configuration));
       alreadyResolved.put(node.getName(), node);
       spec.getDependsOn()
           .forEach(depend -> node.addDependsOnRelation(alreadyResolved.get((String) depend)));

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/workflow/DependentResourceNode.java

@@ -16,7 +16,21 @@ public class DependentResourceNode<R, P extends HasMetadata> {
   private Condition<R, P> reconcilePrecondition;
   private Condition<R, P> deletePostcondition;
   private Condition<R, P> readyPostcondition;
-  private DependentResource<R, P> dependentResource;
+  private final DependentResource<R, P> dependentResource;
+
+  DependentResourceNode(DependentResource<R, P> dependentResource) {
+    this(getNameFor(dependentResource), null, null, null, dependentResource);
+  }
+
+  public DependentResourceNode(String name, Condition<R, P> reconcilePrecondition,
+      Condition<R, P> deletePostcondition, Condition<R, P> readyPostcondition,
+      DependentResource<R, P> dependentResource) {
+    this.name = name;
+    this.reconcilePrecondition = reconcilePrecondition;
+    this.deletePostcondition = deletePostcondition;
+    this.readyPostcondition = readyPostcondition;
+    this.dependentResource = dependentResource;
+  }
 
   public List<? extends DependentResourceNode> getDependsOn() {
     return dependsOn;
@@ -69,10 +83,6 @@ public DependentResource<R, P> getDependentResource() {
     return dependentResource;
   }
 
-  public void setDependentResource(DependentResource<R, P> dependentResource) {
-    this.dependentResource = dependentResource;
-  }
-
   @Override
   public boolean equals(Object o) {
     if (this == o) {
@@ -90,16 +100,6 @@ public int hashCode() {
     return name.hashCode();
   }
 
-
-  DependentResourceNode(DependentResource<R, P> dependentResource) {
-    this(getNameFor(dependentResource), dependentResource);
-  }
-
-  DependentResourceNode(String name, DependentResource<R, P> dependentResource) {
-    this.name = name;
-    setDependentResource(dependentResource);
-  }
-
   @SuppressWarnings("rawtypes")
   static String getNameFor(DependentResource dependentResource) {
     return DependentResource.defaultNameFor(dependentResource.getClass()) + "#"