From: Daniel Schwarz Date: Thu, 23 Nov 2017 14:32:45 +0000 (+0100) Subject: SONAR-10119 Set default page size of api/organizations/search to 100 X-Git-Tag: 7.0-RC1~302 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=818a1ed6d1a180adc065b7aac6214255c4e7576b;p=sonarqube.git SONAR-10119 Set default page size of api/organizations/search to 100 --- diff --git a/server/sonar-server/src/main/java/org/sonar/server/organization/ws/SearchAction.java b/server/sonar-server/src/main/java/org/sonar/server/organization/ws/SearchAction.java index 26383c15dec..cdafe498a10 100644 --- a/server/sonar-server/src/main/java/org/sonar/server/organization/ws/SearchAction.java +++ b/server/sonar-server/src/main/java/org/sonar/server/organization/ws/SearchAction.java @@ -67,7 +67,7 @@ public class SearchAction implements OrganizationsWsAction { .setRequired(false) .setSince("6.3"); - action.addPagingParams(25); + action.addPagingParams(100); } @Override diff --git a/server/sonar-server/src/test/java/org/sonar/server/organization/ws/SearchActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/organization/ws/SearchActionTest.java index a638eeb8bce..1ccaad72e78 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/organization/ws/SearchActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/organization/ws/SearchActionTest.java @@ -95,7 +95,7 @@ public class SearchActionTest { assertThat(pParam.description()).isEqualTo("1-based page number"); WebService.Param psParam = action.param("ps"); assertThat(psParam.isRequired()).isFalse(); - assertThat(psParam.defaultValue()).isEqualTo("25"); + assertThat(psParam.defaultValue()).isEqualTo("100"); assertThat(psParam.description()).isEqualTo("Page size. Must be greater than 0."); } @@ -116,7 +116,7 @@ public class SearchActionTest { .setName("Foo Company") .setGuarded(true)); - String response = executeJsonRequest(null, null); + String response = executeJsonRequest(null, 25); assertJson(response).isSimilarTo(wsTester.getDef().responseExampleAsString()); } @@ -224,7 +224,7 @@ public class SearchActionTest { response = executeRequest(null, null); assertThat(response.getOrganizationsList()).extracting(Organization::getKey).hasSize(5); - assertThat(response.getPaging()).extracting(Paging::getPageIndex, Paging::getPageSize, Paging::getTotal).containsOnly(1, 25, 5); + assertThat(response.getPaging()).extracting(Paging::getPageIndex, Paging::getPageSize, Paging::getTotal).containsOnly(1, 100, 5); } @Test