From 2ba539f449a95f38463a61b189e203a5fe306fc0 Mon Sep 17 00:00:00 2001 From: zeertzjq Date: Sat, 30 Apr 2022 13:11:35 +0800 Subject: fix(input): only disable mapped CTRL-C interrupts when getting input (#18310) --- test/functional/ex_cmds/ctrl_c_spec.lua | 32 ++++++++++++----- test/functional/legacy/mapping_spec.lua | 57 +++++++++++++++---------------- test/functional/vimscript/system_spec.lua | 45 +++++++++++++++++++++++- 3 files changed, 95 insertions(+), 39 deletions(-) (limited to 'test') diff --git a/test/functional/ex_cmds/ctrl_c_spec.lua b/test/functional/ex_cmds/ctrl_c_spec.lua index f19fab5550..c2e4bf0fb7 100644 --- a/test/functional/ex_cmds/ctrl_c_spec.lua +++ b/test/functional/ex_cmds/ctrl_c_spec.lua @@ -2,10 +2,15 @@ local helpers = require('test.functional.helpers')(after_each) local Screen = require('test.functional.ui.screen') local clear, feed, source = helpers.clear, helpers.feed, helpers.source local command = helpers.command +local sleep = helpers.sleep describe("CTRL-C (mapped)", function() + local screen + before_each(function() clear() + screen = Screen.new(52, 6) + screen:attach() end) it("interrupts :global", function() @@ -20,14 +25,6 @@ describe("CTRL-C (mapped)", function() ]]) command("silent edit! test/functional/fixtures/bigfile.txt") - local screen = Screen.new(52, 6) - screen:attach() - screen:set_default_attr_ids({ - [0] = {foreground = Screen.colors.White, - background = Screen.colors.Red}, - [1] = {bold = true, - foreground = Screen.colors.SeaGreen} - }) screen:expect([[ ^0000;;Cc;0;BN;;;;;N;NULL;;;; | @@ -56,4 +53,23 @@ describe("CTRL-C (mapped)", function() end end end) + + it('interrupts :sleep', function() + command('nnoremap ') + feed(':sleep 100') + -- wait for :sleep to start + sleep(10) + feed('foo') + -- wait for input buffer to be flushed + sleep(10) + feed('i') + screen:expect([[ + ^ | + ~ | + ~ | + ~ | + ~ | + -- INSERT -- | + ]]) + end) end) diff --git a/test/functional/legacy/mapping_spec.lua b/test/functional/legacy/mapping_spec.lua index 0f65d5eb65..552c26e7f2 100644 --- a/test/functional/legacy/mapping_spec.lua +++ b/test/functional/legacy/mapping_spec.lua @@ -2,7 +2,7 @@ local helpers = require('test.functional.helpers')(after_each) local clear, feed, insert = helpers.clear, helpers.feed, helpers.insert -local feed_command, expect, poke_eventloop = helpers.feed_command, helpers.expect, helpers.poke_eventloop +local expect, poke_eventloop = helpers.expect, helpers.poke_eventloop local command, eq, eval, meths = helpers.command, helpers.eq, helpers.eval, helpers.meths local sleep = helpers.sleep @@ -15,7 +15,7 @@ describe('mapping', function() ]]) -- Abbreviations with р (0x80) should work. - feed_command('inoreab чкпр vim') + command('inoreab чкпр vim') feed('GAчкпр ') expect([[ @@ -25,17 +25,15 @@ describe('mapping', function() it('Ctrl-c works in Insert mode', function() -- Mapping of ctrl-c in insert mode - feed_command('set cpo-=< cpo-=k') - feed_command('inoremap ') - feed_command('cnoremap dummy') - feed_command('cunmap ') + command('set cpo-=< cpo-=k') + command('inoremap ') + command('cnoremap dummy') + command('cunmap ') feed('GA') - feed('TEST2: CTRL-C |') + -- XXX: editor must be in Insert mode before is put into input buffer poke_eventloop() - feed('A|') - poke_eventloop() - feed_command('unmap ') - feed_command('unmap! ') + feed('TEST2: CTRL-C |A|') + command('unmap! ') expect([[ @@ -44,13 +42,12 @@ describe('mapping', function() end) it('Ctrl-c works in Visual mode', function() - feed_command([[vnoremap :$put ='vmap works']]) + command([[vnoremap :$put ='vmap works']]) feed('GV') - -- XXX: For some reason the mapping is only triggered - -- when is in a separate feed command. + -- XXX: editor must be in Visual mode before is put into input buffer poke_eventloop() - feed('') - feed_command('vunmap ') + feed('vV') + command('vunmap ') expect([[ @@ -59,23 +56,23 @@ describe('mapping', function() it('langmap', function() -- langmap should not get remapped in insert mode. - feed_command('inoremap { FAIL_ilangmap') - feed_command('set langmap=+{ langnoremap') + command('inoremap { FAIL_ilangmap') + command('set langmap=+{ langnoremap') feed('o+') -- Insert mode expr mapping with langmap. - feed_command('inoremap { "FAIL_iexplangmap"') + command('inoremap { "FAIL_iexplangmap"') feed('o+') -- langmap should not get remapped in cmdline mode. - feed_command('cnoremap { FAIL_clangmap') + command('cnoremap { FAIL_clangmap') feed('o+') - feed_command('cunmap {') + command('cunmap {') -- cmdline mode expr mapping with langmap. - feed_command('cnoremap { "FAIL_cexplangmap"') + command('cnoremap { "FAIL_cexplangmap"') feed('o+') - feed_command('cunmap {') + command('cunmap {') -- Assert buffer contents. expect([[ @@ -93,10 +90,10 @@ describe('mapping', function() ]]) -- Vim's issue #212 (feedkeys insert mapping at current position) - feed_command('nnoremap . :call feedkeys(".", "in")') + command('nnoremap . :call feedkeys(".", "in")') feed('/^a b') feed('0qqdw.ifooqj0@q') - feed_command('unmap .') + command('unmap .') expect([[ fooc d fooc d @@ -105,15 +102,15 @@ describe('mapping', function() it('i_CTRL-G_U', function() -- U works only within a single line - feed_command('imapclear') - feed_command('imap ( ()U') + command('imapclear') + command('imap ( ()U') feed('G2okiTest1: text with a (here some more textk.') -- test undo feed('G2okiTest2: text wit a (here some more text [und undo]uk.u') - feed_command('imapclear') - feed_command('set whichwrap=<,>,[,]') + command('imapclear') + command('set whichwrap=<,>,[,]') feed('G3o2k') - feed_command([[:exe ":norm! iTest3: text with a (parenthesis here\U\new line here\\\."]]) + command([[:exe ":norm! iTest3: text with a (parenthesis here\U\new line here\\\."]]) expect([[ diff --git a/test/functional/vimscript/system_spec.lua b/test/functional/vimscript/system_spec.lua index bedf7e5498..9cc6424d31 100644 --- a/test/functional/vimscript/system_spec.lua +++ b/test/functional/vimscript/system_spec.lua @@ -268,7 +268,7 @@ describe('system()', function() :call system("for /L %I in (1,0,2) do @echo y") |]] or [[ :call system("yes") |]])) - feed('') + feed('foo') screen:expect([[ ^ | ~ | @@ -286,6 +286,49 @@ describe('system()', function() Type :qa and press to exit Nvim | ]]) end) + + it('`yes` interrupted with mapped CTRL-C', function() + command('nnoremap i') + feed(':call system("' .. (iswin() + and 'for /L %I in (1,0,2) do @echo y' + or 'yes') .. '")') + screen:expect([[ + | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | +]] .. (iswin() + and [[ + :call system("for /L %I in (1,0,2) do @echo y") |]] + or [[ + :call system("yes") |]])) + feed('foo') + screen:expect([[ + ^ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + ~ | + -- INSERT -- | + ]]) + end) end) describe('passing no input', function() -- cgit