opendev/gerrit
Code Issues Proposed changes

ChangeIT#addReviewer: Remove unnecessary steps

Browse Source 
It's not necessary to reset the clock step or the email sender.

Change-Id: I7beb80edb9a3a7df6c3849ec3f093ab930466164
This commit is contained in:
David Pursehouse
2016-07-14 17:25:48 +09:00
parent a9a885c517
commit fd319fb5d5
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java
View File

@@ -27,7 +27,6 @@ import static com.google.gerrit.server.group.SystemGroupBackend.REGISTERED_USERS
import static com.google.gerrit.server.project.Util.blockLabel;
import static com.google.gerrit.server.project.Util.category;
import static com.google.gerrit.server.project.Util.value;
import static java.util.concurrent.TimeUnit.SECONDS;
import com.google.common.base.Function;
import com.google.common.collect.ImmutableSet;
@@ -533,8 +532,6 @@ public class ChangeIT extends AbstractDaemonTest {
@Test
public void addReviewer() throws Exception {
TestTimeUtil.resetWithClockStep(1, SECONDS);
sender.clear();
PushOneCommit.Result r = createChange();
ChangeResource rsrc = parseResource(r);
String oldETag = rsrc.getETag();