ipython
diff --git a/‎IPython/testing/decorators.py‎
Lines changed: 29 additions & 56 deletions b/‎IPython/testing/decorators.py‎
Lines changed: 29 additions & 56 deletions
diff --git a/‎IPython/testing/plugin/README.txt‎
Lines changed: 0 additions & 34 deletions b/‎IPython/testing/plugin/README.txt‎
Lines changed: 0 additions & 34 deletions
@@ -1,51 +1,15 @@
-# -*- coding: utf-8 -*-
-"""Decorators for labeling test objects.
-
-Decorators that merely return a modified version of the original function
-object are straightforward.  Decorators that return a new function object need
-to use nose.tools.make_decorator(original_function)(decorator) in returning the
-decorator, in order to preserve metadata such as function name, setup and
-teardown functions and so on - see nose.tools for more information.
-
-This module provides a set of useful decorators meant to be ready to use in
-your own tests.  See the bottom of the file for the ready-made ones, and if you
-find yourself writing a new one that may be of generic use, add it here.
-
-Included decorators:
-
-
-Lightweight testing that remains unittest-compatible.
-
-- An @as_unittest decorator can be used to tag any normal parameter-less
-  function as a unittest TestCase.  Then, both nose and normal unittest will
-  recognize it as such.  This will make it easier to migrate away from Nose if
-  we ever need/want to while maintaining very lightweight tests.
-
-NOTE: This file contains IPython-specific decorators. Using the machinery in
-IPython.external.decorators, we import either numpy.testing.decorators if numpy is
-available, OR use equivalent code in IPython.external._decorators, which
-we've copied verbatim from numpy.
-
-"""
-
-# Copyright (c) IPython Development Team.
-# Distributed under the terms of the Modified BSD License.
-
 import os
 import shutil
 import sys
 import tempfile
-import unittest
 from importlib import import_module
 
-from decorator import decorator
-
 # Expose the unittest-driven decorators
 from .ipunittest import ipdoctest, ipdocstring
 
-#-----------------------------------------------------------------------------
+# -----------------------------------------------------------------------------
 # Classes and functions
-#-----------------------------------------------------------------------------
+# -----------------------------------------------------------------------------
 
 # Utility functions
 
@@ -95,10 +59,12 @@ def skip(msg=None):
        decorator : function
          Decorator, which, when applied to a function, causes SkipTest
          to be raised, with the optional message added.
-      """
+    """
     if msg and not isinstance(msg, str):
-        raise ValueError('invalid object passed to `@skip` decorator, did you '
-                         'meant `@skip()` with brackets ?')
+        raise ValueError(
+            "invalid object passed to `@skip` decorator, did you "
+            "meant `@skip()` with brackets ?"
+        )
     return skipif(True, msg)
 
 
@@ -107,7 +73,8 @@ def onlyif(condition, msg):
 
     return skipif(not condition, msg)
 
-#-----------------------------------------------------------------------------
+
+# -----------------------------------------------------------------------------
 # Utility functions for decorators
 def module_not_available(module):
     """Can module be imported?  Returns true if module does NOT import.
@@ -124,15 +91,15 @@ def module_not_available(module):
     return mod_not_avail
 
 
-#-----------------------------------------------------------------------------
+# -----------------------------------------------------------------------------
 # Decorators for public use
 
 # Decorators to skip certain tests on specific platforms.
-skip_win32 = skipif(sys.platform == 'win32',
-                    "This test does not run under Windows")
-skip_linux = skipif(sys.platform.startswith('linux'),
-                    "This test does not run under Linux")
-skip_osx = skipif(sys.platform == 'darwin',"This test does not run under OS X")
+skip_win32 = skipif(sys.platform == "win32", "This test does not run under Windows")
+skip_linux = skipif(
+    sys.platform.startswith("linux"), "This test does not run under Linux"
+)
+skip_osx = skipif(sys.platform == "darwin", "This test does not run under OS X")
 
 
 # Decorators to skip tests if not on specific platforms.
@@ -144,20 +111,23 @@ def module_not_available(module):
     not sys.platform.startswith("darwin"), "This test only runs under macOS"
 )
 
-_x11_skip_cond = (sys.platform not in ('darwin', 'win32') and
-                  os.environ.get('DISPLAY', '') == '')
+_x11_skip_cond = (
+    sys.platform not in ("darwin", "win32") and os.environ.get("DISPLAY", "") == ""
+)
 _x11_skip_msg = "Skipped under *nix when X11/XOrg not available"
 
 skip_if_no_x11 = skipif(_x11_skip_cond, _x11_skip_msg)
 
 # Other skip decorators
 
 # generic skip without module
-skip_without = lambda mod: skipif(module_not_available(mod), "This test requires %s" % mod)
+skip_without = lambda mod: skipif(
+    module_not_available(mod), "This test requires %s" % mod
+)
 
-skipif_not_numpy = skip_without('numpy')
+skipif_not_numpy = skip_without("numpy")
 
-skipif_not_matplotlib = skip_without('matplotlib')
+skipif_not_matplotlib = skip_without("matplotlib")
 
 # A null 'decorator', useful to make more readable code that needs to pick
 # between different decorators based on OS or other conditions
@@ -166,15 +136,18 @@ def module_not_available(module):
 # Some tests only run where we can use unicode paths. Note that we can't just
 # check os.path.supports_unicode_filenames, which is always False on Linux.
 try:
-    f = tempfile.NamedTemporaryFile(prefix=u"tmp€")
+    f = tempfile.NamedTemporaryFile(prefix="tmp€")
 except UnicodeEncodeError:
     unicode_paths = False
+# TODO: should this be finnally ?
 else:
     unicode_paths = True
     f.close()
 
-onlyif_unicode_paths = onlyif(unicode_paths, ("This test is only applicable "
-                                    "where we can use unicode in filenames."))
+onlyif_unicode_paths = onlyif(
+    unicode_paths,
+    ("This test is only applicable where we can use unicode in filenames."),
+)
 
 
 def onlyif_cmds_exist(*commands):