diff options
| author | Björn Linse <bjorn.linse@gmail.com> | 2019-10-13 20:18:22 +0200 |
|---|---|---|
| committer | Björn Linse <bjorn.linse@gmail.com> | 2019-10-13 22:10:42 +0200 |
| commit | 4987311fb5b8f4a11d26995f71f5f402a9e2ace4 (patch) | |
| tree | e070d02687ba338260a004be738e777b1639e508 /test/functional/core/job_spec.lua | |
| parent | a330129a280a34963e42d498f7cb1f53e6723e85 (diff) | |
| download | rneovim-4987311fb5b8f4a11d26995f71f5f402a9e2ace4.tar.gz rneovim-4987311fb5b8f4a11d26995f71f5f402a9e2ace4.tar.bz2 rneovim-4987311fb5b8f4a11d26995f71f5f402a9e2ace4.zip | |
tests/ui: remove unnecessary screen:detach()
It is perfectly fine and expected to detach from the screen just by
the UI disconnecting from nvim or exiting nvim. Just keep detach() in
screen_basic_spec, to get some coverage of the detach method itself.
This avoids hang on failure in many situations (though one could argue
that detach() should be "fast", or at least "as fast as resize",
which works in press-return already).
Never use detach() just to change the size of the screen, try_resize()
method exists for that specifically.
Diffstat (limited to 'test/functional/core/job_spec.lua')
| -rw-r--r-- | test/functional/core/job_spec.lua | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/test/functional/core/job_spec.lua b/test/functional/core/job_spec.lua index 9c37e55f42..d4ce690867 100644 --- a/test/functional/core/job_spec.lua +++ b/test/functional/core/job_spec.lua @@ -963,9 +963,6 @@ describe("pty process teardown", function() | ]]) end) - after_each(function() - screen:detach() - end) it("does not prevent/delay exit. #4798 #4900", function() if helpers.pending_win32(pending) then return end |