Merge branch 'stable-3.1'

* stable-3.1:
  Prepare post 3.1.0 builds
  JGit v3.1.0.201310021548-r
  Fix order of commits in rebase todo file header
  Prepare post 3.1.0 RC1 builds
  JGit v3.1.0.201309270735-rc1
  Attempt to fix graph layout when new heads are introduced
  Prepare re-signing pgm's ueberjar to avoid SecurityException
  Use full branch name when getting ref in BranchTrackingStatus

Change-Id: Ida30175d0c058048cbe4b563aae7f85cd008db29
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

1 files changed, 41 insertions, 10 deletions

diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/revplot/PlotCommitListTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/revplot/PlotCommitListTest.java
index 5db6b86f81..926424c06d 100644
--- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/revplot/PlotCommitListTest.java
+++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/revplot/PlotCommitListTest.java
@@ -156,8 +156,8 @@ public class PlotCommitListTest extends RevWalkTestCase {
 
 		CommitListAssert test = new CommitListAssert(pcl);
 		test.commit(c).lanePos(0).parents(a);
-		test.commit(b).lanePos(1).parents(a);
-		test.commit(a).lanePos(0).parents();
+		test.commit(b).lanePos(0).parents(a);
+		test.commit(a).lanePos(1).parents();
 		test.noMoreCommits();
 	}
 
@@ -179,9 +179,9 @@ public class PlotCommitListTest extends RevWalkTestCase {
 
 		CommitListAssert test = new CommitListAssert(pcl);
 		test.commit(d).lanePos(0).parents(a);
-		test.commit(c).lanePos(1).parents(a);
-		test.commit(b).lanePos(1).parents(a);
-		test.commit(a).lanePos(0).parents();
+		test.commit(c).lanePos(0).parents(a);
+		test.commit(b).lanePos(0).parents(a);
+		test.commit(a).lanePos(1).parents();
 		test.noMoreCommits();
 	}
 
@@ -214,14 +214,45 @@ public class PlotCommitListTest extends RevWalkTestCase {
 		CommitListAssert test = new CommitListAssert(pcl);
 		test.commit(g).lanePos(0).parents(f);
 		test.commit(f).lanePos(0).parents(a);
-		test.commit(e).lanePos(1).parents(a);
-		test.commit(d).lanePos(1).parents(a);
-		test.commit(c).lanePos(1).parents(a);
-		test.commit(b).lanePos(1).parents(a);
-		test.commit(a).lanePos(0).parents();
+		test.commit(e).lanePos(0).parents(a);
+		test.commit(d).lanePos(0).parents(a);
+		test.commit(c).lanePos(0).parents(a);
+		test.commit(b).lanePos(0).parents(a);
+		test.commit(a).lanePos(1).parents();
 		test.noMoreCommits();
 	}
 
+	@Test
+	public void testBug368927() throws Exception {
+		final RevCommit a = commit();
+		final RevCommit b = commit(a);
+		final RevCommit c = commit(b);
+		final RevCommit d = commit(b);
+		final RevCommit e = commit(c);
+		final RevCommit f = commit(e, d);
+		final RevCommit g = commit(a);
+		final RevCommit h = commit(f);
+		final RevCommit i = commit(h);
+
+		PlotWalk pw = new PlotWalk(db);
+		pw.markStart(pw.lookupCommit(i.getId()));
+		pw.markStart(pw.lookupCommit(g.getId()));
+
+		PlotCommitList<PlotLane> pcl = new PlotCommitList<PlotLane>();
+		pcl.source(pw);
+		pcl.fillTo(Integer.MAX_VALUE);
+		CommitListAssert test = new CommitListAssert(pcl);
+		test.commit(i).lanePos(1).parents(h);
+		test.commit(h).lanePos(1).parents(f);
+		test.commit(g).lanePos(0).parents(a);
+		test.commit(f).lanePos(1).parents(e, d);
+		test.commit(e).lanePos(0).parents(c);
+		test.commit(d).lanePos(1).parents(b);
+		test.commit(c).lanePos(0).parents(b);
+		test.commit(b).lanePos(1).parents(a);
+		test.commit(a).lanePos(2).parents();
+	}
+
 	// test the history of the egit project between 9fdaf3c1 and e76ad9170f
 	@Test
 	public void testEgitHistory() throws Exception {