run-tests: don't add python lines to expected dict stable
authorAdrian Buehlmann <adrian@cadifra.com>
Wed, 30 May 2012 14:28:57 +0200
branchstable
changeset 1085 a2e7456b162e
parent 1084 b668ecd81629
child 1086 a2e8ccb65224
run-tests: don't add python lines to expected dict For test input lines of *.t files starting with ' >>> ', the code block for ' >>> ' 609: if l.startswith(' >>> '): # python inlines 610: after.setdefault(pos, []).append(l) was (unsurprisingly) executed, but because there was an "if" instead of an "elif" on the condition "l.startswith(' ... ')", program execution proceeded to line 636 635: elif l.startswith(' '): # results 636: # queue up a list of expected results 637: expected.setdefault(pos, []).append(l[2:]) due to the fact that if l starts with ' >>> ' it also starts with ' '. The net effect was that python command lines in *.t files were (surprisingly) also added to the "expected" dict. This caused no externally observable bad behavior, as the "expected" dict was not consulted for these lines. [ original upstream message ]
tests/run-tests.py
--- a/tests/run-tests.py	Sun Jun 03 19:35:23 2012 +0200
+++ b/tests/run-tests.py	Wed May 30 14:28:57 2012 +0200
@@ -617,7 +617,7 @@
                 script.append('%s -m heredoctest <<EOF\n' % PYTHON)
             addsalt(n, True)
             script.append(l[2:])
-        if l.startswith('  ... '): # python inlines
+        elif l.startswith('  ... '): # python inlines
             after.setdefault(prepos, []).append(l)
             script.append(l[2:])
         elif l.startswith('  $ '): # commands