fix(jobwait): always drain process event queues #15402

Problem:
jobwait() returns early if the job was stopped, but the job might have
pending callbacks on its event queue which are required to complete its
teardown. State such as term->closed might not be updated yet (by the
pending callbacks), so codepaths such as :bdelete think the job is still
running.

Solution:
Always flush the job's event queue before returning from jobwait().

ref #15349

1 files changed, 14 insertions, 10 deletions

diff --git a/test/functional/terminal/buffer_spec.lua b/test/functional/terminal/buffer_spec.lua
index 16f7fe5178..c0af932e68 100644
--- a/test/functional/terminal/buffer_spec.lua
+++ b/test/functional/terminal/buffer_spec.lua
@@ -7,12 +7,7 @@ local eq, neq = helpers.eq, helpers.neq
 local write_file = helpers.write_file
 local command= helpers.command
 local exc_exec = helpers.exc_exec
-local retry = helpers.retry
-local funcs = helpers.funcs
-local pesc = helpers.pesc
 local matches = helpers.matches
-local nvim_dir = helpers.nvim_dir
-local iswin = helpers.iswin
 
 describe(':terminal buffer', function()
   local screen
@@ -261,16 +256,25 @@ describe(':terminal buffer', function()
     command('bdelete!')
   end)
 
-  it("requires bang (!) to close a running job", function()
-    local cwd = funcs.fnamemodify('.', ':p:~'):gsub([[[\/]*$]], '')
-    local ext_pat = iswin() and '%.EXE' or ''
+  it('requires bang (!) to close a running job #15402', function()
     eq('Vim(wqall):E948: Job still running', exc_exec('wqall'))
-    matches('^Vim%(bdelete%):E89: term://'..pesc(cwd)..'//%d+:'..nvim_dir..'/tty%-test'..ext_pat..' will be killed %(add %! to override%)$', exc_exec('bdelete'))
+    for _, cmd in ipairs({ 'bdelete', '%bdelete', 'bwipeout', 'bunload' }) do
+      matches('^Vim%('..cmd:gsub('%%', '')..'%):E89: term://.*tty%-test.* will be killed %(add %! to override%)$',
+        exc_exec(cmd))
+    end
     command('call jobstop(&channel)')
-    retry(nil, nil, function() assert(0 >= eval('jobwait([&channel], 1000)[0]')) end)
+    assert(0 >= eval('jobwait([&channel], 1000)[0]'))
     command('bdelete')
   end)
 
+  it('stops running jobs with :quit', function()
+    -- Open in a new window to avoid terminating the nvim instance
+    command('split')
+    command('terminal')
+    command('set nohidden')
+    command('quit')
+  end)
+
   it('does not segfault when pasting empty buffer #13955', function()
     feed_command('terminal')
     feed('<c-\\><c-n>')