##// END OF EJS Templates
MAINT: mock slowest test....
MAINT: mock slowest test. The slowest test of our CI is ~3sec as it use subprocess os/system. Mocking make it instantaneous. With our big elements matrix in CI that should save us a bunch of time in total from PR submission to green.

File last commit:

r26716:4142eaaf
r28098:86559125
Show More
test_editorhooks.py
32 lines | 884 B | text/x-python | PythonLexer
/ IPython / lib / tests / test_editorhooks.py
Min RK
fix editorhooks typo...
r18869 """Test installing editor hooks"""
import sys
Srinivas Reddy Thatiparthy
import mock from unittest
r23058 from unittest import mock
Min RK
fix editorhooks typo...
r18869
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,
})
Segev Finer
IPython/lib/editorhooks.py: wait for process even if wait=False...
r23294 return mock.MagicMock(**{'wait.return_value': 0})
Min RK
fix editorhooks typo...
r18869 editorhooks.install_editor('foo -l {line} -f {filename}', wait=False)
with mock.patch('subprocess.Popen', fake_popen):
get_ipython().hooks.editor('the file', 64)
Matthias Bussonnier
remove nose tools from test_editorhooks
r26716 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"]
Min RK
fix editorhooks typo...
r18869 else:
expected = "foo -l 64 -f 'the file'"
cmd = args[0]
Matthias Bussonnier
remove nose tools from test_editorhooks
r26716 assert cmd == expected