fix: Ensure feature gates are enabled for unit tests

Extra changes:
* Ensure lateInitialize is not called for readOnly policy
* Add mock call for PopulateResourceFromAnnotation in adopt tests
* Ensure FilterSystemTags is called for `adopt` policy

Author: michaelhtm

pkg/runtime/reconciler.go

@@ -461,6 +461,7 @@ func (r *resourceReconciler) Sync(
 			return resolved, err
 		}
 	}
+	fmt.Println(adoptionPolicy, "\n\n\n\n", needAdoption)
 
 	rlog.Enter("rm.ReadOne")
 	latest, err = rm.ReadOne(ctx, resolved)
@@ -488,7 +489,9 @@ func (r *resourceReconciler) Sync(
 		if err != nil {
 			return latest, err
 		}
-	} else if !isReadOnly {
+	} else if isReadOnly {
+		return latest, nil
+	} else {
 		if adoptionPolicy == AdoptionPolicy_AdoptOrCreate {
 			// set adopt-or-create resource as managed before attempting
 			// update

pkg/runtime/reconciler_test.go

@@ -35,6 +35,7 @@ import (
 	ackcondition "github.com/aws-controllers-k8s/runtime/pkg/condition"
 	ackcfg "github.com/aws-controllers-k8s/runtime/pkg/config"
 	ackerr "github.com/aws-controllers-k8s/runtime/pkg/errors"
+	"github.com/aws-controllers-k8s/runtime/pkg/featuregate"
 	ackmetrics "github.com/aws-controllers-k8s/runtime/pkg/metrics"
 	"github.com/aws-controllers-k8s/runtime/pkg/requeue"
 	ackrt "github.com/aws-controllers-k8s/runtime/pkg/runtime"
@@ -92,7 +93,12 @@ func reconcilerMocks(
 		Level:       zapcore.InfoLevel,
 	}
 	fakeLogger := ctrlrtzap.New(ctrlrtzap.UseFlagOptions(&zapOptions))
-	cfg := ackcfg.Config{}
+	cfg := ackcfg.Config{
+		FeatureGates: featuregate.FeatureGates{
+			featuregate.ReadOnlyResources: {Enabled: true},
+			featuregate.ResourceAdoption:  {Enabled: true},
+		},
+	}
 	metrics := ackmetrics.NewMetrics("bookstore")
 
 	sc := &ackmocks.ServiceController{}
@@ -241,12 +247,7 @@ func TestReconcilerReadOnlyResource(t *testing.T) {
 		latest, nil,
 	)
 	rm.On("IsSynced", ctx, latest).Return(true, nil)
-	rmf, rd := managedResourceManagerFactoryMocks(desired, latest)
-
-	rm.On("LateInitialize", ctx, latest).Return(latest, nil)
-	rd.On("IsManaged", desired).Return(true)
-	rd.On("Delta", desired, latest).Return(ackcompare.NewDelta())
-	rd.On("Delta", latest, latest).Return(ackcompare.NewDelta())
+	rmf, _ := managedResourceManagerFactoryMocks(desired, latest)
 
 	r, kc, scmd := reconcilerMocks(rmf)
 	rm.On("EnsureTags", ctx, desired, scmd).Return(nil)
@@ -266,28 +267,32 @@ func TestReconcilerAdoptResource(t *testing.T) {
 	require := require.New(t)
 
 	ctx := context.TODO()
+	adoptionFieldsString := "{\"arn\": \"my-adopt-book-arn\"}"
+	adoptionFields := map[string]string{
+		"arn": "my-adopt-book-arn",
+	}
 
 	desired, _, metaObj := resourceMocks()
 	desired.On("ReplaceConditions", []*ackv1alpha1.Condition{}).Return()
 	metaObj.SetAnnotations(map[string]string{
 		ackv1alpha1.AnnotationAdoptionPolicy: "adopt",
-		ackv1alpha1.AnnotationAdoptionFields: "{\"arn\": \"my-adopt-book-arn\"}",
+		ackv1alpha1.AnnotationAdoptionFields: adoptionFieldsString,
 	})
 
 	latest, latestRTObj, _ := resourceMocks()
 	latest.On("Conditions").Return([]*ackv1alpha1.Condition{})
 	latest.On("MetaObject").Return(metav1.ObjectMeta{
 		Annotations: map[string]string{
 			ackv1alpha1.AnnotationAdoptionPolicy: "adopt",
-			ackv1alpha1.AnnotationAdoptionFields: "{\"arn\": \"my-adopt-book-arn\"}",
+			ackv1alpha1.AnnotationAdoptionFields: adoptionFieldsString,
 		},
 	})
 	latest.On("Conditions").Return([]*ackv1alpha1.Condition{})
 	latest.On(
 		"ReplaceConditions",
 		mock.AnythingOfType("[]*v1alpha1.Condition"),
 	).Return()
-
+	desired.On("PopulateResourceFromAnnotation", adoptionFields).Return(nil)
 	rm := &ackmocks.AWSResourceManager{}
 	rm.On("ResolveReferences", ctx, nil, desired).Return(
 		desired, false, nil,
@@ -302,10 +307,11 @@ func TestReconcilerAdoptResource(t *testing.T) {
 
 	rm.On("LateInitialize", ctx, latest).Return(latest, nil)
 	rd.On("IsManaged", desired).Return(false)
-	rd.On("Delta", desired, latest).Return(ackcompare.NewDelta())
 	rd.On("Delta", latest, latest).Return(ackcompare.NewDelta())
 
 	r, kc, scmd := reconcilerMocks(rmf)
+	rm.On("FilterSystemTags", latest).Return()
+	rd.On("MarkAdopted", latest).Return()
 	rm.On("EnsureTags", ctx, desired, scmd).Return(nil)
 	statusWriter := &ctrlrtclientmock.SubResourceWriter{}
 	kc.On("Patch", ctx, latestRTObj, mock.AnythingOfType("*client.mergeFromPatch")).Return(nil)
@@ -324,13 +330,18 @@ func TestReconcilerAdoptOrCreateResource_Create(t *testing.T) {
 	require := require.New(t)
 
 	ctx := context.TODO()
+	adoptionFieldsString := "{\"arn\": \"my-adopt-book-arn\"}"
+	adoptionFields := map[string]string{
+		"arn": "my-adopt-book-arn",
+	}
 
 	desired, _, metaObj := resourceMocks()
 	desired.On("ReplaceConditions", []*ackv1alpha1.Condition{}).Return()
 	metaObj.SetAnnotations(map[string]string{
 		ackv1alpha1.AnnotationAdoptionPolicy: "adopt-or-create",
-		ackv1alpha1.AnnotationAdoptionFields: "{\"arn\": \"my-adopt-book-arn\"}",
+		ackv1alpha1.AnnotationAdoptionFields: adoptionFieldsString,
 	})
+	desired.On("PopulateResourceFromAnnotation", adoptionFields).Return(nil)
 
 	ids := &ackmocks.AWSResourceIdentifiers{}
 
@@ -339,8 +350,8 @@ func TestReconcilerAdoptOrCreateResource_Create(t *testing.T) {
 	latest.On("Conditions").Return([]*ackv1alpha1.Condition{})
 	latest.On("MetaObject").Return(metav1.ObjectMeta{
 		Annotations: map[string]string{
-			ackv1alpha1.AnnotationAdoptionPolicy: "adopt",
-			ackv1alpha1.AnnotationAdoptionFields: "{\"arn\": \"my-adopt-book-arn\"}",
+			ackv1alpha1.AnnotationAdoptionPolicy: "adopt-or-create",
+			ackv1alpha1.AnnotationAdoptionFields: adoptionFieldsString,
 		},
 	})
 	latest.On("Conditions").Return([]*ackv1alpha1.Condition{})
@@ -366,6 +377,7 @@ func TestReconcilerAdoptOrCreateResource_Create(t *testing.T) {
 	).Once()
 	rm.On("IsSynced", ctx, latest).Return(true, nil)
 	rmf, rd := managedResourceManagerFactoryMocks(desired, latest)
+	rd.On("MarkAdopted", latest).Return()
 
 	rm.On("LateInitialize", ctx, latest).Return(latest, nil)
 	rd.On("IsManaged", desired).Return(false).Once()
@@ -409,7 +421,7 @@ func TestReconcilerAdoptOrCreateResource_Adopt(t *testing.T) {
 	latest.On("Conditions").Return([]*ackv1alpha1.Condition{})
 	latest.On("MetaObject").Return(metav1.ObjectMeta{
 		Annotations: map[string]string{
-			ackv1alpha1.AnnotationAdoptionPolicy: "adopt",
+			ackv1alpha1.AnnotationAdoptionPolicy: "adopt-or-create",
 			ackv1alpha1.AnnotationAdoptionFields: "{\"arn\": \"my-adopt-book-arn\"}",
 		},
 	})