Merge pull request #59 from vmvarela/fix/53-scim-buildtwopartid

fix(scim): use buildTwoPartID for data source IDs

Author: vmvarela

github/data_source_github_enterprise_scim_group.go

@@ -2,7 +2,6 @@ package github
 
 import (
 	"context"
-	"fmt"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -39,7 +38,7 @@ func dataSourceGithubEnterpriseSCIMGroupRead(ctx context.Context, d *schema.Reso
 		return diag.FromErr(err)
 	}
 
-	d.SetId(fmt.Sprintf("%s/%s", enterprise, scimGroupID))
+	d.SetId(buildTwoPartID(enterprise, scimGroupID))
 
 	if err := d.Set("schemas", group.Schemas); err != nil {
 		return diag.FromErr(err)

github/data_source_github_enterprise_scim_user.go

@@ -2,7 +2,6 @@ package github
 
 import (
 	"context"
-	"fmt"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -39,7 +38,7 @@ func dataSourceGithubEnterpriseSCIMUserRead(ctx context.Context, d *schema.Resou
 		return diag.FromErr(err)
 	}
 
-	d.SetId(fmt.Sprintf("%s/%s", enterprise, scimUserID))
+	d.SetId(buildTwoPartID(enterprise, scimUserID))
 
 	if err := d.Set("schemas", user.Schemas); err != nil {
 		return diag.FromErr(err)