Convert test_imp over to unittest.

Lib/test/test_imp.py

@@ -1,43 +1,47 @@
 import imp
-from test.test_support import TestFailed, TestSkipped
+import thread
-try:
+import unittest
-    import thread
+from test import test_support
-except ImportError:
-    raise TestSkipped("test only valid when thread support is available")
 
-def verify_lock_state(expected):
-    if imp.lock_held() != expected:
-        raise TestFailed("expected imp.lock_held() to be %r" % expected)
 
-def testLock():
+class LockTests(unittest.TestCase):
-    LOOPS = 50
 
-    # The import lock may already be held, e.g. if the test suite is run
+    """Very basic test of import lock functions."""
-    # via "import test.autotest".
-    lock_held_at_start = imp.lock_held()
-    verify_lock_state(lock_held_at_start)
 
-    for i in range(LOOPS):
+    def verify_lock_state(self, expected):
-        imp.acquire_lock()
+        self.failUnlessEqual(imp.lock_held(), expected,
-        verify_lock_state(True)
+                             "expected imp.lock_held() to be %r" % expected)
+    def testLock(self):
+        LOOPS = 50
 
-    for i in range(LOOPS):
+        # The import lock may already be held, e.g. if the test suite is run
-        imp.release_lock()
+        # via "import test.autotest".
+        lock_held_at_start = imp.lock_held()
+        self.verify_lock_state(lock_held_at_start)
 
-    # The original state should be restored now.
+        for i in range(LOOPS):
-    verify_lock_state(lock_held_at_start)
+            imp.acquire_lock()
+            self.verify_lock_state(True)
 
-    if not lock_held_at_start:
+        for i in range(LOOPS):
-        try:
             imp.release_lock()
-        except RuntimeError:
+
-            pass
+        # The original state should be restored now.
-        else:
+        self.verify_lock_state(lock_held_at_start)
-            raise TestFailed("release_lock() without lock should raise "
+
-                             "RuntimeError")
+        if not lock_held_at_start:
+            try:
+                imp.release_lock()
+            except RuntimeError:
+                pass
+            else:
+                self.fail("release_lock() without lock should raise "
+                            "RuntimeError")
 
 def test_main():
-    testLock()
+    test_support.run_unittest(
+                LockTests,
+            )
 
 if __name__ == "__main__":
     test_main()

Misc/NEWS

@@ -139,6 +139,8 @@ Extension Modules
 Tests
 -----
 
+- Converted test_imp to use unittest.
+
 - Fix bsddb test_basics.test06_Transactions to check the version
   number properly.