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

From: Shawn O. Pearce <spearce@spearce.org>
Date: Aug 22, 2008, 23:31 UTC
Message-ID: <20080822233149.GR3483@spearce.org>
In-Reply-To: <200808230128.19035.robin.rosenberg.lists@dewire.com>

Robin Rosenberg <robin.rosenberg.lists@dewire.com> wrote:

Show 10 quoted lines

> 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.