diff options
author | Edwin Kempin <ekempin@google.com> | 2016-12-20 10:39:43 +0100 |
---|---|---|
committer | Edwin Kempin <ekempin@google.com> | 2016-12-20 10:53:57 +0100 |
commit | d062a34e8303e18713526fe7afcae1b48c9da451 (patch) | |
tree | 326f5624532c24297709509c3cbbf9994b2794fc | |
parent | 1d0e90957f10ab542c2ac5dfa18a7fd7eac85f7f (diff) |
SuggestReviewersIT: Avoid side-effects between tests
The setUp() method created new accounts before any test. These
accounts had a test method specific prefix, but the total number of
accounts increased the more test methods were executed. This made the
test assertions in suggestReviewersFullTextSearch flaky because it
searched accounts by name parts while it ignored the test method
specific prefix, hence it saw all accounts which were created by
previously executed test methods.
Avoid these side-effects by sandboxing the tests. This means each test
method gets a fresh Gerrit instance.
Also remove the unneeded suggestion limits in
suggestReviewersFullTextSearch as they could potentially obfuscate the
issue.
Change-Id: I84d4f8084e795dc67e30cf82f2922685f284cd29
Signed-off-by: Edwin Kempin <ekempin@google.com>
-rw-r--r-- | gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/change/SuggestReviewersIT.java | 40 |
1 files changed, 25 insertions, 15 deletions
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 e4f56f829e..565d66de34 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 @@ -22,6 +22,7 @@ import com.google.common.collect.Iterables; import com.google.gerrit.acceptance.AbstractDaemonTest; import com.google.gerrit.acceptance.GerritConfig; import com.google.gerrit.acceptance.GerritConfigs; +import com.google.gerrit.acceptance.Sandboxed; import com.google.gerrit.acceptance.TestAccount; import com.google.gerrit.common.data.GlobalCapability; import com.google.gerrit.common.data.GroupDescription; @@ -41,6 +42,7 @@ import org.junit.Test; import java.util.Arrays; import java.util.List; +@Sandboxed public class SuggestReviewersIT extends AbstractDaemonTest { @Inject private CreateGroup.Factory createGroupFactory; @@ -172,49 +174,49 @@ public class SuggestReviewersIT extends AbstractDaemonTest { String changeId = createChange().getChangeId(); List<SuggestedReviewerInfo> reviewers; - reviewers = suggestReviewers(changeId, "first", 4); + reviewers = suggestReviewers(changeId, "first"); assertThat(reviewers).hasSize(3); - reviewers = suggestReviewers(changeId, "first1", 2); + reviewers = suggestReviewers(changeId, "first1"); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, "last", 4); + reviewers = suggestReviewers(changeId, "last"); assertThat(reviewers).hasSize(3); - reviewers = suggestReviewers(changeId, "last1", 2); + reviewers = suggestReviewers(changeId, "last1"); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, "fi la", 4); + reviewers = suggestReviewers(changeId, "fi la"); assertThat(reviewers).hasSize(3); - reviewers = suggestReviewers(changeId, "la fi", 4); + reviewers = suggestReviewers(changeId, "la fi"); assertThat(reviewers).hasSize(3); - reviewers = suggestReviewers(changeId, "first1 la", 2); + reviewers = suggestReviewers(changeId, "first1 la"); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, "fi last1", 2); + reviewers = suggestReviewers(changeId, "fi last1"); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, "first1 last2", 1); + reviewers = suggestReviewers(changeId, "first1 last2"); assertThat(reviewers).hasSize(0); - reviewers = suggestReviewers(changeId, name("user"), 7); + reviewers = suggestReviewers(changeId, name("user")); assertThat(reviewers).hasSize(6); - reviewers = suggestReviewers(changeId, user1.username, 2); + reviewers = suggestReviewers(changeId, user1.username); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, "example.com", 7); + reviewers = suggestReviewers(changeId, "example.com"); assertThat(reviewers).hasSize(6); - reviewers = suggestReviewers(changeId, user1.email, 2); + reviewers = suggestReviewers(changeId, user1.email); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, user1.username + " example", 2); + reviewers = suggestReviewers(changeId, user1.username + " example"); assertThat(reviewers).hasSize(1); - reviewers = suggestReviewers(changeId, user4.email.toLowerCase(), 2); + reviewers = suggestReviewers(changeId, user4.email.toLowerCase()); assertThat(reviewers).hasSize(1); assertThat(reviewers.get(0).account.email).isEqualTo(user4.email); } @@ -270,6 +272,14 @@ public class SuggestReviewersIT extends AbstractDaemonTest { } private List<SuggestedReviewerInfo> suggestReviewers(String changeId, + String query) throws Exception { + return gApi.changes() + .id(changeId) + .suggestReviewers(query) + .get(); + } + + private List<SuggestedReviewerInfo> suggestReviewers(String changeId, String query, int n) throws Exception { return gApi.changes() .id(changeId) |