resource/aws_vpc_peering_connection_accepter: Finish import implement…

…ation

Reference: #4486

Output from acceptance testing:

```
--- PASS: TestAccAWSVPCPeeringConnectionAccepter_sameRegionSameAccount (38.63s)
--- PASS: TestAccAWSVPCPeeringConnectionAccepter_sameRegionDifferentAccount (39.03s)
--- PASS: TestAccAWSVPCPeeringConnectionAccepter_differentRegionDifferentAccount (39.65s)
--- PASS: TestAccAWSVPCPeeringConnectionAccepter_differentRegionSameAccount (42.12s)
```

aws/resource_aws_vpc_peering_connection_accepter.go

@@ -1,36 +1,28 @@
 package aws
 
 import (
-	"log"
-
 	"fmt"
+	"log"
 
 	"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
 )
 
-const (
-	schemaNameVPCPeeringConnectionID = "vpc_peering_connection_id"
-)
-
 func resourceAwsVpcPeeringConnectionAccepter() *schema.Resource {
 	return &schema.Resource{
 		Create: resourceAwsVPCPeeringAccepterCreate,
 		Read:   resourceAwsVPCPeeringRead,
 		Update: resourceAwsVPCPeeringUpdate,
 		Delete: resourceAwsVPCPeeringAccepterDelete,
 		Importer: &schema.ResourceImporter{
-			State: func(data *schema.ResourceData, m interface{}) (result []*schema.ResourceData, err error) {
-				err = data.Set(schemaNameVPCPeeringConnectionID, data.Id())
-				if err != nil {
-					return nil, fmt.Errorf("setting attribute '%s': %v", schemaNameVPCPeeringConnectionID, err)
-				}
+			State: func(d *schema.ResourceData, m interface{}) (result []*schema.ResourceData, err error) {
+				d.Set("vpc_peering_connection_id", d.Id())
 
-				return []*schema.ResourceData{data}, nil
+				return []*schema.ResourceData{d}, nil
 			},
 		},
 
 		Schema: map[string]*schema.Schema{
-			schemaNameVPCPeeringConnectionID: {
+			"vpc_peering_connection_id": {
 				Type:     schema.TypeString,
 				Required: true,
 				ForceNew: true,
@@ -67,7 +59,7 @@ func resourceAwsVpcPeeringConnectionAccepter() *schema.Resource {
 }
 
 func resourceAwsVPCPeeringAccepterCreate(d *schema.ResourceData, meta interface{}) error {
-	id := d.Get(schemaNameVPCPeeringConnectionID).(string)
+	id := d.Get("vpc_peering_connection_id").(string)
 	d.SetId(id)
 
 	if err := resourceAwsVPCPeeringRead(d, meta); err != nil {

aws/resource_aws_vpc_peering_connection_accepter_test.go

@@ -52,6 +52,13 @@ func TestAccAWSVPCPeeringConnectionAccepter_sameRegionSameAccount(t *testing.T)
 					resource.TestCheckResourceAttr(resourceNameAccepter, "accept_status", "active"),
 				),
 			},
+			{
+				Config:                  testAccAwsVPCPeeringConnectionAccepterConfigSameRegionSameAccount(rName),
+				ResourceName:            resourceNameAccepter,
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"auto_accept"},
+			},
 		},
 	})
 }
@@ -91,6 +98,13 @@ func TestAccAWSVPCPeeringConnectionAccepter_differentRegionSameAccount(t *testin
 					resource.TestCheckResourceAttr(resourceNameAccepter, "accept_status", "active"),
 				),
 			},
+			{
+				Config:                  testAccAwsVPCPeeringConnectionAccepterConfigDifferentRegionSameAccount(rName),
+				ResourceName:            resourceNameAccepter,
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"auto_accept"},
+			},
 		},
 	})
 }

website/docs/r/vpc_peering_connection_accepter.html.markdown

@@ -116,3 +116,24 @@ connection in the peer VPC over the VPC Peering Connection.
 In addition to all arguments above, the following attributes are exported:
 
 * `id` - The ID of the VPC Peering Connection.
+
+## Import
+
+VPC Peering Connection Accepters can be imported by using the Peering Connection ID, e.g.
+
+```sh
+$ terraform import aws_vpc_peering_connection_accepter.example pcx-12345678
+```
+
+Certain resource arguments, like `auto_accept`, do not have an EC2 API method for reading the information after peering connection creation. If the argument is set in the Terraform configuration on an imported resource, Terraform will always show a difference. To workaround this behavior, either omit the argument from the Terraform configuration or use [`ignore_changes`](/docs/configuration/resources.html#ignore_changes) to hide the difference, e.g.
+
+```hcl
+resource "aws_vpc_peering_connection_accepter" "example" {
+  # ... other configuration ...
+
+  # There is no AWS EC2 API for reading auto_accept
+  lifecycle {
+    ignore_changes = ["auto_accept"]
+  }
+}
+```