Fix pagination for ListCopilotSeats (#3112)

google · Mar 26, 2024 · 1891239 · 1891239
1 parent 8bec011
commit 1891239
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/github/copilot.go b/github/copilot.go
@@ -161,8 +161,12 @@ func (s *CopilotService) GetCopilotBilling(ctx context.Context, org string) (*Co
 //meta:operation GET /orgs/{org}/copilot/billing/seats
 func (s *CopilotService) ListCopilotSeats(ctx context.Context, org string, opts *ListOptions) (*ListCopilotSeatsResponse, *Response, error) {
 	u := fmt.Sprintf("orgs/%v/copilot/billing/seats", org)
+	u, err := addOptions(u, opts)
+	if err != nil {
+		return nil, nil, err
+	}
 
-	req, err := s.client.NewRequest("GET", u, opts)
+	req, err := s.client.NewRequest("GET", u, nil)
 	if err != nil {
 		return nil, nil, err
 	}

diff --git a/github/copilot_test.go b/github/copilot_test.go
@@ -584,8 +584,9 @@ func TestCopilotService_ListCopilotSeats(t *testing.T) {
 
 	const methodName = "ListCopilotSeats"
 
+	opts := &ListOptions{Page: 2}
 	testBadOptions(t, methodName, func() (err error) {
-		_, _, err = client.Copilot.ListCopilotSeats(ctx, "\n", nil)
+		_, _, err = client.Copilot.ListCopilotSeats(ctx, "\n", opts)
 		return err
 	})