allow ignore nested fieldpaths (#260)

When implementing the update code paths for OpenSearchService, I found
out I needed to ignore the AutoTuneOptions.RollbackOnDisable nested
field within the UpdateDomainConfigInput shape. When I went to put the
following in the `generator.yaml` file:

```yaml
ignore:
  field_paths:
    - UpdateDomainConfigInput.AutoTuneOptions.RollbackOnDisable
```

What ended up happening is that the *entire* AutoTuneOptions field was
ignored instead of just the RollbackOnDisable field.

This patch fixes this situation by modifying the
`pkg/model.Model:ApplyIgnoreShapeRules()` method to account for nested
field paths and not just a two-level field path that has the Input or
Output shape name as the first part of the field path and the top-level
member field name as the second part of the field path.

Signed-off-by: Jay Pipes <[email protected]>

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

Author: jaypipes

pkg/model/model.go

@@ -19,6 +19,7 @@ import (
 	"sort"
 	"strings"
 
+	ackfp "github.com/aws-controllers-k8s/code-generator/pkg/fieldpath"
 	ackgenconfig "github.com/aws-controllers-k8s/code-generator/pkg/generate/config"
 	"github.com/aws-controllers-k8s/code-generator/pkg/generate/templateset"
 	"github.com/aws-controllers-k8s/code-generator/pkg/names"
@@ -718,14 +719,6 @@ func (m *Model) ApplyShapeIgnoreRules() {
 		return
 	}
 	for sdkShapeID, shape := range m.SDKAPI.API.Shapes {
-		for _, fieldpath := range m.cfg.Ignore.FieldPaths {
-			sn := strings.Split(fieldpath, ".")[0]
-			fn := strings.Split(fieldpath, ".")[1]
-			if shape.ShapeName != sn {
-				continue
-			}
-			delete(shape.MemberRefs, fn)
-		}
 		for _, sn := range m.cfg.Ignore.ShapeNames {
 			if shape.ShapeName == sn {
 				delete(m.SDKAPI.API.Shapes, sdkShapeID)
@@ -739,6 +732,31 @@ func (m *Model) ApplyShapeIgnoreRules() {
 			}
 		}
 	}
+	for _, fieldpath := range m.cfg.Ignore.FieldPaths {
+		fp := ackfp.FromString(fieldpath)
+		sn := fp.At(0)
+		if shape, found := m.SDKAPI.API.Shapes[sn]; !found {
+			msg := fmt.Sprintf(
+				"referred to unknown shape %s in config's Ignore.FieldPaths", sn,
+			)
+			panic(msg)
+		} else {
+			// This is just some tomfoolery to make the Input and Output shapes
+			// into ShapeRefs because the SDKAPI.Shapes is a map of Shape
+			// pointers not a map of ShapeRefs...
+			wrapper := &awssdkmodel.ShapeRef{
+				ShapeName: sn,
+				Shape:     shape,
+			}
+			// The last element of the fieldpath is the field/shape we want to
+			// ignore...
+			ignoreShape := fp.Pop()
+			parentShapeRef := fp.ShapeRef(wrapper)
+			// OK, now we delete the ignored shape by removing the shape from
+			// the parent's member references...
+			delete(parentShapeRef.Shape.MemberRefs, ignoreShape)
+		}
+	}
 }
 
 // GetConfig returns the configuration option used to define the current