Skip to content

Commit 9f2984e

Browse files
rocktaviousrocktavious
authored
Bump default page size from 100 to 500 (#576)
* Bump default page size from 100 to 500 * Fix tests what include information for pagesize
1 parent a097777 commit 9f2984e

File tree

10 files changed

+30
-27
lines changed

10 files changed

+30
-27
lines changed

‎.changes/unreleased/Feature-20250730-094006.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
kind: Feature
2+
body: Increase default pagesize from 100 to 500
3+
time: 2025-07-30T09:40:06.51662-04:00

‎cache_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,42 +21,42 @@ func TestCache(t *testing.T) {
2121
)
2222
testRequestThree := autopilot.NewTestRequest(
2323
`query SystemsList($after:String!$first:Int!){account{systems(after: $after, first: $first){nodes{id,aliases,description,htmlUrl,managedAliases,name,note,owner{... on Team{teamAlias:alias,id}},parent{id,aliases,description,htmlUrl,managedAliases,name,note,owner{... on Team{teamAlias:alias,id}}}},{{ template "pagination_request" }}}}}`,
24-
`{ "after": "", "first": 100 }`,
24+
`{ "after": "", "first": 500 }`,
2525
`{"data":{"account":{ "systems":{ "nodes":[{{ template "system1_response" }}] } }}}`,
2626
)
2727
testRequestFour := autopilot.NewTestRequest(
2828
`query TeamList($after:String!$first:Int!){account{teams(after: $after, first: $first){nodes{alias,id,aliases,managedAliases,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,manager{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},memberships{nodes{role,team{alias,id},user{id,email,name}},{{ template "pagination_request" }}},name,parentTeam{alias,id},responsibilities,tags{nodes{id,key,value},{{ template "pagination_request" }}}},{{ template "pagination_request" }}}}}`,
29-
`{ "after": "", "first": 100 }`,
29+
`{ "after": "", "first": 500 }`,
3030
`{"data":{"account":{ "teams":{ "nodes":[{{ template "team_1" }}] } }}}`,
3131
)
3232
testRequestFive := autopilot.NewTestRequest(
3333
`query CategoryList($after:String!$first:Int!){account{rubric{categories(after: $after, first: $first){nodes{description,id,name},{{ template "pagination_request" }}}}}}`,
34-
`{ "after": "", "first": 100 }`,
34+
`{ "after": "", "first": 500 }`,
3535
`{"data":{"account":{"rubric":{ "categories":{ "nodes":[{{ template "category_1" }}] } }}}}`,
3636
)
3737
testRequestSix := autopilot.NewTestRequest(
3838
`query LevelsList($after:String!$first:Int!){account{rubric{levels(after: $after, first: $first){nodes{alias,checks{id,name},description,id,index,name},{{ template "pagination_request" }}}}}}`,
39-
`{ "after": "", "first": 100 }`,
39+
`{ "after": "", "first": 500 }`,
4040
`{"data":{"account":{"rubric":{ "levels":{ "nodes":[{{ template "level_1" }}] } }}}}`,
4141
)
4242
testRequestSeven := autopilot.NewTestRequest(
4343
`query FilterList($after:String!$first:Int!){account{filters(after: $after, first: $first){nodes{id,name,connective,htmlUrl,predicates{caseSensitive,key,keyData,type,value}},{{ template "pagination_request" }}}}}`,
44-
`{ "after": "", "first": 100 }`,
44+
`{ "after": "", "first": 500 }`,
4545
`{"data":{"account":{ "filters":{ "nodes":[{{ template "filter_1" }}] } }}}`,
4646
)
4747
testRequestEight := autopilot.NewTestRequest(
4848
`query IntegrationList($after:String!$first:Int!){account{integrations(after: $after, first: $first){nodes{{ template "integration_request" }},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
49-
`{ "after": "", "first": 100 }`,
49+
`{ "after": "", "first": 500 }`,
5050
`{"data":{"account":{ "integrations":{ "nodes":[{{ template "integration_1" }}] } }}}`,
5151
)
5252
testRequestNine := autopilot.NewTestRequest(
5353
`query RepositoryList($after:String!$first:Int!$visible:Boolean!){account{repositories(after: $after, first: $first, visible: $visible){hiddenCount,nodes{archivedAt,createdOn,defaultAlias,defaultBranch,description,forked,htmlUrl,id,languages{name,usage},lastOwnerChangedAt,locked,name,organization,owner{alias,id},private,repoKey,sbomGenerationConfiguration{disabledReason,enabled,nextGenerationAt,state},services{edges{atRoot,node{id,aliases},paths{href,path},serviceRepositories{baseDirectory,displayName,id,repository{id,defaultAlias},service{id,aliases}}},{{ template "pagination_request" }}},tags{nodes{id,key,value},{{ template "pagination_request" }}},tier{alias,description,id,index,name},type,url,visible},organizationCount,ownedCount,{{ template "pagination_request" }},visibleCount}}}`,
54-
`{ "after": "", "first": 100, "visible": true }`,
54+
`{ "after": "", "first": 500, "visible": true }`,
5555
`{"data":{"account":{ "repositories":{ "hiddenCount": 0, "nodes":[{{ template "repository_1" }}] } }}}`,
5656
)
5757
testRequestTen := autopilot.NewTestRequest(
5858
`query InfrastructureResourceSchemaList($after:String!$first:Int!){account{infrastructureResourceSchemas(after: $after, first: $first){nodes{type,schema},{{ template "pagination_request" }}}}}`,
59-
`{ "after": "", "first": 100 }`,
59+
`{ "after": "", "first": 500 }`,
6060
`{"data":{"account":{ "infrastructureResourceSchemas":{ "nodes":[ {{ template "infra_schema_1" }} ] }}}}`,
6161
)
6262
testRequestEleven := autopilot.TestRequest{}

‎campaign_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,12 @@ func TestListCampaigns(t *testing.T) {
1111
// Arrange
1212
testRequestOne := autopilot.NewTestRequest(
1313
`query CampaignsList($after:String!$first:Int!$sortBy:CampaignSortEnum!$status:String!){account{campaigns(first: $first, after: $after, sortBy: $sortBy, filter: [{key: status, arg: $status}]){nodes{checkStats{total,totalSuccessful},endedDate,filter{id,name},htmlUrl,id,name,owner{alias,id},projectBrief,rawProjectBrief,reminder{channels,daysOfWeek,defaultSlackChannel,frequency,frequencyUnit,message,nextOccurrence,timeOfDay,timezone},serviceStats{total,totalSuccessful},startDate,status,targetDate},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
14-
`{ "after": "", "first": 100, "sortBy": "start_date_DESC", "status": "in_progress" }`,
14+
`{ "after": "", "first": 500, "sortBy": "start_date_DESC", "status": "in_progress" }`,
1515
`{ "data": { "account": { "campaigns": { "nodes": [ {{ template "campaign1_response" }}, {{ template "campaign2_response" }} ], {{ template "pagination_initial_pageInfo_response" }} }}}}`,
1616
)
1717
testRequestTwo := autopilot.NewTestRequest(
1818
`query CampaignsList($after:String!$first:Int!$sortBy:CampaignSortEnum!$status:String!){account{campaigns(first: $first, after: $after, sortBy: $sortBy, filter: [{key: status, arg: $status}]){nodes{checkStats{total,totalSuccessful},endedDate,filter{id,name},htmlUrl,id,name,owner{alias,id},projectBrief,rawProjectBrief,reminder{channels,daysOfWeek,defaultSlackChannel,frequency,frequencyUnit,message,nextOccurrence,timeOfDay,timezone},serviceStats{total,totalSuccessful},startDate,status,targetDate},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
19-
`{ "after": "OA", "first": 100, "sortBy": "start_date_DESC", "status": "in_progress" }`,
19+
`{ "after": "OA", "first": 500, "sortBy": "start_date_DESC", "status": "in_progress" }`,
2020
`{ "data": { "account": { "campaigns": { "nodes": [ {{ template "campaign3_response" }} ], {{ template "pagination_second_pageInfo_response" }} }}}}`,
2121
)
2222
requests := []autopilot.TestRequest{testRequestOne, testRequestTwo}
@@ -90,7 +90,7 @@ func TestListCampaignsEmpty(t *testing.T) {
9090
// Arrange
9191
testRequest := autopilot.NewTestRequest(
9292
`query CampaignsList($after:String!$first:Int!$sortBy:CampaignSortEnum!$status:String!){account{campaigns(first: $first, after: $after, sortBy: $sortBy, filter: [{key: status, arg: $status}]){nodes{checkStats{total,totalSuccessful},endedDate,filter{id,name},htmlUrl,id,name,owner{alias,id},projectBrief,rawProjectBrief,reminder{channels,daysOfWeek,defaultSlackChannel,frequency,frequencyUnit,message,nextOccurrence,timeOfDay,timezone},serviceStats{total,totalSuccessful},startDate,status,targetDate},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
93-
`{ "after": "", "first": 100, "sortBy": "start_date_DESC", "status": "in_progress" }`,
93+
`{ "after": "", "first": 500, "sortBy": "start_date_DESC", "status": "in_progress" }`,
9494
`{ "data": { "account": { "campaigns": { "nodes": [], "pageInfo": { "hasNextPage": false, "hasPreviousPage": false, "startCursor": null, "endCursor": null } }}}}`,
9595
)
9696

‎client.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func newClientSettings(options ...Option) *ClientSettings {
2727
token: os.Getenv("OPSLEVEL_API_TOKEN"),
2828
timeout: time.Second * 10,
2929
retries: 10,
30-
pageSize: 100,
30+
pageSize: 500,
3131
transport: http.DefaultTransport,
3232
headers: map[string]string{
3333
"User-Agent": buildUserAgent(""),

‎infra_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,12 +102,12 @@ func TestListInfra(t *testing.T) {
102102
// Arrange
103103
testRequestOne := autopilot.NewTestRequest(
104104
`query InfrastructureResourceList($after:String!$all:Boolean!$first:Int!){account{infrastructureResources(after: $after, first: $first){nodes{id,aliases,name,type @include(if: $all),providerResourceType @include(if: $all),providerData @include(if: $all){accountName,externalUrl,providerName},owner @include(if: $all){... on Team{teamAlias:alias,id}},ownerLocked @include(if: $all),data @include(if: $all),rawData @include(if: $all)},{{ template "pagination_request" }}}}}`,
105-
`{ "after": "", "all": true, "first": 100 }`,
105+
`{ "after": "", "all": true, "first": 500 }`,
106106
`{ "data": { "account": { "infrastructureResources": { "nodes": [ {{ template "infra_1" }}, {{ template "infra_2" }} ], {{ template "pagination_initial_pageInfo_response" }} }}}}`,
107107
)
108108
testRequestTwo := autopilot.NewTestRequest(
109109
`query InfrastructureResourceList($after:String!$all:Boolean!$first:Int!){account{infrastructureResources(after: $after, first: $first){nodes{id,aliases,name,type @include(if: $all),providerResourceType @include(if: $all),providerData @include(if: $all){accountName,externalUrl,providerName},owner @include(if: $all){... on Team{teamAlias:alias,id}},ownerLocked @include(if: $all),data @include(if: $all),rawData @include(if: $all)},{{ template "pagination_request" }}}}}`,
110-
`{ "after": "OA", "all": true, "first": 100 }`,
110+
`{ "after": "OA", "all": true, "first": 500 }`,
111111
`{ "data": { "account": { "infrastructureResources": { "nodes": [ {{ template "infra_3" }} ], {{ template "pagination_second_pageInfo_response" }} }}}}`,
112112
)
113113
requests := []autopilot.TestRequest{testRequestOne, testRequestTwo}

‎level_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ func TestListRubricLevels(t *testing.T) {
7070
// Arrange
7171
testRequest := autopilot.NewTestRequest(
7272
`query LevelsList($after:String!$first:Int!){account{rubric{levels(after: $after, first: $first){nodes{alias,checks{id,name},description,id,index,name},{{ template "pagination_request" }}}}}}`,
73-
`{"after":"","first":100}`,
73+
`{"after":"","first":500}`,
7474
`{
7575
"data": {
7676
"account": {

‎maturity_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ func TestListServicesMaturity(t *testing.T) {
134134
// Arrange
135135
testRequest := autopilot.NewTestRequest(
136136
`query ServiceMaturityList($after:String!$first:Int!){account{services(after: $after, first: $first){nodes{name,maturityReport{categoryBreakdown{category{description,id,name},level{alias,checks{id,name},description,id,index,name}},overallLevel{alias,checks{id,name},description,id,index,name}}},{{ template "pagination_request" }}}}}`,
137-
`{"after":"", "first":100}`,
137+
`{"after":"", "first":500}`,
138138
`{
139139
"data": {
140140
"account": {

‎team_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,7 @@ func TestListTeamsWithManager(t *testing.T) {
431431
// Arrange
432432
testRequestOne := autopilot.NewTestRequest(
433433
`query TeamList($after:String!$email:String!$first:Int!){account{teams(managerEmail: $email, after: $after, first: $first){nodes{alias,id,aliases,managedAliases,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,manager{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},memberships{nodes{role,team{alias,id},user{id,email,name}},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}},name,parentTeam{alias,id},responsibilities,tags{nodes{id,key,value},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
434-
`{ "after": "", "first": 100, "email": "kyle@opslevel.com" }`,
434+
`{ "after": "", "first": 500, "email": "kyle@opslevel.com" }`,
435435
`{ "data": {
436436
"account": {
437437
"teams": {
@@ -502,7 +502,7 @@ func TestListTeamsWithManager(t *testing.T) {
502502
)
503503
testRequestTwo := autopilot.NewTestRequest(
504504
`query TeamList($after:String!$email:String!$first:Int!){account{teams(managerEmail: $email, after: $after, first: $first){nodes{alias,id,aliases,managedAliases,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,manager{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},memberships{nodes{role,team{alias,id},user{id,email,name}},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}},name,parentTeam{alias,id},responsibilities,tags{nodes{id,key,value},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
505-
`{ "after": "OA", "first": 100, "email": "kyle@opslevel.com" }`,
505+
`{ "after": "OA", "first": 500, "email": "kyle@opslevel.com" }`,
506506
`{ "data": {
507507
"account": {
508508
"teams": {
@@ -936,7 +936,7 @@ func TestSearchTeams(t *testing.T) {
936936
// Arrange
937937
testRequest := autopilot.NewTestRequest(
938938
`query TeamSearch($after:String!$first:Int!$searchTerm:String!){account{teams(searchTerm: $searchTerm, after: $after, first: $first){nodes{alias,id,aliases,managedAliases,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,manager{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},memberships{nodes{role,team{alias,id},user{id,email,name}},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}},name,parentTeam{alias,id},responsibilities,tags{nodes{id,key,value},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}},pageInfo{hasNextPage,hasPreviousPage,startCursor,endCursor}}}}`,
939-
`{"searchTerm": "DevOps", "after": "", "first": 100}`,
939+
`{"searchTerm": "DevOps", "after": "", "first": 500}`,
940940
`{ "data": { "account": { "teams": { "nodes": [ { "alias": "devops", "id": "id1", "aliases": ["devops"], "managedAliases": ["devops"], "contacts": [], "htmlUrl": "https://app.opslevel.com/teams/devops", "manager": { "id": "user1", "email": "manager@opslevel.com", "name": "Manager" }, "memberships": { "nodes": [], "pageInfo": { "hasNextPage": false, "hasPreviousPage": false, "startCursor": null, "endCursor": null } }, "name": "DevOps", "parentTeam": null, "responsibilities": "Own Infra & Tools.", "tags": { "nodes": [], "pageInfo": { "hasNextPage": false, "hasPreviousPage": false, "startCursor": null, "endCursor": null } } } ], "pageInfo": { "hasNextPage": false, "hasPreviousPage": false, "startCursor": null, "endCursor": null } } } } }`,
941941
)
942942

‎testdata/templates/common.tpl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,22 +46,22 @@
4646
}{{ end }}
4747
{{- define "first_page_variables" }}
4848
"after": "",
49-
"first": 100
49+
"first": 500
5050
{{ end }}
5151
{{- define "second_page_variables" }}
5252
"after": "OA",
53-
"first": 100
53+
"first": 500
5454
{{ end }}
5555
{{- define "pagination_initial_query_variables" }}
5656
{
5757
"after": "",
58-
"first": 100
58+
"first": 500
5959
}
6060
{{ end }}
6161
{{- define "pagination_second_query_variables" }}
6262
{
6363
"after": "OA",
64-
"first": 100
64+
"first": 500
6565
}
6666
{{ end }}
6767
{{- define "next_page_false" }}

‎user_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,12 @@ func TestListUser(t *testing.T) {
105105
// Arrange
106106
testRequestOne := autopilot.NewTestRequest(
107107
`query UserList($after:String!$filter:[UsersFilterInput!]$first:Int!){account{users(after: $after, first: $first, filter: $filter){nodes{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},{{ template "pagination_request" }}}}}`,
108-
`{ "after": "", "filter": null, "first": 100 }`,
108+
`{ "after": "", "filter": null, "first": 500 }`,
109109
`{ "data": { "account": { "users": { "nodes": [ {{ template "user_1" }}, {{ template "user_2" }} ], {{ template "pagination_initial_pageInfo_response" }} }}}}`,
110110
)
111111
testRequestTwo := autopilot.NewTestRequest(
112112
`query UserList($after:String!$filter:[UsersFilterInput!]$first:Int!){account{users(after: $after, first: $first, filter: $filter){nodes{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},{{ template "pagination_request" }}}}}`,
113-
`{ "after": "OA", "filter": null, "first": 100 }`,
113+
`{ "after": "OA", "filter": null, "first": 500 }`,
114114
`{ "data": { "account": { "users": { "nodes": [ {{ template "user_3" }} ], {{ template "pagination_second_pageInfo_response" }} }}}}`,
115115
)
116116
requests := []autopilot.TestRequest{testRequestOne, testRequestTwo}
@@ -133,12 +133,12 @@ func TestListUserOmitDeactivated(t *testing.T) {
133133
// Arrange
134134
testRequestOne := autopilot.NewTestRequest(
135135
`query UserList($after:String!$filter:[UsersFilterInput!]$first:Int!){account{users(after: $after, first: $first, filter: $filter){nodes{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},{{ template "pagination_request" }}}}}`,
136-
`{ "after": "", "filter": [ {"key": "deactivated_at", "type": "equals"} ], "first": 100 }`,
136+
`{ "after": "", "filter": [ {"key": "deactivated_at", "type": "equals"} ], "first": 500 }`,
137137
`{ "data": { "account": { "users": { "nodes": [ {{ template "user_1" }}, {{ template "user_2" }} ], {{ template "pagination_initial_pageInfo_response" }} }}}}`,
138138
)
139139
testRequestTwo := autopilot.NewTestRequest(
140140
`query UserList($after:String!$filter:[UsersFilterInput!]$first:Int!){account{users(after: $after, first: $first, filter: $filter){nodes{id,email,name,contacts{address,displayName,displayType,externalId,id,isDefault,type},htmlUrl,provisionedBy,role},{{ template "pagination_request" }}}}}`,
141-
`{ "after": "OA", "filter": [ {"key": "deactivated_at", "type": "equals"} ], "first": 100 }`,
141+
`{ "after": "OA", "filter": [ {"key": "deactivated_at", "type": "equals"} ], "first": 500 }`,
142142
`{ "data": { "account": { "users": { "nodes": [], {{ template "pagination_second_pageInfo_response" }} }}}}`,
143143
)
144144
requests := []autopilot.TestRequest{testRequestOne, testRequestTwo}

0 commit comments

Comments
 (0)