stackit-dev-tools/terraform-provider-stackitprivatepreview
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: tests

Browse source
This commit is contained in:
Andre_Harms 2026-02-10 13:58:06 +01:00
parent b737875c68
commit 8a27483b81
4 changed files with 20 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

7
stackit/internal/services/postgresflexalpha/database/mapper_test.go
View file

@ -34,7 +34,12 @@ func TestMapFields(t *testing.T) {
Name: utils.Ptr("my-db"), Name: utils.Ptr("my-db"),
Owner: utils.Ptr("\"my-owner\""), Owner: utils.Ptr("\"my-owner\""),
}, },
model: &dataSourceModel{}, model: &dataSourceModel{
DatabaseModel: datasource.DatabaseModel{
ProjectId: types.StringValue("my-project"),
InstanceId: types.StringValue("my-instance"),
},
},
region: "eu01", region: "eu01",
}, },
expected: expected{ expected: expected{

5
stackit/internal/services/postgresflexalpha/user/mapper_test.go
View file

@ -164,6 +164,7 @@ func TestMapFieldsCreate(t *testing.T) {
}, },
testRegion, testRegion,
resourceModel{ resourceModel{
Id: types.Int64Value(1),
UserId: types.Int64Value(1), UserId: types.Int64Value(1),
InstanceId: types.StringValue("iid"), InstanceId: types.StringValue("iid"),
ProjectId: types.StringValue("pid"), ProjectId: types.StringValue("pid"),
@ -185,6 +186,7 @@ func TestMapFieldsCreate(t *testing.T) {
}, },
testRegion, testRegion,
resourceModel{ resourceModel{
Id: types.Int64Value(1),
UserId: types.Int64Value(1), UserId: types.Int64Value(1),
InstanceId: types.StringValue("iid"), InstanceId: types.StringValue("iid"),
ProjectId: types.StringValue("pid"), ProjectId: types.StringValue("pid"),
@ -193,7 +195,7 @@ func TestMapFieldsCreate(t *testing.T) {
Password: types.StringNull(), Password: types.StringNull(),
Region: types.StringValue(testRegion), Region: types.StringValue(testRegion),
Status: types.StringValue("status"), Status: types.StringValue("status"),
ConnectionString: types.StringValue("connection_string"), ConnectionString: types.StringNull(),
}, },
true, true,
}, },
@ -206,6 +208,7 @@ func TestMapFieldsCreate(t *testing.T) {
}, },
testRegion, testRegion,
resourceModel{ resourceModel{
Id: types.Int64Value(1),
UserId: types.Int64Value(1), UserId: types.Int64Value(1),
InstanceId: types.StringValue("iid"), InstanceId: types.StringValue("iid"),
ProjectId: types.StringValue("pid"), ProjectId: types.StringValue("pid"),

2
stackit/internal/services/sqlserverflexalpha/user/resource.go
View file

@ -416,6 +416,7 @@ func mapFieldsCreate(userResp *sqlserverflexalpha.CreateUserResponse, model *res
return fmt.Errorf("user id not present") return fmt.Errorf("user id not present")
} }
userId := *user.Id userId := *user.Id
model.Id = types.Int64Value(userId)
model.UserId = types.Int64Value(userId) model.UserId = types.Int64Value(userId)
model.Username = types.StringPointerValue(user.Username) model.Username = types.StringPointerValue(user.Username)
@ -467,6 +468,7 @@ func mapFields(userResp *sqlserverflexalpha.GetUserResponse, model *resourceMode
return fmt.Errorf("user id not present") return fmt.Errorf("user id not present")
} }
model.Id = types.Int64Value(userId)
model.UserId = types.Int64Value(userId) model.UserId = types.Int64Value(userId)
model.Username = types.StringPointerValue(user.Username) model.Username = types.StringPointerValue(user.Username)

14
stackit/internal/wait/postgresflexalpha/wait.go
View file

@ -67,9 +67,11 @@ func CreateInstanceWaitHandler(
if s == nil || s.Id == nil || *s.Id != instanceId || s.Status == nil { if s == nil || s.Id == nil || *s.Id != instanceId || s.Status == nil {
return false, nil, nil return false, nil, nil
} }
tflog.Debug(ctx, "waiting for instance ready", map[string]interface{}{ tflog.Debug(
"status": *s.Status, ctx, "waiting for instance ready", map[string]interface{}{
}) "status": *s.Status,
},
)
switch *s.Status { switch *s.Status {
default: default:
return true, s, fmt.Errorf("instance with id %s has unexpected status %s", instanceId, *s.Status) return true, s, fmt.Errorf("instance with id %s has unexpected status %s", instanceId, *s.Status)
@ -118,13 +120,13 @@ func CreateInstanceWaitHandler(
instanceCreated = true instanceCreated = true
instanceGetResponse = s instanceGetResponse = s
case InstanceStateSuccess: case InstanceStateSuccess:
if *s.Network.AccessScope == "SNA" { if s.Network != nil && s.Network.AccessScope != nil && *s.Network.AccessScope == "SNA" {
if s.Network == nil || s.Network.InstanceAddress == nil { if s.Network.InstanceAddress == nil {
tflog.Warn(ctx, "Waiting for instance_address") tflog.Warn(ctx, "Waiting for instance_address")
return false, nil, nil return false, nil, nil
} }
if s.Network.RouterAddress == nil { if s.Network.RouterAddress == nil {
tflog.Info(ctx, "Waiting for router_address") tflog.Warn(ctx, "Waiting for router_address")
return false, nil, nil return false, nil, nil
} }
} }