opendev/gerrit
Code Issues Proposed changes

InlineEdit: Add DELETE /changes/<id>/edit/path%2fto%2ffile REST endpoint

Browse Source 
Allow to delete file from change edit.

Change-Id: I62ea1f3df02a2872ed51456f44fa961dfe518b37
This commit is contained in:
David Ostrovsky
2014-08-01 02:24:31 +02:00
parent 138edb474a
commit 2830c29fab
4 changed files with 70 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/edit/ChangeEditIT.java
View File

@@ -262,6 +262,22 @@ public class ChangeEditIT extends AbstractDaemonTest {
}
}
@Test
public void deleteExistingFileRest() throws Exception {
assertEquals(RefUpdate.Result.NEW,
modifier.createEdit(
change,
ps));
assertEquals(SC_NO_CONTENT, session.delete(urlEditFile()).getStatusCode());
Optional<ChangeEdit> edit = editUtil.byChange(change);
try {
fileUtil.getContent(edit.get().getChange().getProject(),
edit.get().getRevision().get(), FILE_NAME);
fail("ResourceNotFoundException expected");
} catch (ResourceNotFoundException rnfe) {
}
}
@Test
public void restoreDeletedFileInEdit() throws Exception {
assertEquals(RefUpdate.Result.NEW,