In the svnsync tests, remove a load-and-dump cycle from the repository comparison function.
In the svnsync tests, remove a load-and-dump cycle from the repository
comparison function.

The load-and-dump cycle was introduced in r1230676 to account for different
ordering of nodes in dumpfiles generated by different versions of
Subversion.

In r1292507 the test suite was changed to compare dumpfiles in parsed form,
to account for variable ordering of nodes produced by different runs of
Subversion. This makes the previous patch redundant.

Loading and dumping can change data in some cases -- for example 'load'
currently rewrites svn:mergeinfo -- and so could mask errors or cause errors
in certain tests. The current tests seem to be unaffected but it broke a
test that I am writing for issue #4476 "Mergeinfo containing r0 makes
svnsync and dump and load fail". That is not to say that comparing dumpfiles
in parsed form cannot also mask errors, but one procedure is better than
two.

* subversion/tests/cmdline/svnsync_tests.py
  (verify_mirror): Take the expected dump file contents directly instead
    of in a repository in a sandbox.
  (run_test,
   delete_revprops): Pass the expected dump file contents directly instead
    of in a repository in a sandbox.


git-svn-id: f8a4e5e023278da1e04e203c7fe051e3c4285d88@1646230 13f79535-47bb-0310-9956-ffa450edef68

YueLinHo/Subversion,YueLinHo/Subversion,YueLinHo/Subversion,YueLinHo/Subversion,YueLinHo/Subversion,YueLinHo/Subversion,YueLinHo/Subversion,YueLinHo/Subversion