Fix the project metadata test in CI + fix new linter errors (#3267)

<!-- This change is generated by MagicModules. -->
/cc @rileykarson

google/resource_bigtable_instance.go

@@ -138,9 +138,6 @@ func resourceBigtableInstanceCreate(d *schema.ResourceData, meta interface{}) er
 	}
 
 	conf.Clusters = expandBigtableClusters(d.Get("cluster").(*schema.Set).List(), conf.InstanceID)
-	if err != nil {
-		return fmt.Errorf("error expanding clusters: %s", err.Error())
-	}
 
 	c, err := config.bigtableClientFactory.NewInstanceAdminClient(project)
 	if err != nil {

google/resource_compute_project_metadata_item_test.go

@@ -161,7 +161,7 @@ func testAccCheckProjectMetadataItemDestroy(s *terraform.State) error {
 }
 
 func testAccProjectMetadataItem_basic(key, val string) string {
-	return testAccProjectMetadataItem_basicWithResourceName(acctest.RandString(10), key, val)
+	return testAccProjectMetadataItem_basicWithResourceName(fmt.Sprintf("test_%s", acctest.RandString(10)), key, val)
 }
 
 func testAccProjectMetadataItem_basicWithResourceName(resourceName, key, val string) string {

google/resource_storage_transfer_job.go

@@ -5,7 +5,6 @@ import (
 	"github.com/hashicorp/terraform/helper/resource"
 	"github.com/hashicorp/terraform/helper/schema"
 	"github.com/hashicorp/terraform/helper/validation"
-	"google.golang.org/api/googleapi"
 	"google.golang.org/api/storagetransfer/v1"
 	"log"
 	"strings"
@@ -365,7 +364,7 @@ func resourceStorageTransferJobRead(d *schema.ResourceData, meta interface{}) er
 		return err
 	}
 
-	d.Set("transfer_spec", flattenTransferSpec(res.TransferSpec, d))
+	err = d.Set("transfer_spec", flattenTransferSpec(res.TransferSpec, d))
 	if err != nil {
 		return err
 	}
@@ -458,9 +457,7 @@ func resourceStorageTransferJobDelete(d *schema.ResourceData, meta interface{})
 		if err != nil {
 			return resource.RetryableError(err)
 		}
-		if gerr, ok := err.(*googleapi.Error); ok && gerr.Code == 429 {
-			return resource.RetryableError(gerr)
-		}
+
 		return nil
 	})