Rewrote. As reported on c.l.py, when the test suite is run via

"import test.autotest", temp_imp failed because the import lock was
still held at the test's end (the test assumed it wouldn't be), and
then a RuntimeError got raised at the end of the entire suite run because
test_imp cleared the import lock as a side effect of trying to test that
the import lock wasn't held (but a legitimate import is in progress,
so the lock should be held, and the import machinery complained when it
found that the lock was unexpectedly cleareed).

Also removed the unittest scaffolding.  It didn't buy anything here, and
the test was raising regrtest's TestFailed instead of using the unittest
failure-reporting mechanisms.

diff --git a/Lib/test/test_imp.py b/Lib/test/test_imp.py
index 22e25ba5ec57f928ee3eaff3c667b0e5580064f4..9b7e3b79e6b53e1b8c29d1e11539cf447b468a07 100644 (file)
--- a/Lib/test/test_imp.py
+++ b/Lib/test/test_imp.py
@@ -1,29 +1,39 @@
-
 import imp
-import unittest
-from test.test_support import TestFailed, run_unittest
+from test.test_support import TestFailed
 
-class ImpLock(unittest.TestCase):
+def verify_lock_state(expected):
+    if imp.lock_held() != expected:
+        raise TestFailed("expected imp.lock_held() to be %r" % expected)
 
-    # XXX this test is woefully inadequate, please fix me
-    def testLock(self):
-        LOOPS = 50
-        for i in range(LOOPS):
-            imp.acquire_lock()
-        for i in range(LOOPS):
-            imp.release_lock()
+def testLock():
+    LOOPS = 50
+
+    # The import lock may already be held, e.g. if the test suite is run
+    # via "import test.autotest".
+    lock_held_at_start = imp.lock_held()
+    verify_lock_state(lock_held_at_start)
+
+    for i in range(LOOPS):
+        imp.acquire_lock()
+        verify_lock_state(True)
 
-        for i in range(LOOPS):
-            try:
-                imp.release_lock()
-            except RuntimeError:
-                pass
-            else:
-                raise TestFailed, \
-                    "release_lock() without lock should raise RuntimeError"
+    for i in range(LOOPS):
+        imp.release_lock()
+
+    # The original state should be restored now.
+    verify_lock_state(lock_held_at_start)
+
+    if not lock_held_at_start:
+        try:
+            imp.release_lock()
+        except RuntimeError:
+            pass
+        else:
+            raise TestFailed("release_lock() without lock should raise "
+                             "RuntimeError")
 
 def test_main():
-    run_unittest(ImpLock)
+    testLock()
 
 if __name__ == "__main__":
     test_main()