Update inventory docs

Author: lranjbar

docs/data-sources/inventory.md

@@ -45,14 +45,16 @@ output "inventory_details" {
 <!-- schema generated by tfplugindocs -->
 ## Schema
 
-### Required
+### Optional
 
 - `id` (Number) Inventory id
+- `name` (String) Name of the inventory
+- `organization_name` (String) The name for the organization to which the inventory belongs
 
 ### Read-Only
 
 - `description` (String) Description of the inventory
-- `name` (String) Name of the inventory
+- `named_url` (String) The Named Url of the inventory
 - `organization` (Number) Identifier for the organization to which the inventory belongs
 - `url` (String) Url of the inventory
 - `variables` (String) Variables of the inventory. Will be either JSON or YAML string depending on how the variables were entered into AAP.

docs/resources/inventory.md

@@ -111,4 +111,6 @@ output "inventory_xyz" {
 ### Read-Only
 
 - `id` (Number) Inventory id
+- `named_url` (String) Named URL of the inventory
+- `organization_name` (String) Name for the organization.
 - `url` (String) URL of the inventory

internal/provider/inventory_data_source.go

@@ -12,7 +12,6 @@ import (
 	"github.com/hashicorp/terraform-plugin-framework/datasource/schema"
 	"github.com/hashicorp/terraform-plugin-framework/diag"
 	tfpath "github.com/hashicorp/terraform-plugin-framework/path"
-	"github.com/hashicorp/terraform-plugin-framework/provider"
 	"github.com/hashicorp/terraform-plugin-framework/types"
 )
 
@@ -38,6 +37,7 @@ var (
 	_ datasource.DataSource                     = &InventoryDataSource{}
 	_ datasource.DataSourceWithConfigure        = &InventoryDataSource{}
 	_ datasource.DataSourceWithConfigValidators = &InventoryDataSource{}
+	_ datasource.DataSourceWithValidateConfig   = &InventoryDataSource{}
 )
 
 // NewInventoryDataSource is a helper function to simplify the provider implementation.
@@ -96,6 +96,7 @@ func (d *InventoryDataSource) Schema(_ context.Context, _ datasource.SchemaReque
 
 // Read refreshes the Terraform state with the latest data.
 func (d *InventoryDataSource) Read(ctx context.Context, req datasource.ReadRequest, resp *datasource.ReadResponse) {
+
 	var state InventoryDataSourceModel
 	var diags diag.Diagnostics
 
@@ -107,7 +108,6 @@ func (d *InventoryDataSource) Read(ctx context.Context, req datasource.ReadReque
 
 	URI := path.Join(d.client.getApiEndpoint(), "inventories")
 	resourceURL, err := ReturnAAPNamedURL(state.Id, state.Name, state.OrganizationName, URI)
-	// resourceURL, err := state.ResourceUrlFromParameters(d)
 	if err != nil {
 		resp.Diagnostics.AddError("Minimal Data Not Supplied", "Expected either [id] or [name + organization_name] pair")
 		return
@@ -164,7 +164,7 @@ func (d *InventoryDataSource) ConfigValidators(_ context.Context) []datasource.C
 	}
 }
 
-func (d *InventoryDataSource) ValidateConfig(ctx context.Context, req provider.ValidateConfigRequest, resp *provider.ValidateConfigResponse) {
+func (d *InventoryDataSource) ValidateConfig(ctx context.Context, req datasource.ValidateConfigRequest, resp *datasource.ValidateConfigResponse) {
 	var data InventoryDataSourceModel
 
 	resp.Diagnostics.Append(req.Config.Get(ctx, &data)...)
@@ -181,23 +181,23 @@ func (d *InventoryDataSource) ValidateConfig(ctx context.Context, req provider.V
 		return
 	}
 
-	if IsValueNotProvided(data.Id) && IsValueNotProvided(data.Name) {
+	if !IsValueProvided(data.Id) && !IsValueProvided(data.Name) {
 		resp.Diagnostics.AddAttributeWarning(
 			tfpath.Root("id"),
 			"Missing Atribute Configuration",
 			"Expected either [id] or [name + organization_name] pair",
 		)
 	}
 
-	if IsValueProvided(data.Name) && IsValueNotProvided(data.OrganizationName) {
+	if IsValueProvided(data.Name) && !IsValueProvided(data.OrganizationName) {
 		resp.Diagnostics.AddAttributeWarning(
 			tfpath.Root("organization_name"),
 			"Missing Attribute Configuration",
 			"Expected organization_name to be configured with name.",
 		)
 	}
 
-	if IsValueNotProvided(data.Name) && IsValueProvided(data.OrganizationName) {
+	if !IsValueProvided(data.Name) && IsValueProvided(data.OrganizationName) {
 		resp.Diagnostics.AddAttributeWarning(
 			tfpath.Root("name"),
 			"Missing Attribute Configuration",

internal/provider/inventory_resource.go

@@ -291,7 +291,7 @@ func (r *InventoryResourceModel) generateRequestBody() ([]byte, diag.Diagnostics
 		organizationId = r.Organization.ValueInt64()
 	}
 
-	if IsValueNotProvided(r.NamedUrl) {
+	if !IsValueProvided(r.NamedUrl) {
 		namedURL, err := ReturnAAPNamedURL(r.Id, r.Name, r.OrganizationName, "inventories")
 		// Squashing error here. If we can't generate the named url just leave it blank
 		if err == nil {

internal/provider/utils.go

@@ -19,16 +19,15 @@ import (
 func ReturnAAPNamedURL(id types.Int64, name types.String, orgName types.String, URI string) (string, error) {
 	if IsValueProvided(id) {
 		return path.Join(URI, id.String()), nil
-	} else if IsValueProvided(name) && IsValueProvided(orgName) {
+	}
+
+	if IsValueProvided(name) && IsValueProvided(orgName) {
 		namedUrl := fmt.Sprintf("%s++%s", name.ValueString(), orgName.ValueString())
 		return path.Join(URI, namedUrl), nil
-	} else {
-		return "", errors.New("invalid lookup parameters")
 	}
-}
 
-func IsValueNotProvided(value attr.Value) bool {
-	return value.IsNull() || value.IsUnknown()
+	return "", errors.New("invalid lookup parameters")
+
 }
 
 func IsValueProvided(value attr.Value) bool {