Always reset after `%run -i` in tests.

Currently, `IPython/core/tests/run_tests.py` has a few tests that do `run -i`,
which has side-effects on the IPython environment shared by all tests. In
particular, setting `__file__` in the global namespace leads to traceback
handling in `ultratb` getting confused, since it no longer knows which file to
look in.

This fixes existing uses of `run -i`, and adds a note reminding future
travelers to be careful when adding new cases.

IPython/core/tests/test_run.py

@@ -6,6 +6,9 b' verify subtle object deletion and reference counting issues, the %run tests'
 will be kept in this separate file.  This makes it easier to aggregate in one
 place the tricks needed to handle it; most other magics are much easier to test
 and we do so in a common test_magic file.
+
+Note that any test using `run -i` should make sure to do a `reset` afterwards,
+as otherwise it may influence later tests.
 """
 
 # Copyright (c) IPython Development Team.
@@ -317,12 +320,18 b' tclass.py: deleting object: C-third'
         src = "yy = zz\n"
         self.mktmp(src)
         _ip.run_cell("zz = 23")
+        try:
         _ip.magic('run -i %s' % self.fname)
         nt.assert_equal(_ip.user_ns['yy'], 23)
+        finally:
         _ip.magic('reset -f')
+            
         _ip.run_cell("zz = 23")
+        try:
         _ip.magic('run -i %s' % self.fname)
         nt.assert_equal(_ip.user_ns['yy'], 23)
+        finally:
+            _ip.magic('reset -f')
     
     def test_unicode(self):
         """Check that files in odd encodings are accepted."""
@@ -505,8 +514,11 b' def test_run__name__():'
         _ip.magic('run -n {}'.format(path))
         nt.assert_equal(_ip.user_ns.pop('q'), 'foo')
 
+        try:
         _ip.magic('run -i -n {}'.format(path))
         nt.assert_equal(_ip.user_ns.pop('q'), 'foo')
+        finally:
+            _ip.magic('reset -f')
 
 
 def test_run_tb():