| // Copyright (C) 2015 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.acceptance.server.project; |
| |
| import static com.google.common.truth.Truth.assertThat; |
| import static com.google.gerrit.server.StarredChangesUtil.IGNORE_LABEL; |
| |
| import com.google.common.collect.ImmutableSet; |
| import com.google.gerrit.acceptance.AbstractDaemonTest; |
| import com.google.gerrit.acceptance.NoHttpd; |
| import com.google.gerrit.acceptance.PushOneCommit; |
| import com.google.gerrit.acceptance.Sandboxed; |
| import com.google.gerrit.acceptance.TestAccount; |
| import com.google.gerrit.common.data.Permission; |
| import com.google.gerrit.extensions.api.changes.ReviewInput; |
| import com.google.gerrit.extensions.api.changes.StarsInput; |
| import com.google.gerrit.extensions.common.GroupInfo; |
| import com.google.gerrit.reviewdb.client.Account; |
| import com.google.gerrit.reviewdb.client.AccountGroup; |
| import com.google.gerrit.reviewdb.client.Project; |
| import com.google.gerrit.server.account.WatchConfig; |
| import com.google.gerrit.server.account.WatchConfig.NotifyType; |
| import com.google.gerrit.server.account.WatchConfig.ProjectWatchKey; |
| import com.google.gerrit.server.git.NotifyConfig; |
| import com.google.gerrit.server.git.ProjectConfig; |
| import com.google.gerrit.server.mail.Address; |
| import com.google.gerrit.testutil.FakeEmailSender.Message; |
| import com.google.inject.Inject; |
| import java.util.EnumSet; |
| import java.util.HashMap; |
| import java.util.List; |
| import java.util.Map; |
| import java.util.Set; |
| import org.eclipse.jgit.internal.storage.dfs.InMemoryRepository; |
| import org.eclipse.jgit.junit.TestRepository; |
| import org.junit.Test; |
| |
| @NoHttpd |
| @Sandboxed |
| public class ProjectWatchIT extends AbstractDaemonTest { |
| @Inject private WatchConfig.Accessor watchConfig; |
| |
| @Test |
| public void newPatchSetsNotifyConfig() throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("new-patch-set"); |
| nc.setHeader(NotifyConfig.Header.CC); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_PATCHSETS)); |
| nc.setFilter("message:sekret"); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("watch", nc); |
| saveProjectConfig(project, cfg); |
| |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "original subject", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| r = |
| pushFactory |
| .create( |
| db, admin.getIdent(), testRepo, "super sekret subject", "a", "a2", r.getChangeId()) |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "back to original subject", "a", "a3") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(addr); |
| assertThat(m.body()).contains("Change subject: super sekret subject\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 2\n"); |
| } |
| |
| @Test |
| public void noNotificationForDraftChangesForWatchersInNotifyConfig() throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("team"); |
| nc.setHeader(NotifyConfig.Header.TO); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_CHANGES, NotifyType.ALL_COMMENTS)); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("team", nc); |
| saveProjectConfig(project, cfg); |
| |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "draft change", "a", "a1") |
| .to("refs/for/master%draft"); |
| r.assertOkStatus(); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| |
| setApiUser(admin); |
| ReviewInput in = new ReviewInput(); |
| in.message = "comment"; |
| gApi.changes().id(r.getChangeId()).current().review(in); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void noNotificationForDraftPatchSetsForWatchersInNotifyConfig() throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("team"); |
| nc.setHeader(NotifyConfig.Header.TO); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_PATCHSETS, NotifyType.ALL_COMMENTS)); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("team", nc); |
| saveProjectConfig(project, cfg); |
| |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "subject", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| sender.clear(); |
| |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "subject", "a", "a2", r.getChangeId()) |
| .to("refs/for/master%draft"); |
| r.assertOkStatus(); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| |
| setApiUser(admin); |
| ReviewInput in = new ReviewInput(); |
| in.message = "comment"; |
| gApi.changes().id(r.getChangeId()).current().review(in); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void noNotificationForPrivateChangesForWatchersInNotifyConfig() throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("team"); |
| nc.setHeader(NotifyConfig.Header.TO); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_CHANGES, NotifyType.ALL_COMMENTS)); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("team", nc); |
| saveProjectConfig(project, cfg); |
| |
| sender.clear(); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "private change", "a", "a1") |
| .to("refs/for/master%private"); |
| r.assertOkStatus(); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| |
| setApiUser(admin); |
| ReviewInput in = new ReviewInput(); |
| in.message = "comment"; |
| gApi.changes().id(r.getChangeId()).current().review(in); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void noNotificationForChangeThatIsTurnedPrivateForWatchersInNotifyConfig() |
| throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("team"); |
| nc.setHeader(NotifyConfig.Header.TO); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_PATCHSETS)); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("team", nc); |
| saveProjectConfig(project, cfg); |
| |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "subject", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| sender.clear(); |
| |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "subject", "a", "a2", r.getChangeId()) |
| .to("refs/for/master%private"); |
| r.assertOkStatus(); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void noNotificationForWipChangesForWatchersInNotifyConfig() throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("team"); |
| nc.setHeader(NotifyConfig.Header.TO); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_CHANGES, NotifyType.ALL_COMMENTS)); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("team", nc); |
| saveProjectConfig(project, cfg); |
| |
| sender.clear(); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "wip change", "a", "a1") |
| .to("refs/for/master%wip"); |
| r.assertOkStatus(); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| |
| setApiUser(admin); |
| ReviewInput in = new ReviewInput(); |
| in.message = "comment"; |
| gApi.changes().id(r.getChangeId()).current().review(in); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void noNotificationForChangeThatIsTurnedWipForWatchersInNotifyConfig() throws Exception { |
| Address addr = new Address("Watcher", "watcher@example.com"); |
| NotifyConfig nc = new NotifyConfig(); |
| nc.addEmail(addr); |
| nc.setName("team"); |
| nc.setHeader(NotifyConfig.Header.TO); |
| nc.setTypes(EnumSet.of(NotifyType.NEW_PATCHSETS)); |
| |
| ProjectConfig cfg = projectCache.checkedGet(project).getConfig(); |
| cfg.putNotifyConfig("team", nc); |
| saveProjectConfig(project, cfg); |
| |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "subject", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| sender.clear(); |
| |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), testRepo, "subject", "a", "a2", r.getChangeId()) |
| .to("refs/for/master%wip"); |
| r.assertOkStatus(); |
| |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void watchProject() throws Exception { |
| // watch project |
| String watchedProject = createProject("watchedProject").get(); |
| setApiUser(user); |
| watch(watchedProject, null); |
| |
| // push a change to watched project -> should trigger email notification |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "TRIGGER", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // push a change to non-watched project -> should not trigger email |
| // notification |
| String notWatchedProject = createProject("otherProject").get(); |
| TestRepository<InMemoryRepository> notWatchedRepo = |
| cloneProject(new Project.NameKey(notWatchedProject), admin); |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), notWatchedRepo, "DONT_TRIGGER", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| } |
| |
| @Test |
| public void watchFile() throws Exception { |
| String watchedProject = createProject("watchedProject").get(); |
| String otherWatchedProject = createProject("otherWatchedProject").get(); |
| setApiUser(user); |
| |
| // watch file in project as user |
| watch(watchedProject, "file:a.txt"); |
| |
| // watch other project as user |
| watch(otherWatchedProject, null); |
| |
| // push a change to watched file -> should trigger email notification for |
| // user |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "TRIGGER", "a.txt", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification for user |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| sender.clear(); |
| |
| // watch project as user2 |
| TestAccount user2 = accounts.create("user2", "user2@test.com", "User2"); |
| setApiUser(user2); |
| watch(watchedProject, null); |
| |
| // push a change to non-watched file -> should not trigger email |
| // notification for user, only for user2 |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "TRIGGER_USER2", "b.txt", "b1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user2.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER_USER2\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| } |
| |
| @Test |
| public void watchKeyword() throws Exception { |
| String watchedProject = createProject("watchedProject").get(); |
| setApiUser(user); |
| |
| // watch keyword in project as user |
| watch(watchedProject, "multimaster"); |
| |
| // push a change with keyword -> should trigger email notification |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "Document multimaster setup", "a.txt", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification for user |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user.emailAddress); |
| assertThat(m.body()).contains("Change subject: Document multimaster setup\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| sender.clear(); |
| |
| // push a change without keyword -> should not trigger email notification |
| r = |
| pushFactory |
| .create( |
| db, admin.getIdent(), watchedRepo, "Cleanup cache implementation", "b.txt", "b1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void watchAllProjects() throws Exception { |
| String anyProject = createProject("anyProject").get(); |
| setApiUser(user); |
| |
| // watch the All-Projects project to watch all projects |
| watch(allProjects.get(), null); |
| |
| // push a change to any project -> should trigger email notification |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> anyRepo = |
| cloneProject(new Project.NameKey(anyProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), anyRepo, "TRIGGER", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| } |
| |
| @Test |
| public void watchFileAllProjects() throws Exception { |
| String anyProject = createProject("anyProject").get(); |
| setApiUser(user); |
| |
| // watch file in All-Projects project as user to watch the file in all |
| // projects |
| watch(allProjects.get(), "file:a.txt"); |
| |
| // push a change to watched file in any project -> should trigger email |
| // notification for user |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> anyRepo = |
| cloneProject(new Project.NameKey(anyProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), anyRepo, "TRIGGER", "a.txt", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification for user |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| sender.clear(); |
| |
| // watch project as user2 |
| TestAccount user2 = accounts.create("user2", "user2@test.com", "User2"); |
| setApiUser(user2); |
| watch(anyProject, null); |
| |
| // push a change to non-watched file in any project -> should not trigger |
| // email notification for user, only for user2 |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), anyRepo, "TRIGGER_USER2", "b.txt", "b1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user2.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER_USER2\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| } |
| |
| @Test |
| public void watchKeywordAllProjects() throws Exception { |
| String anyProject = createProject("anyProject").get(); |
| setApiUser(user); |
| |
| // watch keyword in project as user |
| watch(allProjects.get(), "multimaster"); |
| |
| // push a change with keyword to any project -> should trigger email |
| // notification |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> anyRepo = |
| cloneProject(new Project.NameKey(anyProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), anyRepo, "Document multimaster setup", "a.txt", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification for user |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(user.emailAddress); |
| assertThat(m.body()).contains("Change subject: Document multimaster setup\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| sender.clear(); |
| |
| // push a change without keyword to any project -> should not trigger email |
| // notification |
| r = |
| pushFactory |
| .create(db, admin.getIdent(), anyRepo, "Cleanup cache implementation", "b.txt", "b1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void watchProjectNoNotificationForDraftChange() throws Exception { |
| // watch project |
| String watchedProject = createProject("watchedProject").get(); |
| setApiUser(user); |
| watch(watchedProject, null); |
| |
| // push a draft change to watched project -> should not trigger email notification |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "draft change", "a", "a1") |
| .to("refs/for/master%draft"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void watchProjectNotifyOnDraftChange() throws Exception { |
| String watchedProject = createProject("watchedProject").get(); |
| |
| // create group that can view all drafts |
| GroupInfo groupThatCanViewDrafts = gApi.groups().create("groupThatCanViewDrafts").get(); |
| grant( |
| new Project.NameKey(watchedProject), |
| "refs/*", |
| Permission.VIEW_DRAFTS, |
| false, |
| new AccountGroup.UUID(groupThatCanViewDrafts.id)); |
| |
| // watch project as user that can't view drafts |
| setApiUser(user); |
| watch(watchedProject, null); |
| |
| // watch project as user that can view all drafts |
| TestAccount userThatCanViewDrafts = |
| accounts.create("user2", "user2@test.com", "User2", groupThatCanViewDrafts.name); |
| setApiUser(userThatCanViewDrafts); |
| watch(watchedProject, null); |
| |
| // push a draft change to watched project -> should trigger email notification for |
| // userThatCanViewDrafts, but not for user |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "TRIGGER", "a", "a1") |
| .to("refs/for/master%draft"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(userThatCanViewDrafts.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| } |
| |
| @Test |
| public void watchProjectNoNotificationForIgnoredChange() throws Exception { |
| // watch project |
| String watchedProject = createProject("watchedProject").get(); |
| setApiUser(user); |
| watch(watchedProject, null); |
| |
| // push a change to watched project |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "ignored change", "a", "a1") |
| .to("refs/for/master"); |
| r.assertOkStatus(); |
| |
| // ignore the change |
| setApiUser(user); |
| gApi.accounts().self().setStars(r.getChangeId(), new StarsInput(ImmutableSet.of(IGNORE_LABEL))); |
| |
| sender.clear(); |
| |
| // post a comment -> should not trigger email notification since user ignored the change |
| setApiUser(admin); |
| ReviewInput in = new ReviewInput(); |
| in.message = "comment"; |
| gApi.changes().id(r.getChangeId()).current().review(in); |
| |
| // assert email notification |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void deleteAllProjectWatches() throws Exception { |
| Map<ProjectWatchKey, Set<NotifyType>> watches = new HashMap<>(); |
| watches.put(ProjectWatchKey.create(project, "*"), ImmutableSet.of(NotifyType.ALL)); |
| watchConfig.upsertProjectWatches(admin.getId(), watches); |
| assertThat(watchConfig.getProjectWatches(admin.getId())).isNotEmpty(); |
| |
| watchConfig.deleteAllProjectWatches(admin.getId()); |
| assertThat(watchConfig.getProjectWatches(admin.getId())).isEmpty(); |
| } |
| |
| @Test |
| public void deleteAllProjectWatchesIfWatchConfigIsTheOnlyFileInUserBranch() throws Exception { |
| // Create account that has no files in its refs/users/ branch. |
| Account.Id id = accounts.create().id; |
| |
| // Add a project watch so that a watch.config file in the refs/users/ branch is created. |
| Map<ProjectWatchKey, Set<NotifyType>> watches = new HashMap<>(); |
| watches.put(ProjectWatchKey.create(project, "*"), ImmutableSet.of(NotifyType.ALL)); |
| watchConfig.upsertProjectWatches(id, watches); |
| assertThat(watchConfig.getProjectWatches(id)).isNotEmpty(); |
| |
| // Delete all project watches so that the watch.config file in the refs/users/ branch is |
| // deleted. |
| watchConfig.deleteAllProjectWatches(id); |
| assertThat(watchConfig.getProjectWatches(id)).isEmpty(); |
| } |
| |
| @Test |
| public void watchProjectNoNotificationForPrivateChange() throws Exception { |
| // watch project |
| String watchedProject = createProject("watchedProject").get(); |
| setApiUser(user); |
| watch(watchedProject, null); |
| |
| // push a private change to watched project -> should not trigger email notification |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "private change", "a", "a1") |
| .to("refs/for/master%private"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| assertThat(sender.getMessages()).isEmpty(); |
| } |
| |
| @Test |
| public void watchProjectNotifyOnPrivateChange() throws Exception { |
| String watchedProject = createProject("watchedProject").get(); |
| |
| // create group that can view all private changes |
| GroupInfo groupThatCanViewPrivateChanges = |
| gApi.groups().create("groupThatCanViewPrivateChanges").get(); |
| grant( |
| new Project.NameKey(watchedProject), |
| "refs/*", |
| Permission.VIEW_PRIVATE_CHANGES, |
| false, |
| new AccountGroup.UUID(groupThatCanViewPrivateChanges.id)); |
| |
| // watch project as user that can't view private changes |
| setApiUser(user); |
| watch(watchedProject, null); |
| |
| // watch project as user that can view all private change |
| TestAccount userThatCanViewPrivateChanges = |
| accounts.create("user2", "user2@test.com", "User2", groupThatCanViewPrivateChanges.name); |
| setApiUser(userThatCanViewPrivateChanges); |
| watch(watchedProject, null); |
| |
| // push a private change to watched project -> should trigger email notification for |
| // userThatCanViewPrivateChanges, but not for user |
| setApiUser(admin); |
| TestRepository<InMemoryRepository> watchedRepo = |
| cloneProject(new Project.NameKey(watchedProject), admin); |
| PushOneCommit.Result r = |
| pushFactory |
| .create(db, admin.getIdent(), watchedRepo, "TRIGGER", "a", "a1") |
| .to("refs/for/master%private"); |
| r.assertOkStatus(); |
| |
| // assert email notification |
| List<Message> messages = sender.getMessages(); |
| assertThat(messages).hasSize(1); |
| Message m = messages.get(0); |
| assertThat(m.rcpt()).containsExactly(userThatCanViewPrivateChanges.emailAddress); |
| assertThat(m.body()).contains("Change subject: TRIGGER\n"); |
| assertThat(m.body()).contains("Gerrit-PatchSet: 1\n"); |
| } |
| } |