Validate arrays of objects

internal/fields/validate.go

@@ -27,13 +27,20 @@ import (
 	"github.com/elastic/elastic-package/internal/multierror"
 	"github.com/elastic/elastic-package/internal/packages"
 	"github.com/elastic/elastic-package/internal/packages/buildmanifest"
+	"github.com/elastic/package-spec/v2/code/go/pkg/specerrors"
 )
 
+// EPF - Elastic Package Fields [validation]
+const ArrayOfObjectsErrorCode = "EPF00001"
+
 var (
 	semver2_0_0 = semver.MustParse("2.0.0")
 	semver2_3_0 = semver.MustParse("2.3.0")
+	semver3_0_0 = semver.MustParse("3.0.0")
 
 	defaultExternal = "ecs"
+
+	arrayOfObjectsErr = specerrors.NewStructuredError(errors.New("array of objects not used as nested type can lead to unexpected results"), ArrayOfObjectsErrorCode)
 )
 
 // Validator is responsible for fields validation.
@@ -784,10 +791,16 @@ func (v *Validator) parseSingleElementValue(key string, definition FieldDefiniti
 		}
 	// Groups should only contain nested fields, not single values.
 	case "group":
-		switch val.(type) {
+		switch val := val.(type) {
 		case map[string]interface{}:
-			// TODO: This is probably an element from an array of objects,
+			// This is probably an element from an array of objects,
 			// even if not recommended, it should be validated.
+			if v.specVersion.LessThan(semver3_0_0) {
+				break
+			}
+			errs := v.validateMapElement(key, common.MapStr(val), doc)
+			errs = append(errs, arrayOfObjectsErr)
+			return errs
 		default:
 			return fmt.Errorf("field %q is a group of fields, it cannot store values", key)
 		}

internal/fields/validate_test.go

@@ -11,10 +11,12 @@ import (
 	"sort"
 	"testing"
 
+	"github.com/Masterminds/semver/v3"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 
 	"github.com/elastic/elastic-package/internal/common"
+	"github.com/elastic/elastic-package/internal/multierror"
 )
 
 type results struct {
@@ -309,10 +311,12 @@ func TestValidate_ExpectedDatasets(t *testing.T) {
 
 func Test_parseElementValue(t *testing.T) {
 	for _, test := range []struct {
-		key        string
-		value      interface{}
-		definition FieldDefinition
-		fail       bool
+		key         string
+		value       interface{}
+		definition  FieldDefinition
+		fail        bool
+		assertError func(t *testing.T, err error)
+		specVersion semver.Version
 	}{
 		// Arrays
 		{
@@ -619,17 +623,52 @@ func Test_parseElementValue(t *testing.T) {
 				},
 			},
 		},
+		// elements in arrays of objects should be validated
+		{
+			key: "details",
+			value: []interface{}{
+				map[string]interface{}{
+					"id":       "somehost-id",
+					"hostname": "somehost",
+				},
+			},
+			definition: FieldDefinition{
+				Name: "details",
+				Type: "group",
+				Fields: []FieldDefinition{
+					{
+						Name: "id",
+						Type: "keyword",
+					},
+				},
+			},
+			specVersion: *semver3_0_0,
+			fail:        true,
+			assertError: func(t *testing.T, err error) {
+				errs := err.(multierror.Error)
+				if assert.Len(t, errs, 2) {
+					assert.Contains(t, errs[0].Error(), `"details.hostname" is undefined`)
+					assert.ErrorIs(t, errs[1], arrayOfObjectsErr)
+				}
+			},
+		},
 	} {
-		v := Validator{
-			disabledDependencyManagement: true,
-			enabledAllowedIPCheck:        true,
-			allowedCIDRs:                 initializeAllowedCIDRsList(),
-		}
 
 		t.Run(test.key, func(t *testing.T) {
+			v := Validator{
+				Schema:                       []FieldDefinition{test.definition},
+				disabledDependencyManagement: true,
+				enabledAllowedIPCheck:        true,
+				allowedCIDRs:                 initializeAllowedCIDRsList(),
+				specVersion:                  test.specVersion,
+			}
+
 			err := v.parseElementValue(test.key, test.definition, test.value, common.MapStr{})
 			if test.fail {
 				require.Error(t, err)
+				if test.assertError != nil {
+					test.assertError(t, err)
+				}
 			} else {
 				require.NoError(t, err)
 			}