Skip to content

Commit ecb86f7

Browse files
authored
Add List Options for ListRepositoryAccessRunnerGroup (google#2086)
Fixes: google#2085.
1 parent ee49f8e commit ecb86f7

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

github/actions_runner_groups.go

+6-1
Original file line numberDiff line numberDiff line change
@@ -159,8 +159,13 @@ func (s *ActionsService) UpdateOrganizationRunnerGroup(ctx context.Context, org
159159
// ListRepositoryAccessRunnerGroup lists the repositories with access to a self-hosted runner group configured in an organization.
160160
//
161161
// GitHub API docs: https://docs.github.com/en/rest/reference/actions#list-repository-access-to-a-self-hosted-runner-group-in-an-organization
162-
func (s *ActionsService) ListRepositoryAccessRunnerGroup(ctx context.Context, org string, groupID int64) (*ListRepositories, *Response, error) {
162+
func (s *ActionsService) ListRepositoryAccessRunnerGroup(ctx context.Context, org string, groupID int64, opts *ListOptions) (*ListRepositories, *Response, error) {
163163
u := fmt.Sprintf("orgs/%v/actions/runner-groups/%v/repositories", org, groupID)
164+
u, err := addOptions(u, opts)
165+
if err != nil {
166+
return nil, nil, err
167+
}
168+
164169
req, err := s.client.NewRequest("GET", u, nil)
165170
if err != nil {
166171
return nil, nil, err

github/actions_runner_groups_test.go

+5-3
Original file line numberDiff line numberDiff line change
@@ -234,11 +234,13 @@ func TestActionsService_ListRepositoryAccessRunnerGroup(t *testing.T) {
234234

235235
mux.HandleFunc("/orgs/o/actions/runner-groups/2/repositories", func(w http.ResponseWriter, r *http.Request) {
236236
testMethod(t, r, "GET")
237+
testFormValues(t, r, values{"per_page": "1", "page": "1"})
237238
fmt.Fprint(w, `{"total_count": 1, "repositories": [{"id": 43, "node_id": "MDEwOlJlcG9zaXRvcnkxMjk2MjY5", "name": "Hello-World", "full_name": "octocat/Hello-World"}]}`)
238239
})
239240

240241
ctx := context.Background()
241-
groups, _, err := client.Actions.ListRepositoryAccessRunnerGroup(ctx, "o", 2)
242+
opts := &ListOptions{Page: 1, PerPage: 1}
243+
groups, _, err := client.Actions.ListRepositoryAccessRunnerGroup(ctx, "o", 2, opts)
242244
if err != nil {
243245
t.Errorf("Actions.ListRepositoryAccessRunnerGroup returned error: %v", err)
244246
}
@@ -255,12 +257,12 @@ func TestActionsService_ListRepositoryAccessRunnerGroup(t *testing.T) {
255257

256258
const methodName = "ListRepositoryAccessRunnerGroup"
257259
testBadOptions(t, methodName, func() (err error) {
258-
_, _, err = client.Actions.ListRepositoryAccessRunnerGroup(ctx, "\n", 2)
260+
_, _, err = client.Actions.ListRepositoryAccessRunnerGroup(ctx, "\n", 2, opts)
259261
return err
260262
})
261263

262264
testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
263-
got, resp, err := client.Actions.ListRepositoryAccessRunnerGroup(ctx, "o", 2)
265+
got, resp, err := client.Actions.ListRepositoryAccessRunnerGroup(ctx, "o", 2, opts)
264266
if got != nil {
265267
t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
266268
}

0 commit comments

Comments
 (0)