Allow overriding computed_optional_required

This can be useful, for example, when the person writing
the Terraform provider does not control the contents of
the OpenAPI spec.

For example to set an attribute to "required":

```
        attributes:
          overrides:
            name:
             description: The new description for name
             computed_optional_required: required
```

Author: stuart-mclaren-hpe

internal/config/parse.go

@@ -82,6 +82,8 @@ type AttributeOptions struct {
 type Override struct {
 	// Description overrides the description that was mapped/merged from the OpenAPI specification.
 	Description string `yaml:"description"`
+	// ComputedOptionalRequired overrides the inferred value from the OpenAPI specification.
+	ComputedOptionalRequired string `yaml:"computed_optional_required"`
 }
 
 // ParseConfig takes in a byte array (of YAML), unmarshals into a Config struct, and validates the result

internal/explorer/config_explorer.go

@@ -211,7 +211,10 @@ func extractSchemaOptions(cfgSchemaOpts config.SchemaOptions) SchemaOptions {
 func extractOverrides(cfgOverrides map[string]config.Override) map[string]Override {
 	overrides := make(map[string]Override, len(cfgOverrides))
 	for key, cfgOverride := range cfgOverrides {
-		overrides[key] = Override{Description: cfgOverride.Description}
+		overrides[key] = Override{
+			Description:              cfgOverride.Description,
+			ComputedOptionalRequired: cfgOverride.ComputedOptionalRequired,
+		}
 	}
 
 	return overrides

internal/explorer/config_explorer_test.go

@@ -270,7 +270,8 @@ func Test_ConfigExplorer_FindResources(t *testing.T) {
 								},
 								Overrides: map[string]config.Override{
 									"test": {
-										Description: "test description for override",
+										Description:              "test description for override",
+										ComputedOptionalRequired: "computed_optional",
 									},
 								},
 							},
@@ -310,7 +311,8 @@ func Test_ConfigExplorer_FindResources(t *testing.T) {
 							},
 							Overrides: map[string]explorer.Override{
 								"test": {
-									Description: "test description for override",
+									Description:              "test description for override",
+									ComputedOptionalRequired: "computed_optional",
 								},
 							},
 						},

internal/explorer/explorer.go

@@ -50,5 +50,6 @@ type AttributeOptions struct {
 }
 
 type Override struct {
-	Description string
+	Description              string
+	ComputedOptionalRequired string
 }

internal/mapper/attrmapper/resource_attributes_test.go

@@ -282,6 +282,102 @@ func TestResourceAttributes_ApplyOverrides(t *testing.T) {
 				},
 			},
 		},
+		"matching overrides computed": {
+			overrides: map[string]explorer.Override{
+				"string_attribute": {
+					ComputedOptionalRequired: "computed",
+				},
+			},
+			attributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Required,
+					},
+				},
+			},
+			expectedAttributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Computed,
+						Description:              pointer(""),
+					},
+				},
+			},
+		},
+		"matching overrides optional": {
+			overrides: map[string]explorer.Override{
+				"string_attribute": {
+					ComputedOptionalRequired: "optional",
+				},
+			},
+			attributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Required,
+					},
+				},
+			},
+			expectedAttributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Optional,
+						Description:              pointer(""),
+					},
+				},
+			},
+		},
+		"matching overrides required": {
+			overrides: map[string]explorer.Override{
+				"string_attribute": {
+					ComputedOptionalRequired: "required",
+				},
+			},
+			attributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Computed,
+					},
+				},
+			},
+			expectedAttributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Required,
+						Description:              pointer(""),
+					},
+				},
+			},
+		},
+		"matching overrides computed_optional": {
+			overrides: map[string]explorer.Override{
+				"string_attribute": {
+					ComputedOptionalRequired: "computed_optional",
+				},
+			},
+			attributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.Computed,
+					},
+				},
+			},
+			expectedAttributes: attrmapper.ResourceAttributes{
+				&attrmapper.ResourceStringAttribute{
+					Name: "string_attribute",
+					StringAttribute: resource.StringAttribute{
+						ComputedOptionalRequired: schema.ComputedOptional,
+						Description:              pointer(""),
+					},
+				},
+			},
+		},
 		"matching nested overrides": {
 			overrides: map[string]explorer.Override{
 				"single_nested": {

internal/mapper/attrmapper/string.go

@@ -4,11 +4,13 @@
 package attrmapper
 
 import (
+	"fmt"
 	"github.com/hashicorp/terraform-plugin-codegen-openapi/internal/explorer"
 	"github.com/hashicorp/terraform-plugin-codegen-openapi/internal/mapper/util"
 	"github.com/hashicorp/terraform-plugin-codegen-spec/datasource"
 	"github.com/hashicorp/terraform-plugin-codegen-spec/provider"
 	"github.com/hashicorp/terraform-plugin-codegen-spec/resource"
+	"github.com/hashicorp/terraform-plugin-codegen-spec/schema"
 )
 
 type ResourceStringAttribute struct {
@@ -34,6 +36,23 @@ func (a *ResourceStringAttribute) Merge(mergeAttribute ResourceAttribute) (Resou
 func (a *ResourceStringAttribute) ApplyOverride(override explorer.Override) (ResourceAttribute, error) {
 	a.Description = &override.Description
 
+	switch override.ComputedOptionalRequired {
+	case "": // No override
+	case "computed":
+		a.ComputedOptionalRequired = schema.Computed
+	case "optional":
+		a.ComputedOptionalRequired = schema.Optional
+	case "required":
+		a.ComputedOptionalRequired = schema.Required
+	case "computed_optional":
+		a.ComputedOptionalRequired = schema.ComputedOptional
+	default:
+		return nil, fmt.Errorf(
+			"invalid value for computed_optional_required: %s",
+			override.ComputedOptionalRequired,
+		)
+	}
+
 	return a, nil
 }