Re: [PATCH] Add test for filter-branch on a subdirectory that's been added and deleted and re-added
- From
Kevin Ballard <kevin@sb.org>
- Date
- Mar 8, 2008, 18:45 UTC
- Message-ID
- <91DF7A2E-540B-4AD6-91B2-BDB0B04EFC71@sb.org>
- In-Reply-To
- <7vzlt9t5wr.fsf@gitster.siamese.dyndns.org>
On Mar 8, 2008, at 1:41 PM, Junio C Hamano wrote:
Show 12 quoted lines
> Kevin Ballard <kevin@sb.org> writes: > >> Add a currently-failing test for the case when git-filter-branch >> is run with the --subdirectory-filter referencing a folder that >> has been added, deleted, and re-added. Such an action should work, >> but as this test shows it ends up blowing up. > > Such a test is a very good incentive to the people involved in the > failing > part of the system to fix, and is greatly appreciated. However, > > * please mark "currently-failing" test with "test_expect_failure";
Ahh, didn't know about that. In retrospect I should have been able to guess, though.
> * please hint (after --- lines) if you will shortly be following up > with > a fix patch yourself, or if you are only reporting a bug.
Ah. Yeah, I'm only reporting a bug. I don't know anything about how this stuff works, and the bug doesn't bite me, it bit someone else instead, I just thought it deserved to be documented somewhere as a failure.
> I can fix-up and forge your Sign-off.
That would be great, thanks.
Show 31 quoted lines
>> diff --git a/t/t7003-filter-branch.sh b/t/t7003-filter-branch.sh >> index 868babc..933157b 100755 >> --- a/t/t7003-filter-branch.sh >> +++ b/t/t7003-filter-branch.sh >> @@ -179,4 +179,17 @@ test_expect_success 'Name needing quotes' ' >> >> ' >> >> +test_expect_success 'Subdirectory filter on folder with non- >> contiguous history' ' >> + mkdir foo && >> + touch foo/bar && >> + git add foo && >> + git commit -m "Adding foo" && >> + git rm -r foo && >> + git commit -m "Removing foo" && >> + mkdir foo && >> + touch foo/bar && >> + git commit -m "Re-adding foo" && >> + git filter-branch --subdirectory-filter foo >> +' >> + >> test_done >> -- >> 1.5.4.3.487.g5a92 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe git" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >
-- Kevin Ballard http://kevin.sb.org kevin@sb.org http://www.tildesoft.com