opendev/gerrit
Code Issues Proposed changes

CommentsIT: Use @CanonicalWebUrl from AbstractDaemonTest

Browse Source 
This was added in two separate places in the commit-footers and
multidraft topics, and merged cleanly. Remove the redundancy.

Change-Id: If83e53e46916f3037ade4e14bf3d1ec47c74d434
This commit is contained in:
Dave Borowitz 2015-05-04 12:08:04 -07:00
parent 829c429dd6
commit 8da96f9fd1
1 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/server/change/CommentsIT.java
View File

@ -39,7 +39,6 @@ import com.google.gerrit.server.change.ChangesCollection;
import com.google.gerrit.server.change.PostReview;
import com.google.gerrit.server.change.RevisionResource;
import com.google.gerrit.server.change.Revisions;
import com.google.gerrit.server.config.CanonicalWebUrl;
import com.google.gerrit.server.notedb.NotesMigration;
import com.google.gerrit.testutil.ConfigSuite;
import com.google.gerrit.testutil.FakeEmailSender;
@ -72,10 +71,6 @@ public class CommentsIT extends AbstractDaemonTest {
@Inject
private Provider<PostReview> postReview;
@Inject
@CanonicalWebUrl
private Provider<String> urlProvider;
@Inject
private FakeEmailSender email;
@ -334,7 +329,7 @@ public class CommentsIT extends AbstractDaemonTest {
ImmutableList<Message> messages =
email.getMessages(r2.getChangeId(), "comment");
assertThat(messages).hasSize(1);
String url = urlProvider.get();
String url = canonicalWebUrl.get();
int c = r1.getChange().getId().get();
assertThat(messages.get(0).body()).contains(
"\n"