On Jun 21, 2009, at 5:01 AM, Jens Peters wrote:
> Hello there,
>
> I did some research/debugging to get to the ground of this test
> failure:
> http://svn.haxx.se/dev/archive-2009-05/0087.shtml
>
> It turns out that its just a mix up of wc.get_status_editor2
> arguments.
> The following patch fixes that, at least for me.
>
> With this patch I can finally finish the python without a failure
> (yes :)).
>
> Regards,
> jens
>
> [[[
> Fix a python wc testcase failure.
> (SubversionException, "Can't check path '[path][path]': The filename,
> directory name, or volume label syntax is incorrect.")
>
> * subversion/bindings/swig/python/tests/wc.py
> (test_status_editor):
> Put the correct arguments into 'wc.get_status_editor2'. The second
> argument should be 'target'.
>
> Patch by: Jens Peters <jpeters7677 { at } gmx.de>
> ]]]
Hmmm, I wonder if this is really a problem with the underlying
backward compat C code. The test presumably passed when it was
originally added, but when the underlying APIs got rewritten, there
may be some bugs in the backward compatibility wrapper. If that's the
case, we should fix the bugs there, not in the python testsuite.
Caveat: I haven't looked at the history of this test, or the
underlying code, so I could be completely making stuff up. But it's a
thought.
-Hyrum
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2364184
Received on 2009-06-22 16:30:16 CEST