commit fc1aa0fbe518bd9823d69932b5f2aaa7971845e5
author Edwin Kempin <ekempin@google.com> Thu Sep 23 14:19:02 2021 +0200
committer Edwin Kempin <ekempin@google.com> Thu Sep 23 14:23:05 2021 +0200
tree 215e30344c178daa273b4c6606dc1c07787315cb
parent 44b4c88fa4d8f7405e3744a8d3b5be9d88dad28a

CodeOwnersOnAddReviewerIT: Fix test for adding reviewer via PostReview

The changeMessageListsOwnedPathsIfReviewerIsAddedViaPostReview() test is
supposed to verify the change message that is added when a reviewer is
added via the PostReview REST endpoint, but the test added the reviewer
twice, once via PostReview and once via PostReviewers. Drop adding the
reviewer a second time via PostReviewers.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Ia93f88a91aff4c123382be6d3fc4ffae21e6a6c5

javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java
index 595001f..173b74a 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java
@@ -315,7 +315,6 @@
 
     // Add reviewer via PostReview.
     gApi.changes().id(changeId).current().review(ReviewInput.create().reviewer(user.email()));
-    gApi.changes().id(changeId).addReviewer(user.email());
 
     Collection<ChangeMessageInfo> messages = gApi.changes().id(changeId).get().messages;
     assertThat(Iterables.getLast(messages).message)