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

capture_output does not respect trailing semicolon (#13940)...

capture_output does not respect trailing semicolon (#13940) This change tests whether the Python code inside the `%%capture` magic ends with a `';'`. If it does, the code no longer creates the object that is supposed to store the cell output. If such object already exists, it is deleted. Please review and suggest changes if you think they can improve the solution. Thanks.

Matthias Bussonnier - - Load All Authors

File last commit:

r26716:4142eaaf


                r28128:75ecfe93

Download file

             test_editorhooks.py
        
                    32 lines
            
             | 884 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

      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,

              })

              return mock.MagicMock(**{'wait.return_value': 0})

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

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

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

          assert len(called) == 1

          args = called[0]["args"]

          kwargs = called[0]["kwargs"]

          assert 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]

          assert 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

				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,
				})
				return mock.MagicMock(**{'wait.return_value': 0})
				editorhooks.install_editor('foo -l {line} -f {filename}', wait=False)

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

				assert len(called) == 1
				args = called[0]["args"]
				kwargs = called[0]["kwargs"]

				assert 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]
				assert cmd == expected