You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

RevertCommandTest.java 15KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432
  1. /*
  2. * Copyright (C) 2011, Robin Rosenberg
  3. * and other copyright owners as documented in the project's IP log.
  4. *
  5. * This program and the accompanying materials are made available
  6. * under the terms of the Eclipse Distribution License v1.0 which
  7. * accompanies this distribution, is reproduced below, and is
  8. * available at http://www.eclipse.org/org/documents/edl-v10.php
  9. *
  10. * All rights reserved.
  11. *
  12. * Redistribution and use in source and binary forms, with or
  13. * without modification, are permitted provided that the following
  14. * conditions are met:
  15. *
  16. * - Redistributions of source code must retain the above copyright
  17. * notice, this list of conditions and the following disclaimer.
  18. *
  19. * - Redistributions in binary form must reproduce the above
  20. * copyright notice, this list of conditions and the following
  21. * disclaimer in the documentation and/or other materials provided
  22. * with the distribution.
  23. *
  24. * - Neither the name of the Eclipse Foundation, Inc. nor the
  25. * names of its contributors may be used to endorse or promote
  26. * products derived from this software without specific prior
  27. * written permission.
  28. *
  29. * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
  30. * CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
  31. * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
  32. * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  33. * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
  34. * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  35. * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
  36. * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
  37. * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
  38. * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
  39. * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  40. * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  41. * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  42. */
  43. package org.eclipse.jgit.api;
  44. import static org.junit.Assert.assertEquals;
  45. import static org.junit.Assert.assertFalse;
  46. import static org.junit.Assert.assertNotNull;
  47. import static org.junit.Assert.assertNull;
  48. import static org.junit.Assert.assertTrue;
  49. import java.io.File;
  50. import java.io.IOException;
  51. import java.util.Iterator;
  52. import org.eclipse.jgit.api.MergeResult.MergeStatus;
  53. import org.eclipse.jgit.api.ResetCommand.ResetType;
  54. import org.eclipse.jgit.api.errors.GitAPIException;
  55. import org.eclipse.jgit.api.errors.JGitInternalException;
  56. import org.eclipse.jgit.dircache.DirCache;
  57. import org.eclipse.jgit.junit.RepositoryTestCase;
  58. import org.eclipse.jgit.lib.ConfigConstants;
  59. import org.eclipse.jgit.lib.Constants;
  60. import org.eclipse.jgit.lib.FileMode;
  61. import org.eclipse.jgit.lib.ReflogReader;
  62. import org.eclipse.jgit.lib.RepositoryState;
  63. import org.eclipse.jgit.merge.ResolveMerger.MergeFailureReason;
  64. import org.eclipse.jgit.revwalk.RevCommit;
  65. import org.junit.Test;
  66. /**
  67. * Test revert command
  68. */
  69. public class RevertCommandTest extends RepositoryTestCase {
  70. @Test
  71. public void testRevert() throws IOException, JGitInternalException,
  72. GitAPIException {
  73. try (Git git = new Git(db)) {
  74. writeTrashFile("a", "first line\nsec. line\nthird line\n");
  75. git.add().addFilepattern("a").call();
  76. git.commit().setMessage("create a").call();
  77. writeTrashFile("b", "content\n");
  78. git.add().addFilepattern("b").call();
  79. git.commit().setMessage("create b").call();
  80. writeTrashFile("a", "first line\nsec. line\nthird line\nfourth line\n");
  81. git.add().addFilepattern("a").call();
  82. git.commit().setMessage("enlarged a").call();
  83. writeTrashFile("a",
  84. "first line\nsecond line\nthird line\nfourth line\n");
  85. git.add().addFilepattern("a").call();
  86. RevCommit fixingA = git.commit().setMessage("fixed a").call();
  87. writeTrashFile("b", "first line\n");
  88. git.add().addFilepattern("b").call();
  89. git.commit().setMessage("fixed b").call();
  90. git.revert().include(fixingA).call();
  91. assertEquals(RepositoryState.SAFE, db.getRepositoryState());
  92. assertTrue(new File(db.getWorkTree(), "b").exists());
  93. checkFile(new File(db.getWorkTree(), "a"),
  94. "first line\nsec. line\nthird line\nfourth line\n");
  95. Iterator<RevCommit> history = git.log().call().iterator();
  96. RevCommit revertCommit = history.next();
  97. String expectedMessage = "Revert \"fixed a\"\n\n"
  98. + "This reverts commit " + fixingA.getId().getName() + ".\n";
  99. assertEquals(expectedMessage, revertCommit.getFullMessage());
  100. assertEquals("fixed b", history.next().getFullMessage());
  101. assertEquals("fixed a", history.next().getFullMessage());
  102. assertEquals("enlarged a", history.next().getFullMessage());
  103. assertEquals("create b", history.next().getFullMessage());
  104. assertEquals("create a", history.next().getFullMessage());
  105. assertFalse(history.hasNext());
  106. ReflogReader reader = db.getReflogReader(Constants.HEAD);
  107. assertTrue(reader.getLastEntry().getComment()
  108. .startsWith("revert: Revert \""));
  109. reader = db.getReflogReader(db.getBranch());
  110. assertTrue(reader.getLastEntry().getComment()
  111. .startsWith("revert: Revert \""));
  112. }
  113. }
  114. @Test
  115. public void testRevertMultiple() throws IOException, JGitInternalException,
  116. GitAPIException {
  117. try (Git git = new Git(db)) {
  118. writeTrashFile("a", "first\n");
  119. git.add().addFilepattern("a").call();
  120. git.commit().setMessage("add first").call();
  121. writeTrashFile("a", "first\nsecond\n");
  122. git.add().addFilepattern("a").call();
  123. RevCommit secondCommit = git.commit().setMessage("add second").call();
  124. writeTrashFile("a", "first\nsecond\nthird\n");
  125. git.add().addFilepattern("a").call();
  126. RevCommit thirdCommit = git.commit().setMessage("add third").call();
  127. git.revert().include(thirdCommit).include(secondCommit).call();
  128. assertEquals(RepositoryState.SAFE, db.getRepositoryState());
  129. checkFile(new File(db.getWorkTree(), "a"), "first\n");
  130. Iterator<RevCommit> history = git.log().call().iterator();
  131. RevCommit revertCommit = history.next();
  132. String expectedMessage = "Revert \"add second\"\n\n"
  133. + "This reverts commit "
  134. + secondCommit.getId().getName() + ".\n";
  135. assertEquals(expectedMessage, revertCommit.getFullMessage());
  136. revertCommit = history.next();
  137. expectedMessage = "Revert \"add third\"\n\n"
  138. + "This reverts commit " + thirdCommit.getId().getName()
  139. + ".\n";
  140. assertEquals(expectedMessage, revertCommit.getFullMessage());
  141. assertEquals("add third", history.next().getFullMessage());
  142. assertEquals("add second", history.next().getFullMessage());
  143. assertEquals("add first", history.next().getFullMessage());
  144. assertFalse(history.hasNext());
  145. ReflogReader reader = db.getReflogReader(Constants.HEAD);
  146. assertTrue(reader.getLastEntry().getComment()
  147. .startsWith("revert: Revert \""));
  148. reader = db.getReflogReader(db.getBranch());
  149. assertTrue(reader.getLastEntry().getComment()
  150. .startsWith("revert: Revert \""));
  151. }
  152. }
  153. @Test
  154. public void testRevertMultipleWithFail() throws IOException,
  155. JGitInternalException, GitAPIException {
  156. try (Git git = new Git(db)) {
  157. writeTrashFile("a", "first\n");
  158. git.add().addFilepattern("a").call();
  159. git.commit().setMessage("add first").call();
  160. writeTrashFile("a", "first\nsecond\n");
  161. git.add().addFilepattern("a").call();
  162. RevCommit secondCommit = git.commit().setMessage("add second").call();
  163. writeTrashFile("a", "first\nsecond\nthird\n");
  164. git.add().addFilepattern("a").call();
  165. git.commit().setMessage("add third").call();
  166. writeTrashFile("a", "first\nsecond\nthird\nfourth\n");
  167. git.add().addFilepattern("a").call();
  168. RevCommit fourthCommit = git.commit().setMessage("add fourth").call();
  169. git.revert().include(fourthCommit).include(secondCommit).call();
  170. // not SAFE because it failed
  171. assertEquals(RepositoryState.REVERTING, db.getRepositoryState());
  172. checkFile(new File(db.getWorkTree(), "a"), "first\n"
  173. + "<<<<<<< master\n" + "second\n" + "third\n" + "=======\n"
  174. + ">>>>>>> " + secondCommit.getId().abbreviate(7).name()
  175. + " add second\n");
  176. Iterator<RevCommit> history = git.log().call().iterator();
  177. RevCommit revertCommit = history.next();
  178. String expectedMessage = "Revert \"add fourth\"\n\n"
  179. + "This reverts commit " + fourthCommit.getId().getName()
  180. + ".\n";
  181. assertEquals(expectedMessage, revertCommit.getFullMessage());
  182. assertEquals("add fourth", history.next().getFullMessage());
  183. assertEquals("add third", history.next().getFullMessage());
  184. assertEquals("add second", history.next().getFullMessage());
  185. assertEquals("add first", history.next().getFullMessage());
  186. assertFalse(history.hasNext());
  187. ReflogReader reader = db.getReflogReader(Constants.HEAD);
  188. assertTrue(reader.getLastEntry().getComment()
  189. .startsWith("revert: Revert \""));
  190. reader = db.getReflogReader(db.getBranch());
  191. assertTrue(reader.getLastEntry().getComment()
  192. .startsWith("revert: Revert \""));
  193. }
  194. }
  195. @Test
  196. public void testRevertDirtyIndex() throws Exception {
  197. try (Git git = new Git(db)) {
  198. RevCommit sideCommit = prepareRevert(git);
  199. // modify and add file a
  200. writeTrashFile("a", "a(modified)");
  201. git.add().addFilepattern("a").call();
  202. // do not commit
  203. doRevertAndCheckResult(git, sideCommit,
  204. MergeFailureReason.DIRTY_INDEX);
  205. }
  206. }
  207. @Test
  208. public void testRevertDirtyWorktree() throws Exception {
  209. try (Git git = new Git(db)) {
  210. RevCommit sideCommit = prepareRevert(git);
  211. // modify file a
  212. writeTrashFile("a", "a(modified)");
  213. // do not add and commit
  214. doRevertAndCheckResult(git, sideCommit,
  215. MergeFailureReason.DIRTY_WORKTREE);
  216. }
  217. }
  218. @Test
  219. public void testRevertConflictResolution() throws Exception {
  220. try (Git git = new Git(db)) {
  221. RevCommit sideCommit = prepareRevert(git);
  222. RevertCommand revert = git.revert();
  223. RevCommit newHead = revert.include(sideCommit.getId()).call();
  224. assertNull(newHead);
  225. MergeResult result = revert.getFailingResult();
  226. assertEquals(MergeStatus.CONFLICTING, result.getMergeStatus());
  227. assertTrue(new File(db.getDirectory(), Constants.MERGE_MSG).exists());
  228. assertEquals("Revert \"" + sideCommit.getShortMessage()
  229. + "\"\n\nThis reverts commit " + sideCommit.getId().getName()
  230. + ".\n\nConflicts:\n\ta\n",
  231. db.readMergeCommitMsg());
  232. assertTrue(new File(db.getDirectory(), Constants.REVERT_HEAD)
  233. .exists());
  234. assertEquals(sideCommit.getId(), db.readRevertHead());
  235. assertEquals(RepositoryState.REVERTING, db.getRepositoryState());
  236. // Resolve
  237. writeTrashFile("a", "a");
  238. git.add().addFilepattern("a").call();
  239. assertEquals(RepositoryState.REVERTING_RESOLVED,
  240. db.getRepositoryState());
  241. git.commit().setOnly("a").setMessage("resolve").call();
  242. assertEquals(RepositoryState.SAFE, db.getRepositoryState());
  243. }
  244. }
  245. @Test
  246. public void testRevertkConflictReset() throws Exception {
  247. try (Git git = new Git(db)) {
  248. RevCommit sideCommit = prepareRevert(git);
  249. RevertCommand revert = git.revert();
  250. RevCommit newHead = revert.include(sideCommit.getId()).call();
  251. assertNull(newHead);
  252. MergeResult result = revert.getFailingResult();
  253. assertEquals(MergeStatus.CONFLICTING, result.getMergeStatus());
  254. assertEquals(RepositoryState.REVERTING, db.getRepositoryState());
  255. assertTrue(new File(db.getDirectory(), Constants.REVERT_HEAD)
  256. .exists());
  257. git.reset().setMode(ResetType.MIXED).setRef("HEAD").call();
  258. assertEquals(RepositoryState.SAFE, db.getRepositoryState());
  259. assertFalse(new File(db.getDirectory(), Constants.REVERT_HEAD)
  260. .exists());
  261. }
  262. }
  263. @Test
  264. public void testRevertOverExecutableChangeOnNonExectuableFileSystem()
  265. throws Exception {
  266. try (Git git = new Git(db)) {
  267. File file = writeTrashFile("test.txt", "a");
  268. assertNotNull(git.add().addFilepattern("test.txt").call());
  269. assertNotNull(git.commit().setMessage("commit1").call());
  270. assertNotNull(git.checkout().setCreateBranch(true).setName("a").call());
  271. writeTrashFile("test.txt", "b");
  272. assertNotNull(git.add().addFilepattern("test.txt").call());
  273. RevCommit commit2 = git.commit().setMessage("commit2").call();
  274. assertNotNull(commit2);
  275. assertNotNull(git.checkout().setName(Constants.MASTER).call());
  276. DirCache cache = db.lockDirCache();
  277. cache.getEntry("test.txt").setFileMode(FileMode.EXECUTABLE_FILE);
  278. cache.write();
  279. assertTrue(cache.commit());
  280. cache.unlock();
  281. assertNotNull(git.commit().setMessage("commit3").call());
  282. db.getFS().setExecute(file, false);
  283. git.getRepository()
  284. .getConfig()
  285. .setBoolean(ConfigConstants.CONFIG_CORE_SECTION, null,
  286. ConfigConstants.CONFIG_KEY_FILEMODE, false);
  287. RevertCommand revert = git.revert();
  288. RevCommit newHead = revert.include(commit2).call();
  289. assertNotNull(newHead);
  290. }
  291. }
  292. @Test
  293. public void testRevertConflictMarkers() throws Exception {
  294. try (Git git = new Git(db)) {
  295. RevCommit sideCommit = prepareRevert(git);
  296. RevertCommand revert = git.revert();
  297. RevCommit newHead = revert.include(sideCommit.getId())
  298. .call();
  299. assertNull(newHead);
  300. MergeResult result = revert.getFailingResult();
  301. assertEquals(MergeStatus.CONFLICTING, result.getMergeStatus());
  302. String expected = "<<<<<<< master\na(latest)\n=======\na\n>>>>>>> ca96c31 second master\n";
  303. checkFile(new File(db.getWorkTree(), "a"), expected);
  304. }
  305. }
  306. @Test
  307. public void testRevertOurCommitName() throws Exception {
  308. try (Git git = new Git(db)) {
  309. RevCommit sideCommit = prepareRevert(git);
  310. RevertCommand revert = git.revert();
  311. RevCommit newHead = revert.include(sideCommit.getId())
  312. .setOurCommitName("custom name").call();
  313. assertNull(newHead);
  314. MergeResult result = revert.getFailingResult();
  315. assertEquals(MergeStatus.CONFLICTING, result.getMergeStatus());
  316. String expected = "<<<<<<< custom name\na(latest)\n=======\na\n>>>>>>> ca96c31 second master\n";
  317. checkFile(new File(db.getWorkTree(), "a"), expected);
  318. }
  319. }
  320. private RevCommit prepareRevert(final Git git) throws Exception {
  321. // create, add and commit file a
  322. writeTrashFile("a", "a");
  323. git.add().addFilepattern("a").call();
  324. git.commit().setMessage("first master").call();
  325. // First commit
  326. checkoutBranch("refs/heads/master");
  327. // modify, add and commit file a
  328. writeTrashFile("a", "a(previous)");
  329. git.add().addFilepattern("a").call();
  330. RevCommit oldCommit = git.commit().setMessage("second master").call();
  331. // modify, add and commit file a
  332. writeTrashFile("a", "a(latest)");
  333. git.add().addFilepattern("a").call();
  334. git.commit().setMessage("side").call();
  335. return oldCommit;
  336. }
  337. private void doRevertAndCheckResult(final Git git,
  338. final RevCommit sideCommit, final MergeFailureReason reason)
  339. throws Exception {
  340. // get current index state
  341. String indexState = indexState(CONTENT);
  342. // revert
  343. RevertCommand revert = git.revert();
  344. RevCommit resultCommit = revert.include(sideCommit.getId()).call();
  345. assertNull(resultCommit);
  346. MergeResult result = revert.getFailingResult();
  347. assertEquals(MergeStatus.FAILED, result.getMergeStatus());
  348. // staged file a causes DIRTY_INDEX
  349. assertEquals(1, result.getFailingPaths().size());
  350. assertEquals(reason, result.getFailingPaths().get("a"));
  351. assertEquals("a(modified)", read(new File(db.getWorkTree(), "a")));
  352. // index shall be unchanged
  353. assertEquals(indexState, indexState(CONTENT));
  354. assertEquals(RepositoryState.SAFE, db.getRepositoryState());
  355. if (reason == null) {
  356. ReflogReader reader = db.getReflogReader(Constants.HEAD);
  357. assertTrue(reader.getLastEntry().getComment()
  358. .startsWith("revert: "));
  359. reader = db.getReflogReader(db.getBranch());
  360. assertTrue(reader.getLastEntry().getComment()
  361. .startsWith("revert: "));
  362. }
  363. }
  364. }