opendev/gerrit
Code Issues Proposed changes

ChangeEditIT: Always specify charset

Browse Source 
Otherwise ErrorProne complains about it.

Change-Id: I7dc3f841cfcc9d528d0ff4a78329b11898b06bb1
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin 2017-01-27 10:22:00 +01:00
parent dd3b5f2137
commit e5baf1a733
1 changed files with 3 additions and 2 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/edit/ChangeEditIT.java
View File

@ -307,7 +307,7 @@ public class ChangeEditIT extends AbstractDaemonTest {
assertThat(edit).value().baseRevision()
.isEqualTo(currentPatchSet.getRevision().get());
PushOneCommit push = pushFactory.create(db, admin.getIdent(), testRepo,
PushOneCommit.SUBJECT, FILE_NAME, new String(CONTENT_NEW2),
PushOneCommit.SUBJECT, FILE_NAME, new String(CONTENT_NEW2, UTF_8),
changeId2);
push.to("refs/for/master").assertOkStatus();
adminRestSession.post(urlRebase(changeId2)).assertConflict();
@ -627,7 +627,8 @@ public class ChangeEditIT extends AbstractDaemonTest {
public void emptyPutRequest() throws Exception {
createEmptyEditFor(changeId);
adminRestSession.put(urlEditFile(changeId, FILE_NAME)).assertNoContent();
ensureSameBytes(getFileContentOfEdit(changeId, FILE_NAME), "".getBytes());
ensureSameBytes(getFileContentOfEdit(changeId, FILE_NAME),
"".getBytes(UTF_8));
}
@Test