Make a few test non-optional.

And add a few testing dependencies.

IPython/core/tests/test_display.py

@@ -2,7 +2,6 b''
 # Distributed under the terms of the Modified BSD License.
 
 import json
-import tempfile
 import os
 import warnings
 

IPython/core/tests/test_magic.py

@@ -39,6 +39,8 b' else:'
     from StringIO import StringIO
 
 
+_ip = get_ipython()
+
 @magic.magics_class
 class DummyMagics(magic.Magics): pass
 
@@ -89,7 +91,6 b' def test_config():'
 
 def test_rehashx():
     # clear up everything
-    _ip = get_ipython()
     _ip.alias_manager.clear_aliases()
     del _ip.db['syscmdlist']
     
@@ -625,7 +626,6 b' def test_extension():'
         sys.path.remove(daft_path)
 
 
[e-mail]_without('nbformat')
 def test_notebook_export_json():
     _ip = get_ipython()
     _ip.history_manager.reset()   # Clear any existing history.

IPython/core/tests/test_run.py

@@ -254,8 +254,8 b' class TestMagicRunSimple(tt.TempFileMixin):'
 
         Returning from another run magic deletes the namespace"""
         # see ticket https://github.com/ipython/ipython/issues/238
-        class secondtmp(tt.TempFileMixin): pass
-        empty = secondtmp()
+        
+        with tt.TempFileMixin() as empty:
         empty.mktmp('')
         # On Windows, the filename will have \users in it, so we need to use the
         # repr so that the \u becomes \\u.
@@ -278,7 +278,7 b' class TestMagicRunSimple(tt.TempFileMixin):'
                    "def afunc():\n"
                    "  return avar\n")
 
-        empty = tt.TempFileMixin()
+        with tt.TempFileMixin() as empty:
         empty.mktmp("")
         
         _ip.magic('run %s' % self.fname)

IPython/terminal/tests/test_help.py

@@ -25,6 +25,5 b' def test_locate_help():'
 def test_locate_profile_help():
     tt.help_all_output_test("locate profile")
 
-@skip_without('nbformat')  # Requires jsonschema to be installed
 def test_trust_help():
     tt.help_all_output_test("trust")

IPython/testing/iptest.py

@@ -44,6 +44,7 b' pjoin = path.join'
 
 
 # Enable printing all warnings raise by IPython's modules
+warnings.filterwarnings('ignore', message='.*Matplotlib is building the font cache.*', category=UserWarning, module='.*')
 if sys.version_info > (3,0):
     warnings.filterwarnings('error', message='.*', category=ResourceWarning, module='.*')
 warnings.filterwarnings('default', message='.*', category=Warning, module='IPy.*')

IPython/testing/tools.py

@@ -303,6 +303,13 b' class TempFileMixin(object):'
                 # delete it.  I have no clue why
                 pass
 
+    def __enter__(self):
+        return self
+
+    def __exit__(self, exc_type, exc_value, traceback):
+        self.tearDown()
+
+
 pair_fail_msg = ("Testing {0}\n\n"
                 "In:\n"
                 "  {1!r}\n"

setup.py

@@ -182,13 +182,14 b' extras_require = dict('
     parallel = ['ipyparallel'],
     qtconsole = ['qtconsole'],
     doc = ['Sphinx>=1.3'],
-    test = ['nose>=0.10.1', 'requests', 'testpath', 'pygments'],
+    test = ['nose>=0.10.1', 'requests', 'testpath', 'pygments', 'nbformat', 'ipykernel'],
     terminal = [],
     kernel = ['ipykernel'],
     nbformat = ['nbformat'],
     notebook = ['notebook', 'ipywidgets'],
     nbconvert = ['nbconvert'],
 )
+
 install_requires = [
     'setuptools>=18.5',
     'decorator',