fix: node unit tests

loft-sh · Jul 18, 2024 · b13c8e1 · b13c8e1
1 parent e109d51
commit b13c8e1
Show file tree

Hide file tree

Showing 3 changed files with 153 additions and 164 deletions.
diff --git a/pkg/controllers/resources/nodes/syncer.go b/pkg/controllers/resources/nodes/syncer.go
@@ -269,28 +269,22 @@ func (s *nodeSyncer) Sync(ctx *synccontext.SyncContext, pObj client.Object, vObj
 		return ctrl.Result{}, ctx.VirtualClient.Delete(ctx, vObj)
 	}
 
-	withoutLabels := pNode.DeepCopy()
-
-	patch, err := patcher.NewSyncerPatcher(ctx, withoutLabels, vNode)
+	patch, err := patcher.NewSyncerPatcher(ctx, pNode, vNode)
 	if err != nil {
 		return ctrl.Result{}, fmt.Errorf("new syncer patcher: %w", err)
 	}
 	defer func() {
-		fmt.Println("vnode : ", vNode.Labels)
-		fmt.Println("pnode : ", pNode.Labels)
 		if err := patch.Patch(ctx, pNode, vNode); err != nil {
 			retErr = utilerrors.NewAggregate([]error{retErr, err})
 		}
 	}()
 
-	updatedVNode, _, err := s.translateUpdateStatus(ctx, pNode, vNode)
+	err = s.translateUpdateStatus(ctx, pNode, vNode)
 	if err != nil {
 		return ctrl.Result{}, fmt.Errorf("update node status: %w", err)
 	}
-	vNode.Status = updatedVNode.Status
 
 	s.translateUpdateBackwards(pNode, vNode)
-
 	return ctrl.Result{}, nil
 }