Move files to the sweeper and envvar packages (#8198) (#5824)

* Move files to the package sweep and envvar

* Replace function calls

* Rename sweeper package

* Fix rebasing error

Signed-off-by: Modular Magician <[email protected]>

Author: modular-magician

.changelog/8198.txt

@@ -0,0 +1,3 @@
+```release-note:none
+
+```

google-beta/acctest/gcp_sweeper.go

@@ -3,55 +3,21 @@
 package acctest
 
 import (
-	"fmt"
-	"strings"
-
+	"github.com/hashicorp/terraform-provider-google-beta/google-beta/sweeper"
 	transport_tpg "github.com/hashicorp/terraform-provider-google-beta/google-beta/transport"
 )
 
-// List of prefixes used for test resource names
-var testResourcePrefixes = []string{
-	// tf-test and tf_test are automatically prepended to resource ids in examples that
-	// include a "-" or "_" respectively, and they are the preferred prefix for our test resources to use
-	"tf-test",
-	"tf_test",
-	"tfgen",
-	"gke-us-central1-tf",  // composer-created disks which are abandoned by design (https://cloud.google.com/composer/pricing)
-	"gcs-bucket-tf-test-", // https://github.com/hashicorp/terraform-provider-google/issues/8909
-	"df-",                 // https://github.com/hashicorp/terraform-provider-google/issues/8909
-	"resourcegroup-",      // https://github.com/hashicorp/terraform-provider-google/issues/8924
-	"cluster-",            // https://github.com/hashicorp/terraform-provider-google/issues/8924
-	"k8s-fw-",             // firewall rules are getting created and not cleaned up by k8 resources using this prefix
-}
-
 // SharedConfigForRegion returns a common config setup needed for the sweeper
 // functions for a given region
+//
+// Deprecated: For backward compatibility SharedConfigForRegion is still working,
+// but all new code should use SharedConfigForRegion in the sweeper package instead.
 func SharedConfigForRegion(region string) (*transport_tpg.Config, error) {
-	project := GetTestProjectFromEnv()
-	if project == "" {
-		return nil, fmt.Errorf("set project using any of these env variables %v", ProjectEnvVars)
-	}
-
-	if v := transport_tpg.MultiEnvSearch(CredsEnvVars); v == "" {
-		return nil, fmt.Errorf("set credentials using any of these env variables %v", CredsEnvVars)
-	}
-
-	conf := &transport_tpg.Config{
-		Credentials: GetTestCredsFromEnv(),
-		Region:      region,
-		Project:     project,
-	}
-
-	transport_tpg.ConfigureBasePaths(conf)
-
-	return conf, nil
+	return sweeper.SharedConfigForRegion(region)
 }
 
+// Deprecated: For backward compatibility IsSweepableTestResource is still working,
+// but all new code should use IsSweepableTestResource in the sweeper package instead.
 func IsSweepableTestResource(resourceName string) bool {
-	for _, p := range testResourcePrefixes {
-		if strings.HasPrefix(resourceName, p) {
-			return true
-		}
-	}
-	return false
+	return sweeper.IsSweepableTestResource(resourceName)
 }