Add set check

Author: gordonbondon

mongodbatlas/resource_mongodbatlas_vpc_peering_connection.go

@@ -312,9 +312,13 @@ func resourceVpcPeeringConnectionImportState(d *schema.ResourceData, meta interf
 	// https://docs.atlas.mongodb.com/reference/api/vpc-get-connection/#example-response
 	// Atlas API does not return ProviderName, so we have to guess it from other parameters
 	if peer.AwsAccountID != "" {
-		d.Set("provider_name", "AWS")
+		if err := d.Set("provider_name", "AWS"); err != nil {
+			return nil, fmt.Errorf("Error setting provider name: %v", err)
+		}
 	} else if peer.GcpProjectID != "" {
-		d.Set("provider_name", "GCP")
+		if err := d.Set("provider_name", "GCP"); err != nil {
+			return nil, fmt.Errorf("Error setting provider name: %v", err)
+		}
 	}
 
 	d.SetId(peer.ID)

mongodbatlas/resource_mongodbatlas_vpc_peering_connection_migrate.go

@@ -7,48 +7,48 @@ import (
 func resourceVpcPeeringConnectionResourceV0() *schema.Resource {
 	return &schema.Resource{
 		Schema: map[string]*schema.Schema{
-			"group": &schema.Schema{
+			"group": {
 				Type:     schema.TypeString,
 				Required: true,
 				ForceNew: true,
 			},
-			"route_table_cidr_block": &schema.Schema{
+			"route_table_cidr_block": {
 				Type:     schema.TypeString,
 				Required: true,
 			},
-			"aws_account_id": &schema.Schema{
+			"aws_account_id": {
 				Type:     schema.TypeString,
 				Required: true,
 				ForceNew: true,
 			},
-			"vpc_id": &schema.Schema{
+			"vpc_id": {
 				Type:     schema.TypeString,
 				Required: true,
 				ForceNew: true,
 			},
-			"container_id": &schema.Schema{
+			"container_id": {
 				Type:     schema.TypeString,
 				Required: true,
 				ForceNew: true,
 			},
-			"identifier": &schema.Schema{
+			"identifier": {
 				Type:     schema.TypeString,
 				Optional: true,
 				ForceNew: true,
 				Computed: true,
 			},
-			"connection_id": &schema.Schema{
+			"connection_id": {
 				Type:     schema.TypeString,
 				Optional: true,
 				ForceNew: true,
 				Computed: true,
 			},
-			"status_name": &schema.Schema{
+			"status_name": {
 				Type:     schema.TypeString,
 				Optional: true,
 				Computed: true,
 			},
-			"error_state_name": &schema.Schema{
+			"error_state_name": {
 				Type:     schema.TypeString,
 				Optional: true,
 				Computed: true,