From: Shawn O. Pearce <spearce@spearce.org>
Date: Fri, 22 Aug 2008 23:31:49 GMT
Subject: Re: [JGIT PATCH 3/4] Added test for RefUpdate branch deletion
Message-ID: <20080822233149.GR3483@spearce.org>
In-Reply-To: <200808230128.19035.robin.rosenberg.lists@dewire.com>

Robin Rosenberg <robin.rosenberg.lists@dewire.com> 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.