svnsync tests may fail to FAIL?

Hi all,

in svnsync_tests.py, I find these lines in the overall run_test() function:

[[[
  run_sync(dest_sbox.repo_url)
  run_copy_revprops(dest_sbox.repo_url)
]]]

As I was browsing the output of a test, I noticed that the properties were
being synced *twice*. Once by svn_sync(), once by run_copy_revprops().

In fact, if I remove the "run_copy_revprops()", all tests still pass fine.
So I ask epg, who added the line in r29600:

Like this, if a normal sync fails to shovel the props over properly, the
copy_revprops call may mask that by doing it properly afterwards. The test
would fail to FAIL!

Instead, I think we should have either one or two tests specifically for
copy_revprops, or run the relevant or all tests twice, once with and once
without the copy_revprops part.

Any comments?

Thanks,
~Neels

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2353055