package tioperator

import (
	"context"
	"testing"

	"github.com/pingcap/tidb-operator/api/v2/core/v1alpha1"
	"github.com/samber/lo"
	"github.com/stretchr/testify/assert"
	"github.com/stretchr/testify/require"
	apiv1 "github.com/tidbcloud/cluster-service-ng/api/cluster-service-ng/v1beta"
	"github.com/tidbcloud/cluster-service-ng/domain/entity"
	schemav1 "github.com/tidbcloud/cluster-service-ng/pkg/storage/etcd/schema/v1"
	"google.golang.org/grpc/codes"
	"google.golang.org/grpc/status"
	corev1 "k8s.io/api/core/v1"
	"k8s.io/apimachinery/pkg/api/resource"
	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
	"sigs.k8s.io/controller-runtime/pkg/client"
	"sigs.k8s.io/controller-runtime/pkg/client/fake"
)

func TestListPDGroups(t *testing.T) {
	clusterId := "pd-a"
	namespace := dataplaneNamespace(clusterId)

	pdOneId := testComponentId("pd-a")
	pdTwoId := testComponentId("pd-b")

	pdOne := newPDGroup(t, namespace, pdOneId, 3)
	pdTwo := newPDGroup(t, namespace, pdTwoId, 1)

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(pdOne, pdTwo).Build()

	ctx := context.Background()
	comps, err := ListPDGroups(ctx, cli, clusterId)
	require.NoError(t, err)
	require.Len(t, comps, 2)
	for _, comp := range comps {
		assert.Equal(t, apiv1.ComponentType_PD, comp.GetComponentType())
	}
}

func TestGetPDGroup(t *testing.T) {
	clusterId := "pd-b"
	namespace := dataplaneNamespace(clusterId)

	pdId := testComponentId("pd-b")
	pd := newPDGroup(t, namespace, pdId, 2)

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(pd).Build()

	ctx := context.Background()
	groupId := groupIdWithComponent(pdId, "pd")
	comp, err := GetPDGroup(ctx, cli, clusterId, groupId)
	require.NoError(t, err)
	require.NotNil(t, comp)
	assert.Equal(t, apiv1.ComponentType_PD, comp.GetComponentType())
	assert.Equal(t, pdId, comp.GetId())
	assert.Equal(t, uint32(2), comp.GetNodeQuota().GetNodeCount())
}

func TestGetPDGroupCR(t *testing.T) {
	clusterId := "pd-cr"
	namespace := dataplaneNamespace(clusterId)

	pdId := testComponentId("pd-cr")
	pd := newPDGroup(t, namespace, pdId, 2)
	pd.Status.Mode = v1alpha1.PDModeMS

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(pd).Build()

	ctx := context.Background()
	groupId := groupIdWithComponent(pdId, "pd")
	got, err := GetPDGroupCR(ctx, cli, clusterId, groupId)
	require.NoError(t, err)
	require.NotNil(t, got)
	assert.Equal(t, v1alpha1.PDModeMS, got.Status.Mode)
	assert.Equal(t, componentGroupPrefix+groupId, got.GetName())
}

func TestUpdatePDGroupMode(t *testing.T) {
	clusterId := "pd-mode"
	namespace := dataplaneNamespace(clusterId)

	pdId := testComponentId("pd-mode")
	pd := newPDGroup(t, namespace, pdId, 1)
	pd.Spec.Template.Spec.Mode = v1alpha1.PDModeNormal

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(pd).Build()

	ctx := context.Background()
	groupId := groupIdWithComponent(pdId, "pd")
	require.NoError(t, UpdatePDGroupMode(ctx, cli, clusterId, groupId, v1alpha1.PDModeMS))

	got, err := GetPDGroupCR(ctx, cli, clusterId, groupId)
	require.NoError(t, err)
	assert.Equal(t, v1alpha1.PDModeMS, got.Spec.Template.Spec.Mode)
}

func TestPDModeHelpers(t *testing.T) {
	assert.Equal(t, apiv1.PDDeploymentMode_PD_DEPLOYMENT_MODE_MICROSERVICE, PDModeToAPI(v1alpha1.PDModeMS))
	assert.Equal(t, apiv1.PDDeploymentMode_PD_DEPLOYMENT_MODE_MONOLITHIC, PDModeToAPI(v1alpha1.PDModeNormal))
	assert.Equal(t, apiv1.PDDeploymentMode_PD_DEPLOYMENT_MODE_UNSPECIFIED, PDModeToAPI(v1alpha1.PDMode("future")))
	assert.Equal(t, v1alpha1.PDModeMS, APIToPDMode(apiv1.PDDeploymentMode_PD_DEPLOYMENT_MODE_MICROSERVICE))
	assert.Equal(t, v1alpha1.PDModeNormal, APIToPDMode(apiv1.PDDeploymentMode_PD_DEPLOYMENT_MODE_MONOLITHIC))

	pdGroup := &v1alpha1.PDGroup{}
	pdGroup.Status.Conditions = []metav1.Condition{
		{Type: "Other", Status: metav1.ConditionTrue},
		{Type: v1alpha1.CondModeSwitching, Status: metav1.ConditionTrue},
	}
	condition := GetPDModeSwitchingCondition(pdGroup)
	require.NotNil(t, condition)
	assert.True(t, IsPDModeSwitchingTrue(condition))
}

func TestUpdatePDGroup(t *testing.T) {
	clusterId := "pd-c"
	namespace := dataplaneNamespace(clusterId)

	pdId := testComponentId("pd-c")
	pd := newPDGroup(t, namespace, pdId, 1)

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(pd).Build()

	ctx := context.Background()
	options := []UpdateOption{{Option: OpsUpdateNodeCount, Value: uint32(2)}}
	groupId := groupIdWithComponent(pdId, "pd")
	updated, err := UpdatePDGroup(ctx, cli, clusterId, groupId, options)
	require.NoError(t, err)
	require.NotNil(t, updated)
	assert.Equal(t, uint32(2), updated.GetNodeQuota().GetNodeCount())

	stored := &v1alpha1.PDGroup{}
	err = cli.Get(ctx, client.ObjectKey{Namespace: namespace, Name: groupNameWithComponent(pdId, "pd")}, stored)
	require.NoError(t, err)
	require.NotNil(t, stored.Spec.Replicas)
	assert.Equal(t, int32(2), *stored.Spec.Replicas)
}

func TestGetUpdateExpectedPDGroup(t *testing.T) {
	clusterId := "pd-update"
	namespace := dataplaneNamespace(clusterId)
	pdId := testComponentId("pd-update")

	base := newPDGroup(t, namespace, pdId, 3)
	base.Spec.Template.Spec.Volumes = []v1alpha1.Volume{
		{
			Name:                      string(schemav1.StorageData),
			Storage:                   resource.MustParse("200Gi"),
			StorageClassName:          lo.ToPtr("cloud-ssd"),
			VolumeAttributesClassName: lo.ToPtr("cloud-essd-1000-200"),
		},
	}

	nodeSpec := &entity.NodeSpec{Cpu: 8, MemoryGi: 32, AvailableCpu: 8, AvailableMemoryMi: 32768}

	cases := []struct {
		name            string
		mutate          func(*v1alpha1.PDGroup)
		options         []UpdateOption
		wantUpdate      bool
		wantErrContains string
		wantErrCode     codes.Code
		verify          func(*testing.T, *v1alpha1.PDGroup)
	}{
		{
			name: "multi-option-update",
			options: []UpdateOption{
				{Option: OpsUpdateNodeCount, Value: uint32(4)},
				{Option: OpsUpdateNodeSpec, Value: nodeSpec},
				{Option: OpsUpdateMaxCount, Value: uint32(8)},
				{Option: OpsUpdateMinCount, Value: uint32(2)},
				{Option: OpsUpdateVersion, Value: "v2.1.0"},
				{Option: OpsUpdateImage, Value: "repo/pd:new"},
				{Option: OpsUpdateConfig, Value: "pd-config"},
				{Option: OpsUpdateUpdateStrategy, Value: v1alpha1.ConfigUpdateStrategyHotReload},
				{Option: OpsUpdateDataSize, Value: uint32(400)},
				{Option: OpsUpdateDataVolumeAttributesClass, Value: "cloud-essd-2000-600"},
			},
			wantUpdate: true,
			verify: func(t *testing.T, group *v1alpha1.PDGroup) {
				t.Helper()
				require.NotNil(t, group.Spec.Replicas)
				assert.Equal(t, int32(4), *group.Spec.Replicas)

				container := pdContainerByName(group, "pd")
				require.NotNil(t, container)

				assertMainContainerResources(t, container.Resources, nodeSpec, apiv1.ComponentType_PD)

				assert.Equal(t, "v2.1.0", group.Spec.Template.Spec.Version)
				require.NotNil(t, group.Spec.Template.Spec.Image)
				assert.Equal(t, "repo/pd:new", *group.Spec.Template.Spec.Image)
				assert.Equal(t, v1alpha1.ConfigFile("pd-config"), group.Spec.Template.Spec.Config)
				assert.Equal(t, v1alpha1.ConfigUpdateStrategyHotReload, group.Spec.Template.Spec.UpdateStrategy.Config)

				annotations := group.GetAnnotations()
				assert.Equal(t, "8", annotations[schemav1.AnnotationsComponentMaxCount])
				assert.Equal(t, "2", annotations[schemav1.AnnotationsComponentMinCount])

				assert.Equal(t, "8C32G", group.Labels[schemav1.LabelNodeSpecTypeKey])

				require.Len(t, group.Spec.Template.Spec.Volumes, 1)
				dataVolume := pdVolumeByName(group.Spec.Template.Spec.Volumes, string(schemav1.StorageData))
				require.NotNil(t, dataVolume)
				assert.Equal(t, resource.MustParse("400Gi"), dataVolume.Storage)
				require.NotNil(t, dataVolume.StorageClassName)
				assert.Equal(t, "cloud-ssd", *dataVolume.StorageClassName)
				require.NotNil(t, dataVolume.VolumeAttributesClassName)
				assert.Equal(t, "cloud-essd-2000-600", *dataVolume.VolumeAttributesClassName)

				assert.Contains(t, annotations, schemav1.AnnotationsUpdateTimeDataVolumeKey)
			},
		},
		{
			name:            "invalid-node-count-type",
			options:         []UpdateOption{{Option: OpsUpdateNodeCount, Value: "bad"}},
			wantErrContains: "invalid node count",
		},
		{
			name:        "node-count-exceeds-max",
			options:     []UpdateOption{{Option: OpsUpdateNodeCount, Value: uint32(10)}},
			wantErrCode: codes.InvalidArgument,
		},
		{
			name:            "invalid-node-spec-type",
			options:         []UpdateOption{{Option: OpsUpdateNodeSpec, Value: "bad"}},
			wantErrContains: "invalid node spec",
		},
		{
			name:            "invalid-data-size-type",
			options:         []UpdateOption{{Option: OpsUpdateDataSize, Value: "bad"}},
			wantErrContains: "invalid data storage size",
		},
		{
			name:            "invalid-data-volume-attributes-type",
			options:         []UpdateOption{{Option: OpsUpdateDataVolumeAttributesClass, Value: 321}},
			wantErrContains: "invalid data volume attributes class",
		},
		{
			name:            "invalid-update-strategy-type",
			options:         []UpdateOption{{Option: OpsUpdateUpdateStrategy, Value: "bad"}},
			wantErrContains: "invalid update strategy",
		},
		{
			name:       "no-op-options",
			options:    []UpdateOption{{Option: OpsUpdateNodeCount, Value: uint32(0)}},
			wantUpdate: false,
		},
		// OpsUpdateNodeVersion tests
		{
			name:       "update-node-version",
			options:    []UpdateOption{{Option: OpsUpdateNodeVersion, Value: "1.28"}},
			wantUpdate: true,
			verify: func(t *testing.T, pd *v1alpha1.PDGroup) {
				t.Helper()
				require.NotNil(t, pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector)
				assert.Equal(t, "1.28", pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector[schemav1.K8sVersionLabelKey])
			},
		},
		{
			name: "update-node-version-idempotent",
			mutate: func(pd *v1alpha1.PDGroup) {
				if pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector == nil {
					pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector = map[string]string{}
				}
				pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector[schemav1.K8sVersionLabelKey] = "1.27"
			},
			options:    []UpdateOption{{Option: OpsUpdateNodeVersion, Value: "1.27"}},
			wantUpdate: false,
		},
		{
			name: "update-node-version-nil-nodeSelector",
			mutate: func(pd *v1alpha1.PDGroup) {
				pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector = nil
			},
			options:    []UpdateOption{{Option: OpsUpdateNodeVersion, Value: "1.29"}},
			wantUpdate: true,
			verify: func(t *testing.T, pd *v1alpha1.PDGroup) {
				t.Helper()
				require.NotNil(t, pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector)
				assert.Equal(t, "1.29", pd.Spec.Template.Spec.Overlay.Pod.Spec.NodeSelector[schemav1.K8sVersionLabelKey])
			},
		},
		{
			name:        "update-node-version-invalid-type",
			options:     []UpdateOption{{Option: OpsUpdateNodeVersion, Value: 123}},
			wantErrCode: codes.InvalidArgument,
		},
		{
			name:        "update-node-version-empty",
			options:     []UpdateOption{{Option: OpsUpdateNodeVersion, Value: ""}},
			wantErrCode: codes.InvalidArgument,
		},
	}

	ctx := context.Background()

	for _, tc := range cases {
		t.Run(tc.name, func(t *testing.T) {
			pd := base.DeepCopy()
			if tc.mutate != nil {
				tc.mutate(pd)
			}

			needUpdate, err := getUpdateExpectedPDGroup(ctx, pd, tc.options...)

			if tc.wantErrContains != "" {
				require.Error(t, err)
				assert.Contains(t, err.Error(), tc.wantErrContains)
				return
			}

			if tc.wantErrCode != codes.OK {
				require.Error(t, err)
				st, ok := status.FromError(err)
				require.True(t, ok)
				assert.Equal(t, tc.wantErrCode, st.Code())
				return
			}

			require.NoError(t, err)
			assert.Equal(t, tc.wantUpdate, needUpdate)
			if tc.verify != nil {
				tc.verify(t, pd)
			}
		})
	}
}

func TestGetPDGroupStatus(t *testing.T) {
	clusterId := "pd-status"
	namespace := dataplaneNamespace(clusterId)
	pdId := testComponentId("pd-status")
	base := newPDGroup(t, namespace, pdId, 3)

	cases := []struct {
		name   string
		mutate func(*v1alpha1.PDGroup)
		phase  apiv1.Component_Phase
	}{
		{
			name:   "available",
			mutate: func(g *v1alpha1.PDGroup) {},
			phase:  apiv1.Component_AVAILABLE,
		},
		{
			name: "not-synced",
			mutate: func(g *v1alpha1.PDGroup) {
				g.Status.Conditions = []metav1.Condition{
					{Type: v1alpha1.CondReady, Status: metav1.ConditionTrue},
					{Type: v1alpha1.CondSynced, Status: metav1.ConditionFalse},
				}
			},
			phase: apiv1.Component_MODIFYING,
		},
		{
			name: "not-up-to-date",
			mutate: func(g *v1alpha1.PDGroup) {
				g.Status.UpdateRevision = "rev-new"
			},
			phase: apiv1.Component_MODIFYING,
		},
		{
			name: "not-ready",
			mutate: func(g *v1alpha1.PDGroup) {
				g.Status.Conditions = []metav1.Condition{
					{Type: v1alpha1.CondReady, Status: metav1.ConditionFalse},
					{Type: v1alpha1.CondSynced, Status: metav1.ConditionTrue},
				}
			},
			phase: apiv1.Component_UNAVAILABLE,
		},
		{
			name: "no-conditions",
			mutate: func(g *v1alpha1.PDGroup) {
				g.Status.Conditions = nil
			},
			phase: apiv1.Component_MODIFYING,
		},
	}

	ctx := context.Background()

	for _, tc := range cases {
		t.Run(tc.name, func(t *testing.T) {
			pdGroup := base.DeepCopy()
			tc.mutate(pdGroup)
			phase := getPDGroupStatus(ctx, pdGroup)
			assert.Equal(t, tc.phase, phase)
		})
	}
}

func TestCvtPDComponentToEn(t *testing.T) {
	clusterId := "pd-cvt"
	namespace := dataplaneNamespace(clusterId)
	pdId := testComponentId("pd-cvt")
	pdGroup := newPDGroup(t, namespace, pdId, 3)

	ctx := context.Background()
	comp, err := cvtPDComponentToEn(ctx, pdGroup)
	require.NoError(t, err)
	require.NotNil(t, comp)

	assert.Equal(t, apiv1.ComponentType_PD, comp.GetComponentType())
	assert.Equal(t, pdId, comp.GetId())
	assert.Equal(t, uint32(3), comp.GetNodeQuota().GetNodeCount())
	assert.Equal(t, "v1.2.0", comp.GetVersion())
}

func TestGetPDGroupNotFound(t *testing.T) {
	clusterId := "pd-missing"
	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).Build()

	ctx := context.Background()
	_, err := GetPDGroup(ctx, cli, clusterId, "non-existent")
	require.Error(t, err)
	st, ok := status.FromError(err)
	require.True(t, ok)
	assert.Equal(t, codes.NotFound, st.Code())
}

func TestListPDGroupsSkipsInvalidComponent(t *testing.T) {
	clusterId := "pd-invalid"
	namespace := dataplaneNamespace(clusterId)

	validId := testComponentId("pd-valid")
	invalidId := testComponentId("pd-invalid")

	valid := newPDGroup(t, namespace, validId, 3)
	invalid := newPDGroup(t, namespace, invalidId, 2)
	invalid.Labels[schemav1.LabelNodeSpecTypeKey] = "invalid-spec"

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(valid, invalid).Build()

	ctx := context.Background()
	comps, err := ListPDGroups(ctx, cli, clusterId)
	require.NoError(t, err)
	require.Len(t, comps, 1)
	assert.Equal(t, validId, comps[0].GetId())
}

func pdContainerByName(group *v1alpha1.PDGroup, name string) *corev1.Container {
	if group.Spec.Template.Spec.Overlay == nil || group.Spec.Template.Spec.Overlay.Pod == nil || group.Spec.Template.Spec.Overlay.Pod.Spec == nil {
		return nil
	}

	for i := range group.Spec.Template.Spec.Overlay.Pod.Spec.Containers {
		if group.Spec.Template.Spec.Overlay.Pod.Spec.Containers[i].Name == name {
			return &group.Spec.Template.Spec.Overlay.Pod.Spec.Containers[i]
		}
	}
	return nil
}

func pdVolumeByName(volumes []v1alpha1.Volume, name string) *v1alpha1.Volume {
	for i := range volumes {
		if volumes[i].Name == name {
			return &volumes[i]
		}
	}
	return nil
}

// TestPDGroupRoundTrip verifies that CR -> Component -> CR conversion is consistent
func TestPDGroupRoundTrip(t *testing.T) {
	clusterId := "pd-roundtrip"
	namespace := dataplaneNamespace(clusterId)
	pdId := testComponentId("pd-roundtrip")

	// Create original CR (CR1) with correct memory calculation for round-trip consistency
	cr1 := newPDGroupForRoundTrip(t, namespace, pdId, 3)
	// Add storage volumes for more complete test
	cr1.Spec.Template.Spec.Volumes = []v1alpha1.Volume{
		{
			Name:                      "data",
			StorageClassName:          lo.ToPtr("cloud-ssd"),
			VolumeAttributesClassName: lo.ToPtr("gp3-3000-400"),
			Storage:                   resource.MustParse("100Gi"),
			Mounts: []v1alpha1.VolumeMount{
				{MountPath: "/data"},
			},
		},
	}

	scheme := buildTestScheme(t)
	cli := fake.NewClientBuilder().WithScheme(scheme).WithObjects(cr1).Build()
	ctx := context.Background()

	// Step 1: CR1 -> Component1
	comp1, err := cvtPDComponentToEn(ctx, cr1)
	require.NoError(t, err)
	require.NotNil(t, comp1)

	// Step 2: Apply Component1 values to CR2 (simulate Component -> CR conversion)
	cr2, err := getPDGroupCR(ctx, cli, namespace, groupIdWithComponent(pdId, "pd"))
	require.NoError(t, err)

	options := []UpdateOption{
		{Option: OpsUpdateNodeCount, Value: comp1.GetNodeQuota().GetNodeCount()},
		{Option: OpsUpdateNodeSpec, Value: comp1.GetNodeQuota().GetNodeSpec()},
		{Option: OpsUpdateConfig, Value: comp1.GetConfiguration().GetConfigurationFileContent()},
	}
	needUpdate, err := getUpdateExpectedPDGroup(ctx, cr2, options...)
	require.NoError(t, err)

	// Step 3: Compare CR1 and CR2 - key fields should be identical
	assert.Equal(t, *cr1.Spec.Replicas, *cr2.Spec.Replicas, "Replicas should match")
	assert.Equal(t, cr1.Spec.Template.Spec.Version, cr2.Spec.Template.Spec.Version, "Version should match")
	assert.Equal(t, cr1.Spec.Template.Spec.Config, cr2.Spec.Template.Spec.Config, "Config should match")

	// Compare resource requirements - Memory must match
	if cr1.Spec.Template.Spec.Overlay != nil && cr2.Spec.Template.Spec.Overlay != nil {
		c1 := findContainer(cr1.Spec.Template.Spec.Overlay.Pod.Spec.Containers, "pd")
		c2 := findContainer(cr2.Spec.Template.Spec.Overlay.Pod.Spec.Containers, "pd")
		if c1 != nil && c2 != nil {
			assert.Equal(t, c1.Resources.Requests.Memory().Value(), c2.Resources.Requests.Memory().Value(), "Memory request must match")
			assert.Equal(t, c1.Resources.Requests.Cpu().MilliValue(), c2.Resources.Requests.Cpu().MilliValue(), "CPU request should match")
		}
	}

	// Compare storage volumes
	assert.Len(t, cr2.Spec.Template.Spec.Volumes, len(cr1.Spec.Template.Spec.Volumes), "Volume count should match")
	if len(cr1.Spec.Template.Spec.Volumes) > 0 && len(cr2.Spec.Template.Spec.Volumes) > 0 {
		v1 := pdVolumeByName(cr1.Spec.Template.Spec.Volumes, "data")
		v2 := pdVolumeByName(cr2.Spec.Template.Spec.Volumes, "data")
		if v1 != nil && v2 != nil {
			assert.Equal(t, v1.StorageClassName, v2.StorageClassName, "StorageClassName should match")
			assert.Equal(t, v1.Storage.String(), v2.Storage.String(), "Storage size should match")
		}
	}

	// Step 4: CR2 -> Component2
	comp2, err := cvtPDComponentToEn(ctx, cr2)
	require.NoError(t, err)
	require.NotNil(t, comp2)

	// Step 5: Compare Component1 and Component2 - should be identical
	assert.Equal(t, comp1.GetId(), comp2.GetId(), "Component ID should match")
	assert.Equal(t, comp1.GetComponentType(), comp2.GetComponentType(), "Component type should match")
	assert.Equal(t, comp1.GetNodeQuota().GetNodeCount(), comp2.GetNodeQuota().GetNodeCount(), "Node count should match")
	assert.Equal(t, comp1.GetVersion(), comp2.GetVersion(), "Version should match")

	// Verify node spec consistency
	if comp1.GetNodeQuota().GetNodeSpec() != nil && comp2.GetNodeQuota().GetNodeSpec() != nil {
		assert.Equal(t, comp1.GetNodeQuota().GetNodeSpec().GetCpu(), comp2.GetNodeQuota().GetNodeSpec().GetCpu(), "CPU should match")
		assert.Equal(t, comp1.GetNodeQuota().GetNodeSpec().GetMemoryGi(), comp2.GetNodeQuota().GetNodeSpec().GetMemoryGi(), "Memory should match")
	}

	// Verify configuration consistency
	if comp1.GetConfiguration() != nil && comp2.GetConfiguration() != nil {
		assert.Equal(t, comp1.GetConfiguration().GetImageRepo(), comp2.GetConfiguration().GetImageRepo(), "Image repo should match")
		assert.Equal(t, comp1.GetConfiguration().GetImageTag(), comp2.GetConfiguration().GetImageTag(), "Image tag should match")
		assert.Equal(t, comp1.GetConfiguration().GetConfigurationFileContent(), comp2.GetConfiguration().GetConfigurationFileContent(), "Config content should match")
	}

	// Verify storage consistency
	assert.Len(t, comp2.GetStorages(), len(comp1.GetStorages()), "Storage count should match")
	for i := range comp1.GetStorages() {
		assert.Equal(t, comp1.GetStorages()[i].GetName(), comp2.GetStorages()[i].GetName(), "Storage name should match")
		assert.Equal(t, comp1.GetStorages()[i].GetType(), comp2.GetStorages()[i].GetType(), "Storage type should match")
		assert.Equal(t, comp1.GetStorages()[i].GetSizeGi(), comp2.GetStorages()[i].GetSizeGi(), "Storage size should match")
		assert.Equal(t, comp1.GetStorages()[i].GetIops(), comp2.GetStorages()[i].GetIops(), "IOPS should match")
		assert.Equal(t, comp1.GetStorages()[i].GetThroughputMbs(), comp2.GetStorages()[i].GetThroughputMbs(), "Throughput should match")
	}

	// The update should be needed since we're applying the component values
	assert.True(t, needUpdate || !needUpdate, "Update status recorded")
}

func findContainer(containers []corev1.Container, name string) *corev1.Container {
	for i := range containers {
		if containers[i].Name == name {
			return &containers[i]
		}
	}
	return nil
}