Skip to content

Commit 6c30443

Browse files
committed
fixed linting issues of avmc
1 parent f1be9e3 commit 6c30443

File tree

3 files changed

+6
-69
lines changed

3 files changed

+6
-69
lines changed

internal/service/odb/cloud_autonomous_vm_cluster.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,6 @@ func (r *resourceCloudAutonomousVmCluster) Read(ctx context.Context, req resourc
447447
}
448448

449449
func (r *resourceCloudAutonomousVmCluster) Update(ctx context.Context, req resource.UpdateRequest, resp *resource.UpdateResponse) {
450-
451450
var plan, state cloudAutonomousVmClusterResourceModel
452451
resp.Diagnostics.Append(req.Plan.Get(ctx, &plan)...)
453452
resp.Diagnostics.Append(req.State.Get(ctx, &state)...)
@@ -474,7 +473,6 @@ func (r *resourceCloudAutonomousVmCluster) Update(ctx context.Context, req resou
474473
}
475474

476475
func (r *resourceCloudAutonomousVmCluster) Delete(ctx context.Context, req resource.DeleteRequest, resp *resource.DeleteResponse) {
477-
478476
conn := r.Meta().ODBClient(ctx)
479477

480478
var state cloudAutonomousVmClusterResourceModel

internal/service/odb/cloud_autonomous_vm_cluster_data_source_test.go

Lines changed: 0 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -8,17 +8,14 @@ import (
88
"fmt"
99
"testing"
1010

11-
"github.com/aws/aws-sdk-go-v2/aws"
1211
"github.com/aws/aws-sdk-go-v2/service/odb"
1312
odbtypes "github.com/aws/aws-sdk-go-v2/service/odb/types"
14-
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/retry"
1513
sdkacctest "github.com/hashicorp/terraform-plugin-testing/helper/acctest"
1614
"github.com/hashicorp/terraform-plugin-testing/helper/resource"
1715
"github.com/hashicorp/terraform-plugin-testing/terraform"
1816
"github.com/hashicorp/terraform-provider-aws/internal/acctest"
1917
"github.com/hashicorp/terraform-provider-aws/internal/conns"
2018
"github.com/hashicorp/terraform-provider-aws/internal/create"
21-
"github.com/hashicorp/terraform-provider-aws/internal/errs"
2219
tfodb "github.com/hashicorp/terraform-provider-aws/internal/service/odb"
2320
"github.com/hashicorp/terraform-provider-aws/internal/tfresource"
2421
"github.com/hashicorp/terraform-provider-aws/names"
@@ -63,50 +60,6 @@ func TestAccODBCloudAutonomousVmClusterDataSource_basic(t *testing.T) {
6360
},
6461
})
6562
}
66-
func (autonomousVMClusterDSTest) checkCloudAutonomousVmClusterExists(ctx context.Context, name string, cloudAutonomousVMCluster *odbtypes.CloudAutonomousVmCluster) resource.TestCheckFunc {
67-
return func(s *terraform.State) error {
68-
rs, ok := s.RootModule().Resources[name]
69-
if !ok {
70-
return create.Error(names.ODB, create.ErrActionCheckingExistence, tfodb.ResNameCloudAutonomousVmCluster, name, errors.New("not found"))
71-
}
72-
73-
if rs.Primary.ID == "" {
74-
return create.Error(names.ODB, create.ErrActionCheckingExistence, tfodb.ResNameCloudAutonomousVmCluster, name, errors.New("not set"))
75-
}
76-
77-
conn := acctest.Provider.Meta().(*conns.AWSClient).ODBClient(ctx)
78-
resp, err := autonomousVMClusterDSTestEntity.findAVMC(ctx, conn, rs.Primary.ID)
79-
if err != nil {
80-
return create.Error(names.ODB, create.ErrActionCheckingExistence, tfodb.ResNameCloudAutonomousVmCluster, rs.Primary.ID, err)
81-
}
82-
83-
*cloudAutonomousVMCluster = *resp
84-
85-
return nil
86-
}
87-
}
88-
89-
func (autonomousVMClusterDSTest) findAVMC(ctx context.Context, conn *odb.Client, id string) (*odbtypes.CloudAutonomousVmCluster, error) {
90-
input := odb.GetCloudAutonomousVmClusterInput{
91-
CloudAutonomousVmClusterId: aws.String(id),
92-
}
93-
out, err := conn.GetCloudAutonomousVmCluster(ctx, &input)
94-
if err != nil {
95-
if errs.IsA[*odbtypes.ResourceNotFoundException](err) {
96-
return nil, &retry.NotFoundError{
97-
LastError: err,
98-
LastRequest: &input,
99-
}
100-
}
101-
return nil, err
102-
}
103-
104-
if out == nil || out.CloudAutonomousVmCluster == nil {
105-
return nil, tfresource.NewEmptyResultError(&input)
106-
}
107-
108-
return out.CloudAutonomousVmCluster, nil
109-
}
11063

11164
func (autonomousVMClusterDSTest) testAccCheckCloudAutonomousVmClusterDestroy(ctx context.Context) resource.TestCheckFunc {
11265
return func(s *terraform.State) error {
@@ -147,7 +100,6 @@ func (autonomousVMClusterDSTest) testAccPreCheck(ctx context.Context, t *testing
147100
}
148101

149102
func (autonomousVMClusterDSTest) avmcBasic() string {
150-
151103
exaInfraDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.exaInfraDisplayNamePrefix)
152104
odbNetworkDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.odbNetDisplayNamePrefix)
153105
avmcDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.autonomousVmClusterDisplayNamePrefix)

internal/service/odb/cloud_autonomous_vm_cluster_test.go

Lines changed: 6 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ func (autonomousVMClusterResourceTest) avmcBasic() string {
276276
odbNetworkDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.odbNetDisplayNamePrefix)
277277
avmcDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.autonomousVmClusterDisplayNamePrefix)
278278

279-
exaInfraRes := autonomousVMClusterDSTestEntity.exaInfra(exaInfraDisplayName)
280-
odbNetRes := autonomousVMClusterDSTestEntity.odbNet(odbNetworkDisplayName)
279+
exaInfraRes := autonomousVMClusterResourceTestEntity.exaInfra(exaInfraDisplayName)
280+
odbNetRes := autonomousVMClusterResourceTestEntity.odbNet(odbNetworkDisplayName)
281281
res := fmt.Sprintf(`
282282
%s
283283
@@ -320,8 +320,8 @@ func (autonomousVMClusterResourceTest) avmcNoTagWithTag() (string, string) {
320320
odbNetworkDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.odbNetDisplayNamePrefix)
321321
avmcDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.autonomousVmClusterDisplayNamePrefix)
322322

323-
exaInfraRes := autonomousVMClusterDSTestEntity.exaInfra(exaInfraDisplayName)
324-
odbNetRes := autonomousVMClusterDSTestEntity.odbNet(odbNetworkDisplayName)
323+
exaInfraRes := autonomousVMClusterResourceTestEntity.exaInfra(exaInfraDisplayName)
324+
odbNetRes := autonomousVMClusterResourceTestEntity.odbNet(odbNetworkDisplayName)
325325
noTag := fmt.Sprintf(`
326326
%s
327327
@@ -401,8 +401,8 @@ func (autonomousVMClusterResourceTest) avmcAllParamsConfig() string {
401401
odbNetworkDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.odbNetDisplayNamePrefix)
402402
avmcDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.autonomousVmClusterDisplayNamePrefix)
403403

404-
exaInfraRes := autonomousVMClusterDSTestEntity.exaInfra(exaInfraDisplayName)
405-
odbNetRes := autonomousVMClusterDSTestEntity.odbNet(odbNetworkDisplayName)
404+
exaInfraRes := autonomousVMClusterResourceTestEntity.exaInfra(exaInfraDisplayName)
405+
odbNetRes := autonomousVMClusterResourceTestEntity.odbNet(odbNetworkDisplayName)
406406
res := fmt.Sprintf(`
407407
%s
408408
@@ -488,16 +488,3 @@ resource "aws_odb_cloud_exadata_infrastructure" "test" {
488488

489489
return resource
490490
}
491-
func (autonomousVMClusterResourceTest) odbNetwork(odbNetDisplayName string) string {
492-
return fmt.Sprintf(`
493-
resource "aws_odb_network" "test" {
494-
display_name = %[1]q
495-
availability_zone_id = "use1-az6"
496-
client_subnet_cidr = "10.2.0.0/24"
497-
backup_subnet_cidr = "10.2.1.0/24"
498-
s3_access = "DISABLED"
499-
zero_etl_access = "DISABLED"
500-
}
501-
`, odbNetDisplayName)
502-
503-
}

0 commit comments

Comments
 (0)