Eclipse Community Forums: EGit » JGit RevWalk.isMergedInto()

Help

Home

Home » Eclipse Projects » EGit » JGit RevWalk.isMergedInto()

Show: Today's Messages :: Show Polls :: Message Navigator

JGit RevWalk.isMergedInto() [message #1058575]

Tue, 14 May 2013 09:42

Rüdiger Herrmann
Messages: 551
Registered: July 2009

Senior Member

Hi,

I am struggling with RevWalk.isMergedInto(). The two tests below
(should) essentially create the same repository structure:
1. a common ancestor
2. two independent commits (master and topic) with 1. as parent
3. a merge commit of 'master' and 'topic'
4. an 'extra' commit with the merge commit as parent

With this setup, isMergedInto(master, extra) should return true. But for
some reason which I can't figure out, only the first test returns true.

Can anyone point me to what's wrong with the second test?

Both tests require to be members of a class that inherits from
RevWalkTestCase. You may just paste them into the RevWalkMergeBaseTest.

@Test
public void testIsMergedInto1() throws Exception {
RevCommit common = commit();
RevCommit master = commit(common);
RevCommit topic = commit(common);
RevCommit merge = commit(master, topic);
RevCommit extra = commit(merge);

boolean merged = rw.isMergedInto(master, extra);

assertTrue(merged);
}

@Test
public void testIsMergedInto2() throws Exception {
Git git = new Git(db);
git.commit().setMessage("common ancestor").call();
git.branchCreate().setName("topic").call();
RevCommit master = git.commit().setMessage("master commit").call();
git.checkout().setName("topic").call();
git.commit().setMessage("topic commit").call();
git.merge().include(db.getRef("master")).call();
RevCommit extra = git.commit().setMessage("extra commit").call();

boolean merged = rw.isMergedInto(master, extra);

assertTrue(merged);
}

I suspect I am missing something very obvious...
TIA
Rüdiger

Report message to a moderator

Re: JGit RevWalk.isMergedInto() [message #1060232 is a reply to message #1058575]

Thu, 23 May 2013 11:58

Stefan Lay
Messages: 334
Registered: July 2009

Senior Member

I don't yet know why the second test fails, but I can make it work if I change the assert to

boolean merged = rw.isMergedInto(rw.lookupCommit(master),
rw.lookupCommit(extra));

Somehow the revwalk does not correctly walk the topology.

Report message to a moderator

Re: JGit RevWalk.isMergedInto() [message #1060437 is a reply to message #1060232]

Fri, 24 May 2013 11:52

Robin Rosenberg
Messages: 268
Registered: July 2009

Senior Member

Stefan Lay skrev 2013-05-23 17.58:
> I don't yet know why the second test fails, but I can make it work if I change the assert to
> boolean merged = rw.isMergedInto(rw.lookupCommit(master),
> rw.lookupCommit(extra));
>
> Somehow the revwalk does not correctly walk the topology.

I think it is this: Here, and in general, when a method accepts a RevObject of some kind,
(and possibly a RevWalk) it means all objects must be allocated by the same RevWalk instance.
The javadoc should mention this.

-- robin

Report message to a moderator

Re: JGit RevWalk.isMergedInto() [message #1060602 is a reply to message #1060437]

Mon, 27 May 2013 05:27

Stefan Lay
Messages: 334
Registered: July 2009

Senior Member

> I think it is this: Here, and in general, when a method accepts a RevObject of some > kind,
> (and possibly a RevWalk) it means all objects must be allocated by the same RevWalk > instance.
> The javadoc should mention this.

I agree. We should also mention this in the JavaDoc of the Git API methods returning a RevObject.

Stefan

Report message to a moderator

Previous Topic:	Help: My Push was rejected
Next Topic:	http.sslcainfo config option

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Tue May 28 03:07:49 EDT 2013