Author: artagnon
Date: Wed Jul 28 07:41:21 2010
New Revision: 979975

URL: http://svn.apache.org/viewvc?rev=979975&view=rev
Log:
Followup r979295 to handle the expected_is_regexp case in
display_lines.

* subversion/tests/cmdline/svntest/verify.py
  (display_lines): When expected is a string and not a list (in the
  expected_in_regexp case), put it in a one-member list, and don't
  output a diff.

Found by: rhuijben

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

Modified: subversion/trunk/subversion/tests/cmdline/svntest/verify.py
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/verify.py?rev=979975&r1=979974&r2=979975&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/svntest/verify.py (original)
+++ subversion/trunk/subversion/tests/cmdline/svntest/verify.py Wed Jul 28 
07:41:21 2010
@@ -280,25 +280,25 @@ def display_lines(message, label, expect
     output = 'EXPECTED %s' % label
     if expected_is_regexp:
       output += ' (regexp)'
+      expected = [expected + '\n']
     if expected_is_unordered:
       output += ' (unordered)'
     output += ':'
     print(output)
     for x in expected:
       sys.stdout.write(x)
-    if expected_is_regexp:
-      sys.stdout.write('\n')
   if actual is not None:
     print('ACTUAL %s:' % label)
     for x in actual:
       sys.stdout.write(x)
 
   # Additionally print unified diff
-  print('DIFF ' + ' '.join(output.split(' ')[1:]))
-  for x in unified_diff(expected, actual,
-                        fromfile="EXPECTED %s" % label,
-                        tofile="ACTUAL %s" % label):
-    sys.stdout.write(x)
+  if not expected_is_regexp:
+    print('DIFF ' + ' '.join(output.split(' ')[1:]))
+    for x in unified_diff(expected, actual,
+                          fromfile="EXPECTED %s" % label,
+                          tofile="ACTUAL %s" % label):
+      sys.stdout.write(x)
 
 def compare_and_display_lines(message, label, expected, actual,
                               raisable=None):


Reply via email to