Skip to content

Commit 4372513

Browse files
authored
Merge pull request #97 from suzuki-shunsuke/feat/support-pagination-CloudProviderSnapshotRestoreJobsServiceOp.List
feat: support the pagination at CloudProviderSnapshotRestoreJobsService.List
2 parents 9228383 + 2d177cb commit 4372513

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

mongodbatlas/cloud_provider_snapshot_restore_jobs.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const (
1414
// endpoints of the MongoDB Atlas API.
1515
// See more: https://docs.atlas.mongodb.com/reference/api/cloudProviderSnapshotRestoreJobs/
1616
type CloudProviderSnapshotRestoreJobsService interface {
17-
List(context.Context, *SnapshotReqPathParameters) (*CloudProviderSnapshotRestoreJobs, *Response, error)
17+
List(context.Context, *SnapshotReqPathParameters, *ListOptions) (*CloudProviderSnapshotRestoreJobs, *Response, error)
1818
Get(context.Context, *SnapshotReqPathParameters) (*CloudProviderSnapshotRestoreJob, *Response, error)
1919
Create(context.Context, *SnapshotReqPathParameters, *CloudProviderSnapshotRestoreJob) (*CloudProviderSnapshotRestoreJob, *Response, error)
2020
Delete(context.Context, *SnapshotReqPathParameters) (*Response, error)
@@ -57,7 +57,7 @@ type CloudProviderSnapshotRestoreJobs struct {
5757

5858
//List gets all cloud provider snapshot restore jobs for the specified cluster.
5959
//See more: https://docs.atlas.mongodb.com/reference/api/cloud-provider-snapshot-restore-jobs-get-all/
60-
func (s *CloudProviderSnapshotRestoreJobsServiceOp) List(ctx context.Context, requestParameters *SnapshotReqPathParameters) (*CloudProviderSnapshotRestoreJobs, *Response, error) {
60+
func (s *CloudProviderSnapshotRestoreJobsServiceOp) List(ctx context.Context, requestParameters *SnapshotReqPathParameters, listOptions *ListOptions) (*CloudProviderSnapshotRestoreJobs, *Response, error) {
6161
if requestParameters.GroupID == "" {
6262
return nil, nil, NewArgError("groupId", "must be set")
6363
}
@@ -66,6 +66,10 @@ func (s *CloudProviderSnapshotRestoreJobsServiceOp) List(ctx context.Context, re
6666
}
6767

6868
path := fmt.Sprintf("%s/%s/clusters/%s/backup/restoreJobs", cloudProviderSnapshotsBasePath, requestParameters.GroupID, requestParameters.ClusterName)
69+
path, err := setListOptions(path, listOptions)
70+
if err != nil {
71+
return nil, nil, err
72+
}
6973

7074
req, err := s.Client.NewRequest(ctx, http.MethodGet, path, nil)
7175
if err != nil {

mongodbatlas/cloud_provider_snapshot_restore_jobs_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ func TestCloudProviderSnapshotRestoreJobs_List(t *testing.T) {
9595
}`)
9696
})
9797

98-
cloudProviderSnapshots, _, err := client.CloudProviderSnapshotRestoreJobs.List(ctx, requestParameters)
98+
cloudProviderSnapshots, _, err := client.CloudProviderSnapshotRestoreJobs.List(ctx, requestParameters, nil)
9999
if err != nil {
100100
t.Fatalf("CloudProviderSnapshotRestoreJobs.List returned error: %v", err)
101101
}

0 commit comments

Comments
 (0)