| // Copyright (C) 2019 The Android Open Source Project |
| // |
| // Licensed under the Apache License, Version 2.0 (the "License"); |
| // you may not use this file except in compliance with the License. |
| // You may obtain a copy of the License at |
| // |
| // http://www.apache.org/licenses/LICENSE-2.0 |
| // |
| // Unless required by applicable law or agreed to in writing, software |
| // distributed under the License is distributed on an "AS IS" BASIS, |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| // See the License for the specific language governing permissions and |
| // limitations under the License. |
| |
| package com.google.gerrit.server.notedb; |
| |
| import static com.google.common.truth.Truth.assertThat; |
| import static org.mockito.ArgumentMatchers.any; |
| import static org.mockito.Mockito.verify; |
| |
| import com.google.gerrit.entities.Change; |
| import com.google.gerrit.entities.HumanComment; |
| import com.google.gerrit.entities.PatchSet; |
| import com.google.gerrit.server.account.AccountState; |
| import com.google.gerrit.server.config.AllUsersName; |
| import com.google.gerrit.server.util.time.TimeUtil; |
| import org.eclipse.jgit.lib.BatchRefUpdate; |
| import org.eclipse.jgit.lib.ObjectId; |
| import org.junit.Test; |
| import org.mockito.ArgumentCaptor; |
| |
| public class DraftCommentNotesTest extends AbstractChangeNotesTest { |
| |
| @Test |
| public void createAndPublishCommentInOneAction_runsDraftOperationAsynchronously() |
| throws Exception { |
| Change c = newChange(); |
| ChangeUpdate update = newUpdate(c, otherUser); |
| update.setPatchSetId(c.currentPatchSetId()); |
| update.putComment(HumanComment.Status.PUBLISHED, comment(c.currentPatchSetId())); |
| update.commit(); |
| |
| assertThat(newNotes(c).getDraftComments(otherUserId)).isEmpty(); |
| assertableFanOutExecutor.assertInteractions(1); |
| } |
| |
| @Test |
| public void createAndPublishComment_runsPublishDraftOperationAsynchronously() throws Exception { |
| Change c = newChange(); |
| ChangeUpdate update = newUpdate(c, otherUser); |
| |
| update.setPatchSetId(c.currentPatchSetId()); |
| update.putComment(HumanComment.Status.DRAFT, comment(c.currentPatchSetId())); |
| update.commit(); |
| assertThat(newNotes(c).getDraftComments(otherUserId)).hasSize(1); |
| assertableFanOutExecutor.assertInteractions(0); |
| |
| update = newUpdate(c, otherUser); |
| update.putComment(HumanComment.Status.PUBLISHED, comment(c.currentPatchSetId())); |
| update.commit(); |
| |
| assertThat(newNotes(c).getDraftComments(otherUserId)).isEmpty(); |
| assertableFanOutExecutor.assertInteractions(1); |
| } |
| |
| @Test |
| public void createAndDeleteDraftComment_runsDraftOperationSynchronously() throws Exception { |
| Change c = newChange(); |
| |
| ChangeUpdate update = newUpdate(c, otherUser); |
| update.setPatchSetId(c.currentPatchSetId()); |
| update.putComment(HumanComment.Status.DRAFT, comment(c.currentPatchSetId())); |
| update.commit(); |
| |
| ChangeNotes notes = newNotes(c); |
| assertThat(notes.getDraftComments(otherUserId)).hasSize(1); |
| assertableFanOutExecutor.assertInteractions(0); |
| |
| update = newUpdate(c, otherUser); |
| update.setPatchSetId(c.currentPatchSetId()); |
| update.deleteComment(comment(c.currentPatchSetId())); |
| update.commit(); |
| |
| notes = newNotes(c); |
| assertThat(notes.getDraftComments(otherUserId)).isEmpty(); |
| assertableFanOutExecutor.assertInteractions(0); |
| } |
| |
| @Test |
| public void createAndPublishCommentInOneAction_firesRefUpdatedDeletion() throws Exception { |
| Change c = newChange(); |
| ChangeUpdate update = newUpdate(c, otherUser); |
| update.setPatchSetId(c.currentPatchSetId()); |
| update.putComment(HumanComment.Status.PUBLISHED, comment(c.currentPatchSetId())); |
| update.commit(); |
| |
| assertThat(newNotes(c).getDraftComments(otherUserId)).isEmpty(); |
| |
| ArgumentCaptor<AccountState> accountStateCaptor = ArgumentCaptor.forClass(AccountState.class); |
| verify(gitReferenceUpdated) |
| .fire(any(AllUsersName.class), any(BatchRefUpdate.class), accountStateCaptor.capture()); |
| |
| assertThat(accountStateCaptor.getValue()).isEqualTo(otherUser.state()); |
| } |
| |
| private HumanComment comment(PatchSet.Id psId) { |
| return newComment( |
| psId, |
| "filename", |
| "uuid", |
| null, |
| 0, |
| otherUser, |
| null, |
| TimeUtil.now(), |
| "comment", |
| (short) 0, |
| ObjectId.fromString("abcd1234abcd1234abcd1234abcd1234abcd1234"), |
| false); |
| } |
| } |