opendev/gerrit
Code Issues Proposed changes

Merge "EmailIT: Use String#replace method instead String#replaceAll"

Browse Source
This commit is contained in:
David Pursehouse 2017-02-10 11:18:59 +00:00 committed by Gerrit Code Review
commit b44e0e4506
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/rest/account/EmailIT.java
View File

@ -42,7 +42,7 @@ public class EmailIT extends AbstractDaemonTest {
String email = "foo.bar2@example.com";
assertThat(getEmails()).doesNotContain(email);
createEmail(email.replaceAll("@", "%40"));
createEmail(email.replace("@", "%40"));
assertThat(getEmails()).contains(email);
}
@ -67,8 +67,7 @@ public class EmailIT extends AbstractDaemonTest {
createEmail(email);
assertThat(getEmails()).contains(email);
RestResponse r =
adminRestSession.delete("/accounts/self/emails/" + email.replaceAll("@", "%40"));
RestResponse r = adminRestSession.delete("/accounts/self/emails/" + email.replace("@", "%40"));
r.assertNoContent();
assertThat(getEmails()).doesNotContain(email);
}