[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Issue triage: issues 3429 and 3474

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 17 Nov 2010 20:14:16 +0200

Johan Corveleyn wrote on Wed, Nov 17, 2010 at 01:25:24 +0100:
> On Mon, Nov 8, 2010 at 1:02 PM, Philip Martin
> <philip.martin_at_wandisco.com> wrote:
> > Johan Corveleyn <jcorvel_at_gmail.com> writes:
> >
> >> Or, maybe the best approach: I could add a regression test for these
> >> issues, so we can all be sure that they are fixed (and remain fixed),
> >> after which they can be marked as fixed.
> >
> > Yes, please.  Are there any existing XFAIL tests that apply?  They
> > sometimes don't XPASS automatically when the bug is fixed because the
> > test expectation is wrong.
>
> Doh, it seems that issue #3474 already has a test, which PASSes (added
> by Bert, which he mentioned in a comment in the issue):
>
> PASS: copy_tests.py 81: copy of new dir with copied file keeps history
>
> This is exactly what issue #3474 is about. Bert added the test as
> XFAIL in r938071, and it was marked PASS by you, Philip, in r955334.
>
> So, I guess this wraps up that issue: can someone mark it as resolved?
>

Could you mark it as resolved? Or don't you have the necessary
permissions on tigris?

> There was a slight confusion when I read the test code, because a
> comment still talks about the tests as "Currently this fails because
> ...". The following patch removes that obsolete comment:
>
> [[[
> Index: subversion/tests/cmdline/copy_tests.py
> ===================================================================
> --- subversion/tests/cmdline/copy_tests.py (revision 1035851)
> +++ subversion/tests/cmdline/copy_tests.py (working copy)
> @@ -4358,8 +4358,6 @@ def copy_added_dir_with_copy(sbox):
> 'NewDir2/mu' : Item(status='A ', copied='+', wc_rev='-'),
> })
>
> - # Currently this fails because NewDir2/mu loses its history in the copy
> - # from NewDir to NewDir2
> svntest.actions.run_and_verify_status(wc_dir, expected_status)
>
>

+1, but could you add a link to the issue (in a comment) while you're
there?

>
> ]]]
>
>
> For issue #3429, I'll try to write a regression test myself, and post
> a patch for it in a new thread.
>
> Cheers,
> --
> Johan
Received on 2010-11-17 19:17:36 CET

This is an archived mail posted to the Subversion Dev mailing list.