Tweak a few tests

"for i in range(len(foo))" idiom changed to iterate directly over lists
instead of indexing them.  zip() generates the iterator in the case with
two lists. reversed() used for the Windows drive-letter test.

gdbm is not gone, just renamed. change test to find under either name.

Use a context manager for closing StringIO objects opened for capturing
standard I/O streams.

Signed-off-by: Mats Wichmann <mats@linux.com>

1 files changed, 4 insertions, 6 deletions

diff --git a/test/Batch/action-changed.py b/test/Batch/action-changed.py
index 777c7fb..b2b1673 100644
--- a/test/Batch/action-changed.py
+++ b/test/Batch/action-changed.py
@@ -43,12 +43,10 @@ import sys
 sep = sys.argv.index('--')
 targets = sys.argv[1:sep]
 sources = sys.argv[sep+1:]
-for i in range(len(targets)):
-    t = targets[i]
-    s = sources[i]
-    with open(t, 'wb') as fp, open(s, 'rb') as infp:
-        fp.write(bytearray('%s\\n','utf-8'))
-        fp.write(infp.read())
+for t, s in zip(targets, sources):
+    with open(t, 'wb') as ofp, open(s, 'rb') as ifp:
+        ofp.write(bytearray('%s\\n', 'utf-8'))
+        ofp.write(ifp.read())
 sys.exit(0)
 """