Subject: Re: [JGIT PATCH 3/4] Added test for RefUpdate branch deletion

From: "Shawn O. Pearce" <[email-masked]>
Date: Fri, 22 Aug 2008 16:31:49 -0700

Robin Rosenberg <[email-masked]> wrote:
> lördagen den 23 augusti 2008 00.45.46 skrev Charles O'Farrell:
> 
> Having unit tests is lovely and we appreciate that very much. We need more of this. 
> Unfortunately it breaks. Is the fix the obvious one?
> 
> > +	public void testDeleteHead() throws IOException {
> > +		final RefUpdate ref = updateRef(Constants.HEAD);
> > +		delete(ref, Result.NOT_ATTEMPTED, true, false);
> 
> junit.framework.AssertionFailedError: expected:<NOT_ATTEMPTED> but was:<REJECTED_CURRENT_BRANCH>

Yes.  See my reply.  Its already on repo.or.cz with the necessary
squashed patch.

-- 
Shawn.