fix: eip unit test (#933)

* fix: eip unit test

* fix: fmt resource_tc_vpn_ssl_client.go

Co-authored-by: mikatong <mikatong@tencent.com>

Author: tongyiming

tencentcloud/resource_tc_eip_test.go

@@ -3,6 +3,7 @@ package tencentcloud
 import (
 	"context"
 	"fmt"
+	"os"
 	"testing"
 
 	"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
@@ -72,6 +73,10 @@ func TestAccTencentCloudEip_basic(t *testing.T) {
 
 func TestAccTencentCloudEip_anycast(t *testing.T) {
 	t.Parallel()
+	defer func() {
+		os.Setenv(PROVIDER_REGION, "")
+	}()
+	os.Setenv(PROVIDER_REGION, "ap-hongkong")
 	resource.Test(t, resource.TestCase{
 		PreCheck:     func() { testAccPreCheck(t) },
 		Providers:    testAccProviders,
@@ -234,14 +239,14 @@ const testAccEipAnycast = `
 resource "tencentcloud_eip" "foo" {
   name = "eip_anycast"
   type = "AnycastEIP"
-  anycast_zone = "ANYCAST_ZONE_GLOBAL"
+  anycast_zone = "ANYCAST_ZONE_OVERSEAS"
 }
 `
 
 const testAccEipProvider = `
 resource "tencentcloud_eip" "foo" {
   name = "eip_provider"
-  internet_service_provider = "CMCC"
+  internet_service_provider = "CUCC"
 }
 `
 

tencentcloud/resource_tc_vpn_ssl_client.go

@@ -125,7 +125,7 @@ func resourceTencentCloudVpnSslClientCreate(d *schema.ResourceData, meta interfa
 		return fmt.Errorf("get instance list error: %s", err.Error())
 	}
 	if len(newIns) != 1 {
-		return  fmt.Errorf("create ssl client error")
+		return fmt.Errorf("create ssl client error")
 	}
 
 	sslClient := newIns[0]

tencentcloud/service_tencentcloud_vpc.go

@@ -2128,7 +2128,7 @@ func (me *VpcService) UnattachEip(ctx context.Context, eipId string) error {
 	}
 
 	// DisassociateAddress Doesn't support Disassociate NAT Address
-	if strings.HasPrefix(*eip.InstanceId, "nat-") {
+	if eip.InstanceId != nil && strings.HasPrefix(*eip.InstanceId, "nat-") {
 		request := vpc.NewDisassociateNatGatewayAddressRequest()
 		request.NatGatewayId = eip.InstanceId
 		request.PublicIpAddresses = []*string{eip.AddressIp}