removed comments

Author: azeezport

port/blueprint-permissions/refreshBluePrintPermissionsToState_test.go

@@ -7,19 +7,13 @@ import (
 	"github.com/port-labs/terraform-provider-port-labs/v2/internal/cli"
 )
 
-// TestRefreshBlueprintPermissionsStateWithNilUpdateMetadataProperties tests the bug
-// that occurs during import when UpdateMetadataProperties is nil
 func TestRefreshBlueprintPermissionsStateWithNilUpdateMetadataProperties(t *testing.T) {
-	// Simulate the state during import - only ID and BlueprintIdentifier are set
-	// This mimics what happens in ImportState method
 	state := &BlueprintPermissionsModel{
 		ID:                  types.StringValue("testBlueprint"),
 		BlueprintIdentifier: types.StringValue("testBlueprint"),
-		Entities:            nil, // This is nil during import
+		Entities:            nil,
 	}
 
-	// Create API response with metadata properties permissions
-	// This simulates what the Port API returns
 	ownedByTeam := false
 	apiResponse := &cli.BlueprintPermissions{
 		Entities: cli.BlueprintPermissionsEntities{
@@ -58,13 +52,10 @@ func TestRefreshBlueprintPermissionsStateWithNilUpdateMetadataProperties(t *test
 		},
 	}
 
-	// This should panic with the bug, but pass after the fix
 	err := refreshBlueprintPermissionsState(state, apiResponse, "testBlueprint")
 	if err != nil {
 		t.Fatalf("refreshBlueprintPermissionsState failed: %v", err)
 	}
-
-	// Verify the state was populated correctly
 	if state.Entities == nil {
 		t.Fatal("Entities should not be nil after refresh")
 	}
@@ -81,7 +72,6 @@ func TestRefreshBlueprintPermissionsStateWithNilUpdateMetadataProperties(t *test
 		t.Fatal("UpdateMetadataProperties.Identifier should not be nil")
 	}
 
-	// Verify the roles were set correctly
 	if len(state.Entities.UpdateMetadataProperties.Title.Roles) != 1 {
 		t.Errorf("Expected 1 role for Title, got %d", len(state.Entities.UpdateMetadataProperties.Title.Roles))
 	}
@@ -91,10 +81,7 @@ func TestRefreshBlueprintPermissionsStateWithNilUpdateMetadataProperties(t *test
 	}
 }
 
-// TestRefreshBlueprintPermissionsStateWithExistingUpdateMetadataProperties tests
-// that the function works correctly when UpdateMetadataProperties already exists
 func TestRefreshBlueprintPermissionsStateWithExistingUpdateMetadataProperties(t *testing.T) {
-	// Simulate a state where UpdateMetadataProperties already exists (normal update scenario)
 	ownedByTeam := false
 	state := &BlueprintPermissionsModel{
 		ID:                  types.StringValue("testBlueprint"),
@@ -111,7 +98,6 @@ func TestRefreshBlueprintPermissionsStateWithExistingUpdateMetadataProperties(t
 		},
 	}
 
-	// Create API response
 	apiResponse := &cli.BlueprintPermissions{
 		Entities: cli.BlueprintPermissionsEntities{
 			Register: cli.BlueprintPermissionsBlock{
@@ -148,12 +134,10 @@ func TestRefreshBlueprintPermissionsStateWithExistingUpdateMetadataProperties(t
 		t.Fatalf("refreshBlueprintPermissionsState failed: %v", err)
 	}
 
-	// Verify the state was updated correctly
 	if state.Entities.UpdateMetadataProperties.Title == nil {
 		t.Fatal("UpdateMetadataProperties.Title should not be nil")
 	}
 
-	// The users should be sorted according to the old order (user1 first)
 	if len(state.Entities.UpdateMetadataProperties.Title.Users) != 2 {
 		t.Errorf("Expected 2 users for Title, got %d", len(state.Entities.UpdateMetadataProperties.Title.Users))
 	}