Field Config Fixes for Code Generator (#295)

- Support for nested list of secrets
  - Code was getting generated for single SecretRef

Generated code with the fix

```
             authenticationMode:
                description: Denotes the user's authentication properties, such as
                  whether it requires a password to authenticate.
                properties:
                  passwords:
                    items:
                      description: SecretKeyReference combines a k8s corev1.SecretReference
                        with a specific key within the referred-to Secret
                      properties:
                        key:
                          description: Key is the key within the secret
                          type: string
                        name:
                          description: Name is unique within a namespace to reference
                            a secret resource.
                          type: string
                        namespace:
                          description: Namespace defines the space within which the
                            secret name must be unique.
                          type: string
                      required:
                      - key
                      type: object
                    type: array
                  type_:
                    type: string
                type: object
```

- Fix the struct compare logic
  - Spec was appended to field path due to this compare
    config was ignored.
https://github.com/aws-controllers-k8s/code-generator/blob/main/pkg/generate/code/compare.go#L128


Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Author: nmvk

pkg/generate/code/compare.go

@@ -562,7 +562,9 @@ func CompareStruct(
 		secondResAdaptedVarName := secondResVarName + "." + memberNameClean
 
 		var compareConfig *ackgenconfig.CompareFieldConfig
-		fieldConfig := fieldConfigs[memberFieldPath]
+		// memberFieldPath contains the field path along with the prefix cfg.PrefixConfig.SpecField + "." hence we
+		// would need to substring to exclude cfg.PrefixConfig.SpecField + "." to get correct field config.
+		fieldConfig := fieldConfigs[memberFieldPath[len(cfg.PrefixConfig.SpecField) + 1 :len(memberFieldPath)]]
 		if fieldConfig != nil {
 			compareConfig = fieldConfig.Compare
 		}

pkg/model/field.go

@@ -260,8 +260,9 @@ func NewField(
 				cleanMemberNames := names.New(memberName)
 				memberPath := path + "." + cleanMemberNames.Camel
 				memberShape := containerShape.MemberRefs[memberName]
+				fConfigs := crd.cfg.ResourceFields(crd.Names.Original)
 				memberField := NewField(
-					crd, memberPath, cleanMemberNames, memberShape, cfg,
+					crd, memberPath, cleanMemberNames, memberShape, fConfigs[memberPath],
 				)
 				memberFields[cleanMemberNames.Camel] = memberField
 			}

pkg/model/model.go

@@ -541,7 +541,7 @@ func replaceSecretAttrGoType(
 		)
 		panic(msg)
 	}
-	attr.GoType = "*ackv1alpha1.SecretKeyReference"
+	attr.GoType = field.GoType
 }
 
 // processFields is responsible for walking all of the CRDs' Spec and

pkg/model/model_memorydb_test.go

@@ -0,0 +1,24 @@
+package model_test
+
+import (
+	"github.com/aws-controllers-k8s/code-generator/pkg/testutil"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+	"testing"
+)
+
+func TestUserPasswords_IsSecret(t *testing.T) {
+	require := require.New(t)
+
+	g := testutil.NewModelForService(t, "memorydb")
+	crds, err := g.GetCRDs()
+
+	require.Nil(err)
+
+	crd := getCRDByName("User", crds)
+	require.NotNil(crd)
+	assert := assert.New(t)
+	assert.Equal("[]*ackv1alpha1.SecretKeyReference", crd.SpecFields["AuthenticationMode"].MemberFields["Passwords"].GoType)
+	assert.Equal("SecretKeyReference", crd.SpecFields["AuthenticationMode"].MemberFields["Passwords"].GoTypeElem)
+	assert.Equal("[]*ackv1alpha1.SecretKeyReference", crd.SpecFields["AuthenticationMode"].MemberFields["Passwords"].GoTypeWithPkgName)
+}