Skip to content

Commit 5060ec7

Browse files
fix: orgID handling in tests
1 parent 8ee2168 commit 5060ec7

File tree

4 files changed

+576
-555
lines changed

4 files changed

+576
-555
lines changed

internal/meta/errors.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,3 @@ import "errors"
44

55
// ErrProjectIDNotFound is returned when no project ID can be detected
66
var ErrProjectIDNotFound = errors.New("could not detect project id")
7-
8-
// ErrOrganizationIDNotFound is returned when no organization ID can be detected
9-
var ErrOrganizationIDNotFound = errors.New("could not detect organization id")

internal/meta/extractors.go

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -116,24 +116,6 @@ func ExtractProjectID(d terraformResourceData, m any) (projectID string, isDefau
116116
return "", false, ErrProjectIDNotFound
117117
}
118118

119-
// ExtractOrganizationID will try to guess the organization id from the following:
120-
// - organization_id field of the resource data
121-
// - default organization_id from config
122-
func ExtractOrganizationID(d terraformResourceData, m any) (organizationID string, err error) {
123-
rawOrgID, exist := d.GetOk("organization_id")
124-
125-
if exist {
126-
return rawOrgID.(string), nil
127-
}
128-
129-
defaultOrgID, defaultOrgIDExists := m.(*Meta).ScwClient().GetDefaultOrganizationID()
130-
if defaultOrgIDExists {
131-
return defaultOrgID, nil
132-
}
133-
134-
return "", ErrOrganizationIDNotFound
135-
}
136-
137119
func ExtractScwClient(m any) *scw.Client {
138120
return m.(*Meta).ScwClient()
139121
}

internal/services/audittrail/helpers.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ import (
88
audittrailSDK "github.com/scaleway/scaleway-sdk-go/api/audit_trail/v1alpha1"
99
"github.com/scaleway/scaleway-sdk-go/scw"
1010
"github.com/scaleway/terraform-provider-scaleway/v2/internal/meta"
11+
"github.com/scaleway/terraform-provider-scaleway/v2/internal/services/account"
12+
"github.com/scaleway/terraform-provider-scaleway/v2/internal/types"
1113
)
1214

1315
// newAPIWithRegionAndProjectID returns a new Audit Trail API, with region and projectID
@@ -23,10 +25,7 @@ func newAPIWithRegionAndOrgID(d *schema.ResourceData, m any) (*audittrailSDK.API
2325
return nil, "", "", fmt.Errorf("invalid api region, expected one of %s, got: %s", api.Regions(), region)
2426
}
2527

26-
orgID, err := meta.ExtractOrganizationID(d, m)
27-
if err != nil {
28-
return nil, "", "", err
29-
}
28+
orgID := types.FlattenStringPtr(account.GetOrganizationID(m, d)).(string)
3029

3130
return api, region, orgID, nil
3231
}

0 commit comments

Comments
 (0)