Merge pull request #456 from LF-Decentralized-Trust-labs/unit-tests

Adding unit tests to the operator

Author: dwertent

operator/cmd/main.go

@@ -136,10 +136,10 @@ func main() {
 		setupLog.Error(err, "unable to create controller", "controller", "Paladin")
 		os.Exit(1)
 	}
-	if err = (&controller.SmartContractDeploymentReconciler{
-		Client: mgr.GetClient(),
-		Scheme: mgr.GetScheme(),
-	}).SetupWithManager(mgr); err != nil {
+	if err = (controller.NewSmartContractDeploymentReconciler(
+		mgr.GetClient(),
+		mgr.GetScheme(),
+	)).SetupWithManager(mgr); err != nil {
 		setupLog.Error(err, "unable to create controller", "controller", "SmartContractDeployment")
 		os.Exit(1)
 	}

operator/internal/controller/besu_controller.go

@@ -171,6 +171,11 @@ func (r *BesuReconciler) SetupWithManager(mgr ctrl.Manager) error {
 
 	return ctrl.NewControllerManagedBy(mgr).
 		For(&corev1alpha1.Besu{}).
+		Owns(&appsv1.StatefulSet{}).
+		Owns(&corev1.Service{}).
+		Owns(&corev1.ConfigMap{}).
+		Owns(&corev1.Secret{}).
+		Owns(&policyv1.PodDisruptionBudget{}).
 		Complete(r)
 }
 
@@ -531,7 +536,8 @@ func (r *BesuReconciler) createPDB(ctx context.Context, node *corev1alpha1.Besu,
 	} else if err != nil {
 		return nil, err
 	}
-	return &foundPDB, nil
+
+	return pdb, nil
 }
 
 func (r *BesuReconciler) generateStatefulSetTemplate(node *corev1alpha1.Besu, name, configSum string) *appsv1.StatefulSet {