pkg/ansible: Adding unit tests for the ansible operator reconcile loop

* Adding fakes for runner, and runner result.
* Updating status object to be ouput as a map for deep copy functions.

Author: Shawn Hurley

Gopkg.lock

@@ -99,8 +99,8 @@ func (r *AnsibleOperatorReconciler) Reconcile(request reconcile.Request) (reconc
 		}
 	}
 	if !contains(pendingFinalizers, finalizer) && deleted {
-		logger.Info("Resource is terminated, skipping reconcilation")
-		return reconcileResult, nil
+		logrus.Info("Resource is terminated, skipping reconcilation")
+		return reconcile.Result{}, nil
 	}
 
 	spec := u.Object["spec"]
@@ -133,7 +133,7 @@ func (r *AnsibleOperatorReconciler) Reconcile(request reconcile.Request) (reconc
 			ansiblestatus.RunningMessage,
 		)
 		ansiblestatus.SetCondition(&crStatus, *c)
-		u.Object["status"] = crStatus
+		u.Object["status"] = crStatus.GetJSONMap()
 		err = r.Client.Update(context.TODO(), u)
 		if err != nil {
 			return reconcileResult, err
@@ -160,7 +160,7 @@ func (r *AnsibleOperatorReconciler) Reconcile(request reconcile.Request) (reconc
 	// iterate events from ansible, looking for the final one
 	statusEvent := eventapi.StatusJobEvent{}
 	failureMessages := eventapi.FailureMessages{}
-	for event := range result.Events {
+	for event := range result.Events() {
 		for _, eHandler := range r.EventHandlers {
 			go eHandler.Handle(ident, u, event)
 		}
@@ -234,7 +234,7 @@ func (r *AnsibleOperatorReconciler) Reconcile(request reconcile.Request) (reconc
 		ansiblestatus.SetCondition(&crStatus, *c)
 	}
 	// This needs the status subresource to be enabled by default.
-	u.Object["status"] = crStatus
+	u.Object["status"] = crStatus.GetJSONMap()
 	err = r.Client.Update(context.TODO(), u)
 	return reconcileResult, err