Skip to content

Commit a6d8b28

Browse files
committed
feat(mongodb): fix lint
1 parent e0da605 commit a6d8b28

File tree

4 files changed

+16
-3
lines changed

4 files changed

+16
-3
lines changed

internal/services/mongodb/data_source_instance.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import (
1515
)
1616

1717
func DataSourceInstance() *schema.Resource {
18-
1918
dsSchema := datasource.SchemaFromResourceSchema(ResourceInstance().Schema)
2019

2120
datasource.AddOptionalFieldsToSchema(dsSchema, "name", "region", "project_id")

internal/services/mongodb/instance_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,8 +176,10 @@ func IsInstanceDestroyed(tt *acctest.TestTools) resource.TestCheckFunc {
176176
if err != nil {
177177
return err
178178
}
179-
180179
extractRegion, err := zone.Region()
180+
if err != nil {
181+
return err
182+
}
181183
instance, err := mongodbAPI.GetInstance(&mongodbSDK.GetInstanceRequest{
182184
InstanceID: ID,
183185
Region: extractRegion,

internal/services/mongodb/snapshot.go

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,9 @@ func ResourceSnapshotRead(ctx context.Context, d *schema.ResourceData, m interfa
125125
return diag.FromErr(err)
126126
}
127127
zone, snapshotID, err := zonal.ParseID(d.Id())
128+
if err != nil {
129+
return diag.FromErr(err)
130+
}
128131

129132
instanceID := locality.ExpandID(d.Get("instance_id").(string))
130133
snapshot, err := waitForSnapshot(ctx, mongodbAPI, region, instanceID, snapshotID, d.Timeout(schema.TimeoutCreate))
@@ -152,6 +155,9 @@ func ResourceSnapshotUpdate(ctx context.Context, d *schema.ResourceData, m inter
152155
return diag.FromErr(err)
153156
}
154157
_, snapshotID, err := zonal.ParseID(d.Id())
158+
if err != nil {
159+
return diag.FromErr(err)
160+
}
155161

156162
updateReq := &mongodb.UpdateSnapshotRequest{
157163
SnapshotID: snapshotID,
@@ -181,12 +187,15 @@ func ResourceSnapshotUpdate(ctx context.Context, d *schema.ResourceData, m inter
181187
return ResourceSnapshotRead(ctx, d, m)
182188
}
183189

184-
func ResourceSnapshotDelete(ctx context.Context, d *schema.ResourceData, m interface{}) diag.Diagnostics {
190+
func ResourceSnapshotDelete(_ context.Context, d *schema.ResourceData, m interface{}) diag.Diagnostics {
185191
mongodbAPI, region, err := newAPIWithRegion(d, m)
186192
if err != nil {
187193
return diag.FromErr(err)
188194
}
189195
_, snapshotID, err := zonal.ParseID(d.Id())
196+
if err != nil {
197+
return diag.FromErr(err)
198+
}
190199

191200
deleteReq := &mongodb.DeleteSnapshotRequest{
192201
SnapshotID: snapshotID,

internal/services/mongodb/testfuncs/sweep.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ func testSweepMongodbInstance(_ string) error {
2222
mongodbAPI := mongodb.NewAPI(scwClient)
2323
logging.L.Debugf("sweeper: destroying the mongodb instance in (%s)", zone)
2424
extractRegion, err := zone.Region()
25+
if err != nil {
26+
return fmt.Errorf("error extract region in (%s) in sweeper: %w", zone, err)
27+
}
2528
listInstance, err := mongodbAPI.ListInstances(&mongodb.ListInstancesRequest{
2629
Region: extractRegion,
2730
})

0 commit comments

Comments
 (0)