upstream/ipython Files · IPython/lib/tests/test_editorhooks.py

First step in reintegrating Jedi...

First step in reintegrating Jedi If Jedi is installed expose a private API use it with prompt toolkit. Jedi does not _yet_ provide all the completion IPython has, so this is still a bit awkward. In order to debug this (and see what is Jedi provided we for now inject a fake Jedi/IPython delimiter in the menu. Jedi completion and this behavior are enabled by default, but could likely be opt-in. Add also a number of debug flags to be able to track why jedi is not working, and/or what completions are found by IPython and not Jedi. That should give us a bit of heads up and feedback to know whether we can remove part of the IPython completer, and more especially if we can drop `python_matches`. Once `python_matches` is dropped and some other of the current matchers are either dropped or converted to the new API, that should simplify the internal quite a bit. That would just be too much for an already BIG pull-request.

Srinivas Reddy Thatiparthy - - Load All Authors

File last commit:

r23058:e47cb3c5


                r23284:3ff1be2e

Download file

             test_editorhooks.py
        
                    33 lines
            
             | 883 B
            
                | text/x-python
            
             |
                PythonLexer
            
             / IPython / lib / tests / test_editorhooks.py
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      """Test installing editor hooks"""

      import sys

      from unittest import mock

      import nose.tools as nt

      from IPython import get_ipython

      from IPython.lib import editorhooks

      def test_install_editor():

          called = []

          def fake_popen(*args, **kwargs):

              called.append({

                  'args': args,

                  'kwargs': kwargs,

              })

          editorhooks.install_editor('foo -l {line} -f {filename}', wait=False)

          with mock.patch('subprocess.Popen', fake_popen):

              get_ipython().hooks.editor('the file', 64)

          nt.assert_equal(len(called), 1)

          args = called[0]['args']

          kwargs = called[0]['kwargs']

          nt.assert_equal(kwargs, {'shell': True})

          if sys.platform.startswith('win'):

              expected = ['foo', '-l', '64', '-f', 'the file']

          else:

              expected = "foo -l 64 -f 'the file'"

          cmd = args[0]

          nt.assert_equal(cmd, expected)

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				"""Test installing editor hooks"""
				import sys
				from unittest import mock

				import nose.tools as nt

				from IPython import get_ipython
				from IPython.lib import editorhooks

				def test_install_editor():
				called = []
				def fake_popen(args, *kwargs):
				called.append({
				'args': args,
				'kwargs': kwargs,
				})
				editorhooks.install_editor('foo -l {line} -f {filename}', wait=False)

				with mock.patch('subprocess.Popen', fake_popen):
				get_ipython().hooks.editor('the file', 64)

				nt.assert_equal(len(called), 1)
				args = called[0]['args']
				kwargs = called[0]['kwargs']

				nt.assert_equal(kwargs, {'shell': True})

				if sys.platform.startswith('win'):
				expected = ['foo', '-l', '64', '-f', 'the file']
				else:
				expected = "foo -l 64 -f 'the file'"
				cmd = args[0]
				nt.assert_equal(cmd, expected)