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

Re: svn commit: r981905 - /subversion/trunk/subversion/tests/cmdline/depth_tests.py

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Wed, 04 Aug 2010 12:25:01 +0100

Philip Martin wrote:
> Greg Stein <gstein_at_gmail.com> writes:
> > On Tue, Aug 3, 2010 at 13:41, Julian Foad <julian.foad_at_wandisco.com> wrote:
> >> Greg Stein wrote:
> >>> Why would this be true only in single-db? Shouldn't the test perform
> >>> exactly the same in both cases? I see no rationale for WHY these
> >>> should differ.
> >>>
> >>> > + if svntest.main.wc_is_singledb(sbox.wc_dir):
> >>> > + expected_disk.remove('A/B/E', 'A/B/F')
> >>
> >> It's just because in single-db mode, deleted dirs really go away from
> >> disk, isn't it?
>
> Yes. I thought this was a well known feature of wc-ng single-db.
>
> > Beats me. Where is the comment to that effect? Where is the
> > explanation in the log message?
> >
> > Why are these tests allowing *variant* outputs without any explanation?
>
> I'm not sure how much explanation of wc-ng, single-db you expect in
> the log message. Generally I try to avoid having the log message
> simply repeat the code change. The code change already shows that
> it's removing a directory from the expected_disk, I suppose the log
> message could mention that it's a delete operation?

The bit of explanation that was missing is simply the reason why it
expects a different result: "To account for the fact that in single-db
mode directories scheduled for deletion are not present on disk."

> Or do you expect the test to continue to record these directories as
> present but have some other part of the testsuite code ignore them?

Something like that would indeed be good and proper. We should teach
the test suite to mark something as "scheduled for deletion" and to know
that if it's a directory it will be present on disk in multi-db mode but
won't otherwise.

- Julian
Received on 2010-08-04 13:25:44 CEST

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