You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2012/04/26 18:48:41 UTC

svn commit: r1330949 - /subversion/trunk/subversion/tests/cmdline/svntest/main.py

Author: danielsh
Date: Thu Apr 26 16:48:40 2012
New Revision: 1330949

URL: http://svn.apache.org/viewvc?rev=1330949&view=rev
Log:
Follow-up to r1330917: enable the check for Windows too.

* subversion/tests/cmdline/svntest/main.py
  (_line_contains_repos_diskpath): New local-scope helper.
  (run_command_stdin): Use os.path.join().

Modified:
    subversion/trunk/subversion/tests/cmdline/svntest/main.py

Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1330949&r1=1330948&r2=1330949&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original)
+++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Thu Apr 26 16:48:40 2012
@@ -511,14 +511,17 @@ def run_command_stdin(command, error_exp
                                                         stdin_lines,
                                                         *varargs)
 
+  def _line_contains_repos_diskpath(line):
+    return \
+         os.path.join('cmdline', 'svn-test-work', 'repositories') in line \
+      or os.path.join('cmdline', 'svn-test-work', 'local_tmp', 'repos') in line 
+
   for lines, name in [[stdout_lines, "stdout"], [stderr_lines, "stderr"]]:
     if is_ra_type_file() or 'svnadmin' in command:
       break
     # Does the server leak the repository on-disk path?
     # (prop_tests-12 installs a hook script that does that intentionally)
-    if any(map(lambda line: 'cmdline/svn-test-work/repositories' in line \
-                            or 'cmdline/svn-test-work/local_tmp/repos' in line,
-               lines)) \
+    if any(map(_line_contains_repos_diskpath, lines)) \
        and not any(map(lambda arg: 'prop_tests-12' in arg, varargs)):
       raise Failure("Repository diskpath in %s: %r" % (name, lines))