diff --git a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SuggestReviewersIT.java b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SuggestReviewersIT.java index 8a277ca337..a37e484558 100644 --- a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SuggestReviewersIT.java +++ b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SuggestReviewersIT.java @@ -191,6 +191,21 @@ public class SuggestReviewersIT extends AbstractDaemonTest { assertEquals(2, reviewers.size()); } + @Test + public void suggestReviewersWithoutLimitOptionSpecified() throws Exception { + String changeId = createChange().getChangeId(); + String query = "users3"; + List suggestedReviewerInfos = newGson().fromJson( + adminSession.get("/changes/" + + changeId + + "/suggest_reviewers?q=" + + query) + .getReader(), + new TypeToken>() {} + .getType()); + assertEquals(1, suggestedReviewerInfos.size()); + } + private List suggestReviewers(RestSession session, String changeId, String query, int n) throws IOException { return newGson().fromJson( diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/SuggestReviewers.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/SuggestReviewers.java index f0f63eb2a7..ff54e09fb8 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/change/SuggestReviewers.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/SuggestReviewers.java @@ -115,6 +115,7 @@ public class SuggestReviewers implements RestReadView { this.reviewerSuggestionCache = reviewerSuggestionCache; this.maxSuggestedReviewers = cfg.getInt("suggest", "maxSuggestedReviewers", DEFAULT_MAX_SUGGESTED); + this.limit = this.maxSuggestedReviewers; this.fullTextMaxMatches = cfg.getInt("suggest", "fullTextSearchMaxMatches", DEFAULT_MAX_MATCHES);