Merge pull request #218 from port-labs/entity_icon_fix

add icon handle for entity and test

Author: OmriGez

internal/cli/models.go

@@ -35,6 +35,7 @@ type (
 		Meta
 		Identifier string                    `json:"identifier,omitempty"`
 		Title      string                    `json:"title"`
+		Icon       string                    `json:"icon,omitempty"`
 		Blueprint  string                    `json:"blueprint"`
 		Team       []string                  `json:"team,omitempty"`
 		Properties map[string]any            `json:"properties"`

port/entity/entityResourceToPortBody.go

@@ -87,6 +87,10 @@ func entityResourceToBody(ctx context.Context, state *EntityModel, bp *cli.Bluep
 		e.Title = state.Title.ValueString()
 	}
 
+	if !state.Icon.IsNull() {
+		e.Icon = state.Icon.ValueString()
+	}
+
 	if !state.Identifier.IsUnknown() {
 		e.Identifier = state.Identifier.ValueString()
 	}

port/entity/refreshEntityToState.go

@@ -172,6 +172,13 @@ func refreshEntityState(ctx context.Context, state *EntityModel, e *cli.Entity,
 	state.Identifier = types.StringValue(e.Identifier)
 	state.Blueprint = types.StringValue(blueprint.Identifier)
 	state.Title = types.StringValue(e.Title)
+
+	if e.Icon != "" {
+		state.Icon = types.StringValue(e.Icon)
+	} else {
+		state.Icon = types.StringNull()
+	}
+
 	state.CreatedAt = types.StringValue(e.CreatedAt.String())
 	state.CreatedBy = types.StringValue(e.CreatedBy)
 	state.UpdatedAt = types.StringValue(e.UpdatedAt.String())

port/entity/resource_test.go

@@ -59,6 +59,7 @@ func TestAccPortEntity(t *testing.T) {
 	}
 	resource "port_entity" "microservice" {
 		title = "TF Provider Test Entity0"
+		icon = "Terraform"
 		blueprint = port_blueprint.microservice.identifier
 		properties = {
 			"string_props" = {
@@ -100,6 +101,7 @@ func TestAccPortEntity(t *testing.T) {
 				Config: acctest.ProviderConfig + testAccActionConfigCreate,
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("port_entity.microservice", "title", "TF Provider Test Entity0"),
+					resource.TestCheckResourceAttr("port_entity.microservice", "icon", "Terraform"),
 					resource.TestCheckResourceAttr("port_entity.microservice", "blueprint", identifier),
 					resource.TestCheckResourceAttr("port_entity.microservice", "properties.string_props.myStringIdentifier", "My String Value"),
 					resource.TestCheckResourceAttr("port_entity.microservice", "properties.number_props.myNumberIdentifier", "123.456"),