do_edit() no longer edits history, just files #252

1 files changed, 0 insertions, 73 deletions

diff --git a/tests/test_cmd2.py b/tests/test_cmd2.py
index 8ee41096..86296246 100644
--- a/tests/test_cmd2.py
+++ b/tests/test_cmd2.py
@@ -780,84 +780,11 @@ def test_edit_blank(base_app, monkeypatch):
     m = mock.MagicMock(name='system')
     monkeypatch.setattr("os.system", m)
 
-    # Run help command just so we have a command in history
-    run_cmd(base_app, 'help')
-
-    run_cmd(base_app, 'edit')
-
-    # We have an editor, so should expect a system call
-    m.assert_called_once()
-
-def test_edit_empty_history(base_app, capsys):
     run_cmd(base_app, 'edit')
-    out, err = capsys.readouterr()
-    assert out == ''
-    assert err == 'ERROR: edit must be called with argument if history is empty\n'
-
-def test_edit_valid_positive_number(base_app, monkeypatch):
-    # Set a fake editor just to make sure we have one.  We aren't really going to call it due to the mock
-    base_app.editor = 'fooedit'
-
-    # Mock out the os.system call so we don't actually open an editor
-    m = mock.MagicMock(name='system')
-    monkeypatch.setattr("os.system", m)
-
-    # Run help command just so we have a command in history
-    run_cmd(base_app, 'help')
-
-    run_cmd(base_app, 'edit 1')
 
     # We have an editor, so should expect a system call
     m.assert_called_once()
 
-def test_edit_valid_negative_number(base_app, monkeypatch):
-    # Set a fake editor just to make sure we have one.  We aren't really going to call it due to the mock
-    base_app.editor = 'fooedit'
-
-    # Mock out the os.system call so we don't actually open an editor
-    m = mock.MagicMock(name='system')
-    monkeypatch.setattr("os.system", m)
-
-    # Run help command just so we have a command in history
-    run_cmd(base_app, 'help')
-
-    run_cmd(base_app, 'edit "-1"')
-
-    # We have an editor, so should expect a system call
-    m.assert_called_once()
-
-def test_edit_invalid_positive_number(base_app, monkeypatch):
-    # Set a fake editor just to make sure we have one.  We aren't really going to call it due to the mock
-    base_app.editor = 'fooedit'
-
-    # Mock out the os.system call so we don't actually open an editor
-    m = mock.MagicMock(name='system')
-    monkeypatch.setattr("os.system", m)
-
-    # Run help command just so we have a command in history
-    run_cmd(base_app, 'help')
-
-    run_cmd(base_app, 'edit 23')
-
-    # History index is invalid, so should expect a system call
-    m.assert_not_called()
-
-def test_edit_invalid_negative_number(base_app, monkeypatch):
-    # Set a fake editor just to make sure we have one.  We aren't really going to call it due to the mock
-    base_app.editor = 'fooedit'
-
-    # Mock out the os.system call so we don't actually open an editor
-    m = mock.MagicMock(name='system')
-    monkeypatch.setattr("os.system", m)
-
-    # Run help command just so we have a command in history
-    run_cmd(base_app, 'help')
-
-    run_cmd(base_app, 'edit "-23"')
-
-    # History index is invalid, so should expect a system call
-    m.assert_not_called()
-
 
 def test_base_py_interactive(base_app):
     # Mock out the InteractiveConsole.interact() call so we don't actually wait for a user's response on stdin