Change provider code to best practices from terraform-provider-scaffolding project

Author: timofurrer

internal/provider/config.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"crypto/tls"

internal/provider/custom_attribute_helper.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"fmt"

internal/provider/data_source_gitlab_group.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"context"

internal/provider/data_source_gitlab_group_membership.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"context"

internal/provider/data_source_gitlab_group_membership_test.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"fmt"
@@ -12,8 +12,8 @@ func TestAccDataSourceGitlabMembership_basic(t *testing.T) {
 	rInt := acctest.RandInt()
 
 	resource.Test(t, resource.TestCase{
-		PreCheck:  func() { testAccPreCheck(t) },
-		Providers: testAccProviders,
+		PreCheck:          func() { testAccPreCheck(t) },
+		ProviderFactories: providerFactories,
 		Steps: []resource.TestStep{
 			// Create the group and one member
 			{

internal/provider/data_source_gitlab_group_test.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"fmt"
@@ -13,8 +13,8 @@ func TestAccDataSourceGitlabGroup_basic(t *testing.T) {
 	rString := fmt.Sprintf("%s", acctest.RandString(5)) // nolint // TODO: Resolve this golangci-lint issue: S1025: the argument is already a string, there's no need to use fmt.Sprintf (gosimple)
 
 	resource.Test(t, resource.TestCase{
-		PreCheck:  func() { testAccPreCheck(t) },
-		Providers: testAccProviders,
+		PreCheck:          func() { testAccPreCheck(t) },
+		ProviderFactories: providerFactories,
 		Steps: []resource.TestStep{
 			// Get group using its ID
 			{

internal/provider/data_source_gitlab_project.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"context"

internal/provider/data_source_gitlab_project_protected_branch.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"context"

internal/provider/data_source_gitlab_project_protected_branch_test.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"fmt"
@@ -12,8 +12,8 @@ func TestAccDataGitlabProjectProtectedBranch_search(t *testing.T) {
 	projectName := fmt.Sprintf("tf-%s", acctest.RandString(5))
 
 	resource.Test(t, resource.TestCase{
-		PreCheck:  func() { testAccPreCheck(t) },
-		Providers: testAccProviders,
+		PreCheck:          func() { testAccPreCheck(t) },
+		ProviderFactories: providerFactories,
 		Steps: []resource.TestStep{
 			{
 				Config: testAccDataGitlabProjectProtectedBranchConfigGetProjectSearch(projectName),

internal/provider/data_source_gitlab_project_protected_branches.go

@@ -1,4 +1,4 @@
-package gitlab
+package provider
 
 import (
 	"context"