From fd29e1019aa40bd640acbcdb305d55cb7807803e Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 5 Nov 2025 10:53:57 +0000 Subject: [PATCH] Update test assertions to check for cursor parameter instead of page/perPage Co-authored-by: SamMorrowDrums <4811358+SamMorrowDrums@users.noreply.github.com> --- pkg/github/actions_test.go | 3 +-- pkg/github/gists_test.go | 3 +-- pkg/github/issues_test.go | 10 +++------- pkg/github/notifications_test.go | 3 +-- pkg/github/pullrequests_test.go | 9 +++------ pkg/github/repositories_test.go | 9 +++------ pkg/github/search_test.go | 12 ++++-------- 7 files changed, 16 insertions(+), 33 deletions(-) diff --git a/pkg/github/actions_test.go b/pkg/github/actions_test.go index abdbba8d..9a26773a 100644 --- a/pkg/github/actions_test.go +++ b/pkg/github/actions_test.go @@ -429,8 +429,7 @@ func Test_ListWorkflowRunArtifacts(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "owner") assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "run_id") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo", "run_id"}) tests := []struct { diff --git a/pkg/github/gists_test.go b/pkg/github/gists_test.go index c27578ff..a89187d5 100644 --- a/pkg/github/gists_test.go +++ b/pkg/github/gists_test.go @@ -23,8 +23,7 @@ func Test_ListGists(t *testing.T) { assert.NotEmpty(t, tool.Description) assert.Contains(t, tool.InputSchema.Properties, "username") assert.Contains(t, tool.InputSchema.Properties, "since") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.Empty(t, tool.InputSchema.Required) // Setup mock gists for success case diff --git a/pkg/github/issues_test.go b/pkg/github/issues_test.go index 03c57ce7..1a861c8e 100644 --- a/pkg/github/issues_test.go +++ b/pkg/github/issues_test.go @@ -254,8 +254,7 @@ func Test_SearchIssues(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "order") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"query"}) // Setup mock search results @@ -749,7 +748,6 @@ func Test_ListIssues(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "direction") assert.Contains(t, tool.InputSchema.Properties, "since") assert.Contains(t, tool.InputSchema.Properties, "after") - assert.Contains(t, tool.InputSchema.Properties, "perPage") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo"}) // Mock issues data @@ -1598,8 +1596,7 @@ func Test_GetIssueComments(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "owner") assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "issue_number") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"method", "owner", "repo", "issue_number"}) // Setup mock comments for success case @@ -2507,8 +2504,7 @@ func Test_GetSubIssues(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "owner") assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "issue_number") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"method", "owner", "repo", "issue_number"}) // Setup mock sub-issues for success case diff --git a/pkg/github/notifications_test.go b/pkg/github/notifications_test.go index 98b13259..e4c2cf3c 100644 --- a/pkg/github/notifications_test.go +++ b/pkg/github/notifications_test.go @@ -27,8 +27,7 @@ func Test_ListNotifications(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "before") assert.Contains(t, tool.InputSchema.Properties, "owner") assert.Contains(t, tool.InputSchema.Properties, "repo") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") // All fields are optional, so Required should be empty assert.Empty(t, tool.InputSchema.Required) diff --git a/pkg/github/pullrequests_test.go b/pkg/github/pullrequests_test.go index a66e2852..fdbd5d31 100644 --- a/pkg/github/pullrequests_test.go +++ b/pkg/github/pullrequests_test.go @@ -590,8 +590,7 @@ func Test_ListPullRequests(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "base") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "direction") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo"}) // Setup mock PRs for success case @@ -836,8 +835,7 @@ func Test_SearchPullRequests(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "order") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"query"}) mockSearchResult := &github.IssuesSearchResult{ @@ -1142,8 +1140,7 @@ func Test_GetPullRequestFiles(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "owner") assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "pullNumber") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"method", "owner", "repo", "pullNumber"}) // Setup mock PR files for success case diff --git a/pkg/github/repositories_test.go b/pkg/github/repositories_test.go index 8baca434..5ad6d40b 100644 --- a/pkg/github/repositories_test.go +++ b/pkg/github/repositories_test.go @@ -765,8 +765,7 @@ func Test_ListCommits(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "repo") assert.Contains(t, tool.InputSchema.Properties, "sha") assert.Contains(t, tool.InputSchema.Properties, "author") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo"}) // Setup mock commits for success case @@ -1675,8 +1674,7 @@ func Test_ListBranches(t *testing.T) { assert.NotEmpty(t, tool.Description) assert.Contains(t, tool.InputSchema.Properties, "owner") assert.Contains(t, tool.InputSchema.Properties, "repo") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo"}) // Setup mock branches for success case @@ -2923,8 +2921,7 @@ func Test_ListStarredRepositories(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "username") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "direction") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.Empty(t, tool.InputSchema.Required) // All parameters are optional // Setup mock starred repositories diff --git a/pkg/github/search_test.go b/pkg/github/search_test.go index c70682f7..37e59871 100644 --- a/pkg/github/search_test.go +++ b/pkg/github/search_test.go @@ -25,8 +25,7 @@ func Test_SearchRepositories(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "query") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "order") - assert.Contains(t, tool.InputSchema.Properties, "page") - assert.Contains(t, tool.InputSchema.Properties, "perPage") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"query"}) // Setup mock search results @@ -239,8 +238,7 @@ func Test_SearchCode(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "query") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "order") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"query"}) // Setup mock search results @@ -394,8 +392,7 @@ func Test_SearchUsers(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "query") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "order") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"query"}) // Setup mock search results @@ -588,8 +585,7 @@ func Test_SearchOrgs(t *testing.T) { assert.Contains(t, tool.InputSchema.Properties, "query") assert.Contains(t, tool.InputSchema.Properties, "sort") assert.Contains(t, tool.InputSchema.Properties, "order") - assert.Contains(t, tool.InputSchema.Properties, "perPage") - assert.Contains(t, tool.InputSchema.Properties, "page") + assert.Contains(t, tool.InputSchema.Properties, "cursor") assert.ElementsMatch(t, tool.InputSchema.Required, []string{"query"}) // Setup mock search results