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

Re: svn commit: r21837 - in branches/incomplete-directories: . subversion/tests/cmdline

From: Karl Fogel <kfogel_at_google.com>
Date: 2006-10-09 10:06:06 CEST

Kamesh Jayachandran <kamesh@collab.net> writes:
>> +++ branches/incomplete-directories/subversion/tests/cmdline/depth_tests.py Sun Oct 8 21:19:37 2006
>> @@ -0,0 +1,114 @@
>> +#!/usr/bin/env python
>> +#
>> +# depth_tests.py: Testing that operations work as expected at
>> +# various depths (depth-0, depth-1, depth-infinity).
>> +#
>> +# Subversion is a tool for revision control.
>> +# See http://subversion.tigris.org for more information.
>> +#
>> +# ====================================================================
>> +# Copyright (c) 2000-2006 CollabNet. All rights reserved.
>>
> Should it not be only 2006?

Good catch! Will fix, thanks.

>> +# This software is licensed as described in the file COPYING, which
>> +# you should have received as part of this distribution. The terms
>> +# are also available at http://subversion.tigris.org/license-1.html.
>> +# If newer versions of this license are posted there, you may use a
>> +# newer version instead, at your option.
>> +#
>> +######################################################################
>> +
>> +# General modules
>> +import sys, re, os, time
>>
> I could not see 'sys', 're', 'time' used anywhere in the script.

Heh. I copied this from checkout_tests.py, as you can probably tell.
Will fix.

>> +
>> +# Our testing module
>> +import svntest
>> +from svntest import wc, SVNAnyOutput
>> +
>> +# (abbreviation)
>> +Skip = svntest.testcase.Skip
>>
> I could not see 'Skip' used anywhere in the script.
>> +XFail = svntest.testcase.XFail
>> +Item = wc.StateItem
>>
> I could not see 'Item' used anywhere in the script.
>> +XFail = svntest.testcase.XFail
>>
> Duplicate XFail?

Huh, I don't know how the duplicate XFail happened. The others are
standard to have at the top of any test script, and I think they
should be in this one even though they're not used right now, because
they surely will be in the future.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 9 10:06:33 2006

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