You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by br...@apache.org on 2014/05/15 23:53:22 UTC

svn commit: r1595061 - /subversion/trunk/subversion/bindings/swig/python/tests/client.py

Author: breser
Date: Thu May 15 21:53:22 2014
New Revision: 1595061

URL: http://svn.apache.org/r1595061
Log:
swig-py: Sort the paths in the update test to avoid spurious failures.

* subversion/tests/swig/python/tests/client.py
  (test_update4): The ordering isn't necessarily guaranteed so sort.

Modified:
    subversion/trunk/subversion/bindings/swig/python/tests/client.py

Modified: subversion/trunk/subversion/bindings/swig/python/tests/client.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/swig/python/tests/client.py?rev=1595061&r1=1595060&r2=1595061&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/swig/python/tests/client.py (original)
+++ subversion/trunk/subversion/bindings/swig/python/tests/client.py Thu May 15 21:53:22 2014
@@ -473,6 +473,8 @@ class SubversionClientTestCase(unittest.
     # All normal subversion apis process paths in Subversion's canonical format,
     # which isn't the platform specific format
     expected_paths = [x.replace(os.path.sep, '/') for x in expected_paths]
+    self.notified_paths.sort()
+    expected_paths.sort()
 
     self.assertEquals(self.notified_paths, expected_paths)
 
@@ -493,6 +495,8 @@ class SubversionClientTestCase(unittest.
     expected_paths = [x.replace(os.path.sep, '/') for x in expected_paths]
     client.update4((path,), rev, core.svn_depth_unknown, True, False, False,
                    False, False, self.client_ctx)
+    self.notified_paths.sort()
+    expected_paths.sort()
     self.assertEquals(self.notified_paths, expected_paths)