From ea265b14db493d68d70d04c829a5a14b01be43f9 Mon Sep 17 00:00:00 2001 From: Edwin Kempin Date: Fri, 17 Jan 2020 15:16:52 +0100 Subject: [PATCH] Test the change message that is posted on setting a patch set description Signed-off-by: Edwin Kempin Change-Id: Ic86c8ada3ee56344667e46f9b12d83e39b594383 --- .../acceptance/api/revision/RevisionIT.java | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/javatests/com/google/gerrit/acceptance/api/revision/RevisionIT.java b/javatests/com/google/gerrit/acceptance/api/revision/RevisionIT.java index c4dec3190f..d29f1cd4d2 100644 --- a/javatests/com/google/gerrit/acceptance/api/revision/RevisionIT.java +++ b/javatests/com/google/gerrit/acceptance/api/revision/RevisionIT.java @@ -1320,10 +1320,24 @@ public class RevisionIT extends AbstractDaemonTest { public void description() throws Exception { PushOneCommit.Result r = createChange(); assertDescription(r, ""); - gApi.changes().id(r.getChangeId()).revision(r.getCommit().name()).description("test"); - assertDescription(r, "test"); + + // set description + gApi.changes().id(r.getChangeId()).revision(r.getCommit().name()).description("foo"); + assertDescription(r, "foo"); + assertThat(Iterables.getLast(gApi.changes().id(r.getChangeId()).get().messages).message) + .isEqualTo("Description set to \"foo\""); + + // update description + gApi.changes().id(r.getChangeId()).revision(r.getCommit().name()).description("bar"); + assertDescription(r, "bar"); + assertThat(Iterables.getLast(gApi.changes().id(r.getChangeId()).get().messages).message) + .isEqualTo("Description changed to \"bar\""); + + // remove description gApi.changes().id(r.getChangeId()).revision(r.getCommit().name()).description(""); assertDescription(r, ""); + assertThat(Iterables.getLast(gApi.changes().id(r.getChangeId()).get().messages).message) + .isEqualTo("Description \"bar\" removed"); } @Test