apply review

Author: youknowone

Lib/test/test_inspect/test_inspect.py

@@ -30,9 +30,8 @@
 
 try:
     import _pickle
-    MISSING_C_PICKLE = False
 except ImportError:
-    MISSING_C_PICKLE = True
+    _pickle = None
 
 
 try:
@@ -1415,7 +1414,7 @@ def test(): pass
 
     @unittest.skipIf(MISSING_C_DOCSTRINGS,
                      "Signature information for builtins requires docstrings")
-    @unittest.skipIf(MISSING_C_PICKLE, "requires _pickle")
+    @unittest.skipUnless(_pickle, "requires _pickle module")
     def test_getfullargspec_builtin_methods(self):
         self.assertFullArgSpecEquals(_pickle.Pickler.dump, ['self', 'obj'])
 
@@ -4641,7 +4640,7 @@ class D(C): pass
 
     @unittest.skipIf(MISSING_C_DOCSTRINGS,
                      "Signature information for builtins requires docstrings")
-    @unittest.skipIf(MISSING_C_PICKLE, "requires _pickle")
+    @unittest.skipUnless(_pickle, "requires _pickle module")
     def test_signature_on_builtin_class(self):
         expected = ('(file, protocol=None, fix_imports=True, '
                     'buffer_callback=None)')
@@ -5183,7 +5182,7 @@ class foo: pass
 
     @unittest.skipIf(MISSING_C_DOCSTRINGS,
                      "Signature information for builtins requires docstrings")
-    @unittest.skipIf(MISSING_C_PICKLE, "requires _pickle")
+    @unittest.skipUnless(_pickle, "requires _pickle module")
     def test_signature_from_callable_builtin_obj(self):
         class MySignature(inspect.Signature): pass
         sig = MySignature.from_callable(_pickle.Pickler)