]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-10119 Set default page size of api/organizations/search to 100
authorDaniel Schwarz <daniel.schwarz@sonarsource.com>
Thu, 23 Nov 2017 14:32:45 +0000 (15:32 +0100)
committerDaniel Schwarz <bartfastiel@users.noreply.github.com>
Thu, 23 Nov 2017 15:53:42 +0000 (16:53 +0100)
server/sonar-server/src/main/java/org/sonar/server/organization/ws/SearchAction.java
server/sonar-server/src/test/java/org/sonar/server/organization/ws/SearchActionTest.java

index 26383c15dec9168d015cb82cb94c5769078335d4..cdafe498a10368da4337f4c3aecabca81b8e9dc0 100644 (file)
@@ -67,7 +67,7 @@ public class SearchAction implements OrganizationsWsAction {
       .setRequired(false)
       .setSince("6.3");
 
-    action.addPagingParams(25);
+    action.addPagingParams(100);
   }
 
   @Override
index a638eeb8bce9674b485ed043ea3ae8a4f21268fa..1ccaad72e789ada89d4be2259a138e637236fe3f 100644 (file)
@@ -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