Add CR integration test

Azure · Dec 4, 2023 · 9398703 · 9398703
1 parent b9fb3a4
commit 9398703
Showing 1 changed file with 32 additions and 1 deletion.
diff --git a/internal/controllers/reconciliation/controller_test.go b/internal/controllers/reconciliation/controller_test.go
@@ -22,7 +22,7 @@ import (
 	"github.com/Azure/eno/internal/testutil"
 )
 
-// TODO: Add CR test
+// TODO: Cover no-op update, assert on exact k8s api requests
 
 type crudTestCase struct {
 	Name                         string
@@ -33,6 +33,7 @@ type crudTestCase struct {
 
 var crudTests = []crudTestCase{
 	{
+		// TODO: This test has a rare race condition I think
 		Name:  "strategic-merge", // will fail if non-strategic merge is used
 		Empty: &corev1.Service{},
 		Initial: &corev1.Service{
@@ -97,6 +98,36 @@ var crudTests = []crudTestCase{
 			}, svc.Ports)
 		},
 	},
+	{
+		Name:  "cr-basics",
+		Empty: &testv1.TestResource{},
+		Initial: &testv1.TestResource{
+			ObjectMeta: metav1.ObjectMeta{
+				Name:      "test-cr",
+				Namespace: "default",
+			},
+			Spec: testv1.TestResourceSpec{
+				Values: []*testv1.TestValue{{Int: 1}, {Int: 2}},
+			},
+		},
+		AssertCreated: func(t *testing.T, obj client.Object) {
+			tr := obj.(*testv1.TestResource)
+			assert.Equal(t, []*testv1.TestValue{{Int: 1}, {Int: 2}}, tr.Spec.Values)
+		},
+		Updated: &testv1.TestResource{
+			ObjectMeta: metav1.ObjectMeta{
+				Name:      "test-cr",
+				Namespace: "default",
+			},
+			Spec: testv1.TestResourceSpec{
+				Values: []*testv1.TestValue{{Int: 2}},
+			},
+		},
+		AssertUpdated: func(t *testing.T, obj client.Object) {
+			tr := obj.(*testv1.TestResource)
+			assert.Equal(t, []*testv1.TestValue{{Int: 2}}, tr.Spec.Values)
+		},
+	},
 }
 
 func TestCRUD(t *testing.T) {