CommandTests.testgetoutput():
Make sure we aren't masking any errors raised in tempfile.mkdtemp() by
referencing the (then) unbound local 'dir'.
diff --git a/Lib/test/test_commands.py b/Lib/test/test_commands.py
index ef87d79..0f7d15f 100644
--- a/Lib/test/test_commands.py
+++ b/Lib/test/test_commands.py
@@ -27,6 +27,7 @@
# we use mkdtemp in the next line to create an empty directory
# under our exclusive control; from that, we can invent a pathname
# that we _know_ won't exist. This is guaranteed to fail.
+ dir = None
try:
dir = tempfile.mkdtemp()
name = os.path.join(dir, "foo")
@@ -34,7 +35,8 @@
status, output = getstatusoutput('cat ' + name)
self.assertNotEquals(status, 0)
finally:
- os.rmdir(dir)
+ if dir is not None:
+ os.rmdir(dir)
def test_getstatus(self):
# This pattern should match 'ls -ld /.' on any posix